On 03/30/2011 02:46 AM, ext Nicolas PENINGUY wrote:
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)

correct.

We can free pathlist also because it was made a copied to an already allocated buffer (strcpy(buf, *elem)).

         Tiago
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to