Re: scripts/common.c minor memory leak

From: "Walter Cruz" <walter(dot)php(at)gmail(dot)com>
To: "Guido Barosio" <gbarosio(at)gmail(dot)com>
Cc: "PostgreSQL Hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: scripts/common.c minor memory leak
Date: 2006-10-04 02:33:39
Message-ID: 32cabba0610031933j4e39751amb33314c019b12ae8@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

http://en.wikipedia.org/wiki/Coverity

[]'s
- Walter

On 10/3/06, Guido Barosio <gbarosio(at)gmail(dot)com> wrote:
>
> A good spot :)
>
> Sorry for being no{isy}{vice}{wbie}, but what does it means "found by
> coverity" ?
>
> g.-
>
> On 10/3/06, Martijn van Oosterhout <kleptog(at)svana(dot)org> wrote:
> > Just a minor thing. In yesno_prompt(), the value is resp is allocated
> > memory that is never freed.
> >
> > File: src/bin/scripts/common.c
> > Line: 218
> >
> > Not terribly important though, it's not used in critical utilities, but
> > it's used often.
> >
> > Found by coverity.
> > --
> > Martijn van Oosterhout <kleptog(at)svana(dot)org> http://svana.org/kleptog/
> > > From each according to his ability. To each according to his ability
> to litigate.
> >
> >
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.1 (GNU/Linux)
> >
> > iD8DBQFFItO3IB7bNG8LQkwRAk2bAKCB3AKuon35YAMOEjixN7P9HQeoBgCeOWpF
> > 5/GvkeMtRwlncGBP9MG2qXw=
> > =FBVw
> > -----END PGP SIGNATURE-----
> >
> >
> >
>
>
> --
> Guido Barosio
> -----------------------
> http://www.globant.com
> guido(dot)barosio(at)globant(dot)com
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-10-04 05:11:18 Re: putting CHECK_FOR_INTERRUPTS in qsort_comparetup()
Previous Message Guido Barosio 2006-10-04 02:28:10 Re: scripts/common.c minor memory leak

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-10-04 05:11:18 Re: putting CHECK_FOR_INTERRUPTS in qsort_comparetup()
Previous Message Guido Barosio 2006-10-04 02:28:10 Re: scripts/common.c minor memory leak