2011/12/1 Keith Packard <kei...@keithp.com>:
> @@ -852,6 +852,10 @@ OpenConfigDir(const char *path, const char *cmdline, 
> const char *projroot,
>
>                /* match files named *.conf */
>                num = scandir(dirpath, &list, ConfigFilter, alphasort);
> +               if (num < 0) {
> +                       num = 0;
> +                       list = NULL;
> +               }
>                found = AddConfigDirFiles(dirpath, list, num);
>                if (!found) {
>                        free(dirpath);

Well, that solution would depend on AddConfigDirFiles behaving nicely
when called with list=NULL and num=0, which, from a certain point of
view, goes against your first comment on my initial patch.

Anyway, I'd be happy with any of the 3 solutions, since they all fix the bug.

>
> --
> keith.pack...@intel.com



-- 
Paulo Zanoni
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to