[Spread-users] spread 4.0rc2, C and multithreading

Jonathan Stanton jonathan at cnds.jhu.edu
Tue Mar 4 17:12:36 EST 2008

Good point. I do usually take patches like this and actually 'redo' them by searching for the 
common pattern and fixing all the instances of it I can find (as if the bug occured in one 
place it might be in other files also) so I should have caught the others. But I had not 
noticed the difference yet. 



On Tue, Mar 04, 2008 at 04:15:04PM -0500, John Robinson wrote:
> Jonathan Stanton wrote:
> >I took a look at this patch and it makes sense to me. Missing the unlock 
> >on the error path is clearly a bug. I'll apply it. 
> My read of the patch and source is that the patch only corrects 5 out of 
> the 8 places where this missing unlock pattern occurs.  (look for all 
> the errors: "Session disappeared on us, possible in threaded apps").
> Maybe you already noticed this.
> /jr
> _______________________________________________
> Spread-users mailing list
> Spread-users at lists.spread.org
> http://lists.spread.org/mailman/listinfo/spread-users

Jonathan R. Stanton         jonathan at cs.jhu.edu
Dept. of Computer Science   
Johns Hopkins University    

More information about the Spread-users mailing list