php-imlib

[Michal Moskal]

On Thu, Feb 27, 2003 at 10:33:40AM +0100, Artur Frysiak wrote:
> On Thu, Feb 27, 2003 at 02:02:33AM +0100, Piotr Pawłow wrote:
> > > Zamiast robić tego define wystarczy:
> > > } __attribute__((packed)) tga_footer;
> > 
> > OK, niniejszym patch będzie wyglądać następująco:
> > 
> > diff -urN imlib2-1.0.6.orig/src/dynamic_filters.c imlib2-1.0.6/src/dynamic_filters.c
> > --- imlib2-1.0.6.orig/src/dynamic_filters.c     Wed Apr 10 21:11:54 2002
> > +++ imlib2-1.0.6/src/dynamic_filters.c  Wed Feb 26 13:30:16 2003
> > @@ -146,12 +146,12 @@
> >  /* loader dir */
> >  char **__imlib_ListFilters(int *num_ret)
> >  {
> > -   char **list = NULL, **l, *s, *home;
> > +   char **list = NULL, **l, *s;
> >     int num, i, pi = 0;
> > 
> >     *num_ret = 0;
> >     /* same for system loader path */
> > -   s = (char *) realloc(s, sizeof(SYS_LOADERS_PATH) + 7 + 1);
> > +   s = (char *) malloc(sizeof(SYS_LOADERS_PATH) + 7 + 1);
> 
> I to spododuje memleaka, ponieważ jest to wnętrze pętli for.
> Lepiej zrobić "char **list = NULL, **l, *s = NULL;" pare linijek wyżej.

To jest poza pętlą, dalej jest realloc w pętli.

-- 
: Michal Moskal ::::: malekith/at/pld-linux.org :  GCS {C,UL}++++$ a? !tv
: PLD Linux ::::::: Wroclaw University, CS Dept :  {E-,w}-- {b++,e}>+++ h