Re: [evolution-patches] crash at startup, 41448



Looks good. Probably could do without the g_malloc0() in
camel_url_copy() since all the fields are immediately set. The only one
you would have to init to NULL would be the new->params field.

then again, it's not all that important.

Jeff

On Tue, 2003-04-22 at 05:05, Not Zed wrote:
> Patch fixes a race at startup.
> 
> I'm pretty sure this fix will fix it, the new test program included in
> the patch would crash straight away before the code was changed, in the
> same places as 41448 and/or 41629, and others.
> 
> I've seen it before, but its hard to recreate in the main application. 
> And although i knew that race was present, i didn't think it would cause
> the crash, but it seemed to afterall.
-- 
Jeffrey Stedfast
Evolution Hacker - Ximian, Inc.
fejj ximian com  - www.ximian.com




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]