Re: memory leak in psiconv

Subject: Re: memory leak in psiconv
frodol@dds.nl
Date: Tue Jan 29 2002 - 12:56:20 CST

• sorted by: [ date ] [ thread ] [ subject ] [ author ]
• Next message: Hubert Figuiere: "Re: memory leak in psiconv"
• Previous message: Hubert Figuiere: "Re: parsons down"
• Maybe in reply to: Tomas Frydrych: "memory leak in psiconv"
• Maybe reply: frodol@dds.nl: "Re: memory leak in psiconv"

I wrote:
> Tomas Frydrych wrote:
> >
> >
> > I found a memory leak in psiconv; the attached patch might fix that,
> > but the question is whether the missing free should not be located
> > in psiconv_list_free() instead of psiconv_buffer_free(); I will leave
> > that to Frodo to decide.
>
> You are right; there is a glaring, stupid error in psiconv_list_free.
> The right patch is:
[...]

I've installed dmalloc and the psiconv library itself should now be
without any obvious memory leaks (well, I hope...). I'll resync the
AbiWord-distributed version one of these days.

Thanks,
  Frodo

-- 
Frodo Looijaard <frodol@dds.nl>  PGP key and more: http://huizen.dds.nl/~frodol
Defenestration n. (formal or joc.):
  The act of removing Windows from your computer in disgust, usually followed
  by the installation of Linux or some other Unix-like operating system.

• Next message: Hubert Figuiere: "Re: memory leak in psiconv"
• Previous message: Hubert Figuiere: "Re: parsons down"
• Maybe in reply to: Tomas Frydrych: "memory leak in psiconv"
• Maybe reply: frodol@dds.nl: "Re: memory leak in psiconv"

This archive was generated by hypermail 2b25 : Wed Jan 30 2002 - 09:32:48 CST