Re: [PATCH] initialize struct members

On Wed, Sep 10, 2008 at 08:01:12PM +0200, Lieven Govaerts wrote:
> Stefan Küng wrote:
> > Hi,
> >
> > Before I commit this, maybe someone likes to first review it:
> > this patch initializes all struct members in the functions
> > svn_wc_conflict_description_create_text() and
> > svn_wc_conflict_description_create_prop().
> > Found this due to crashes in TSVN - it tried to access the merged_file
> > member which had a non-NULL value but pointed to bogus memory.
> >
> > If nobody objects, I'll commit this in about an hour (shouldn't be much
> > to object to IMHO).
> >
> Using apr_pcalloc has the same effect, but slightly easier for the eyes.

Oops, good point, actually.

Stefan (you, not me :), do you want to make a follow-up commit that
uses apr_pcalloc() instead?

Thanks,
Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-10 20:10:52 CEST