On 10/04/2012 09:55 PM, stef...@apache.org wrote: > Author: stefan2 > Date: Fri Oct 5 01:54:59 2012 > New Revision: 1394346 > > URL: http://svn.apache.org/viewvc?rev=1394346&view=rev > Log: > * subversion/libsvn_ra_svn/client.c > (ra_svn_commit): initialize *all* members of the ccb struct > > Modified: > subversion/trunk/subversion/libsvn_ra_svn/client.c > > Modified: subversion/trunk/subversion/libsvn_ra_svn/client.c > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_svn/client.c?rev=1394346&r1=1394345&r2=1394346&view=diff > ============================================================================== > --- subversion/trunk/subversion/libsvn_ra_svn/client.c (original) > +++ subversion/trunk/subversion/libsvn_ra_svn/client.c Fri Oct 5 01:54:59 > 2012 > @@ -1044,6 +1044,7 @@ static svn_error_t *ra_svn_commit(svn_ra > ccb = apr_palloc(pool, sizeof(*ccb)); > ccb->sess_baton = sess_baton; > ccb->pool = pool; > + ccb->new_rev = NULL; > ccb->callback = callback; > ccb->callback_baton = callback_baton;
We pretty commonly use apr_pcalloc() for exactly this reason. Yeah, it may cost an extra couple of nanoseconds, but... -- C. Michael Pilato <cmpil...@collab.net> CollabNet <> www.collab.net <> Enterprise Cloud Development
signature.asc
Description: OpenPGP digital signature
