Re: SetVariable - Mailing list pgsql-hackers

From Gaetano Mendola
Subject Re: SetVariable
Date
Msg-id 000a01c370d9$43f500d0$10d4a8c0@mm.eutelsat.org
Whole thread Raw
In response to Re: SetVariable  ("Mendola Gaetano" <mendola@bigfoot.com>)
List pgsql-hackers
"Bruce Momjian" <pgman@candle.pha.pa.us> wrote:
> Mendola Gaetano wrote:
> > "Bruce Momjian" <pgman@candle.pha.pa.us> wrote:
> > > Gaetano Mendola wrote:
> > > > "Bruce Momjian" <pgman@candle.pha.pa.us> wrote:
> > > > > I see other strdup() calls that don't check on a return.  Should
we
> > deal
> > > > > with those too?
> > > >
> > > > Well strdup obtain the memory for the new string using a malloc
> > > > and normally is a good habit check the return value of a malloc.
> > >
> > > Right.  My point is that we have lots of other strdup's in the code.
> > > Should we fix those too?  Seems we should be consistent.
> >
> > Shall I post the patch for SetVariable ?
> >
> > I know that this change is not so important but I want to try
> > on my skin the cycle
> > seen somethink wrong => send patch => be applyed
>
> Sure.
>
> It would be good if you would evaluate all the strdups, find the ones
> that don't check properly, and submit a big patch of all those.  The fix
> can be to call xstrdup, or to check the return code.

I will.

Regards
Gaetano Mendola




pgsql-hackers by date:

Previous
From: Andreas Pflug
Date:
Subject: Re: TCP/IP with 7.4 beta2 broken?
Next
From: "Gaetano Mendola"
Date:
Subject: Re: Unixware Patch (Was: Re: Beta2 Tag'd and Bundled ...)