[PATCH 06/10] xfree86: loader: fix memory leaks in LoaderListDirs

Nicolas PENINGUY nico at lostgeeks.org
Tue Mar 29 16:46:12 PDT 2011


On Tue, 2011-03-29 at 18:12 +0300, Tiago Vignatti wrote:

> +++ b/hw/xfree86/loader/loadmod.c

> @@ -556,6 +558,9 @@ LoaderListDirs(const char **subdirlist, const char **patternlist)
>      }
>      if (listing)
>  	listing[n] = NULL;
> +
> +    FreeSubdirs(subdirs);
> +    FreePatterns(patterns);
>      return listing;
>  }

Forgot FreePathList(pathlist); ? (And Peter's comment makes sense also)

Anyway for the patch series :

Reviewed-by: Nicolas Peninguy <nico at lostgeeks.org>




More information about the xorg-devel mailing list