On Fri, 2011-03-25 at 20:41 +0200, Tiago Vignatti wrote: > diff --git a/hw/xfree86/common/xf86Init.c b/hw/xfree86/common/xf86Init.c > index e664ce4..d81303e 100644 > --- a/hw/xfree86/common/xf86Init.c > +++ b/hw/xfree86/common/xf86Init.c > @@ -1433,6 +1433,7 @@ xf86LoadModules(char **list, pointer *optlist) > } > free(name); > } > + free(name); > return !failed; > }
This one seems wrong to me, the only leak I can see is when *name == '\0'. This patch might lead to double free I think. Nicolas _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
