php-imlib

[Artur Frysiak]

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.

>     sprintf(s, SYS_LOADERS_PATH "/filter");
>  #ifndef __EMX__
>     l = __imlib_FileDir(s, &num);
> @@ -174,7 +174,6 @@
>        }
>        __imlib_FileFreeDirList(l, num);
>     }
> -   free(home);
>     free(s);

Pozdrawiam
-- 
Artur Frysiak
http://www.pld-linux.org/