Stefan Sperling wrote:
> On Wed, Jan 21, 2009 at 11:11:59PM +0100, Bert Huijben wrote:
>>
>>> -----Original Message-----
>>> From: Stefan Sperling [mailto:stsp_at_elego.de]
>>> Sent: Wednesday, January 21, 2009 11:06 PM
>>> To: Greg Stein
>>> Cc: Hyrum K. Wright; dev_at_subversion.tigris.org
>>> Subject: Re: segfault in 'svn up' on trunk
>>>
>>> On Wed, Jan 21, 2009 at 04:02:54PM +0100, Greg Stein wrote:
>>>> 0x69737265 looks like ascii characters: i s r e
>>>>
>>>> Somehow the path pointer is getting munged by some text?
>>> We need to audit that code more.
>> Note that this was just a dangling pointer in a svn_wc_notify_t structure
>> that wasn't initialized with apr_pcalloc as I expected, but with apr_palloc.
>> See r35366.
>
> Thanks!
>
> Maybe we should also do an apr_palloc -> apr_pcalloc sweep?
+1. I know, I know, I know that careful coders shouldn't need to force a
memset(0) on all their allocations. But sometimes we just aren't perfectly
careful coders, and the cost of such runtime crutches approaches zero daily
(and has already, in my opinion, dipped well below the threshold of
user-visibility).
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1043240
Received on 2009-01-22 16:11:26 CET