When the array gets down to size zero (which it does in later patches), gcc complains that the index is out of bounds. Avoid this by using ARRAY_SIZE on extensionModules instead.
Signed-off-by: Daniel Stone <[email protected]> --- hw/xfree86/dixmods/extmod/modinit.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/hw/xfree86/dixmods/extmod/modinit.c b/hw/xfree86/dixmods/extmod/modinit.c index afee27a..009852c 100644 --- a/hw/xfree86/dixmods/extmod/modinit.c +++ b/hw/xfree86/dixmods/extmod/modinit.c @@ -101,12 +101,6 @@ static ExtensionModule extensionModules[] = { NULL, NULL}, #endif - { /* DON'T delete this entry ! */ - NULL, - NULL, - NULL, - NULL, - NULL} }; static XF86ModuleVersionInfo VersRec = { @@ -134,7 +128,7 @@ extmodSetup(pointer module, pointer opts, int *errmaj, int *errmin) /* XXX the option stuff here is largely a sample/test case */ - for (i = 0; extensionModules[i].name != NULL; i++) { + for (i = 0; i < ARRAY_SIZE(extensionModules); i++) { #ifdef XSELINUX if (!strcmp(SELINUX_EXTENSION_NAME, extensionModules[i].name)) { pointer o; -- 1.7.10.4 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
