On Thu, 2017-01-05 at 20:15 +0100, Bruno Haible wrote:
> Torvald Riegel wrote:
> > I found two bugs just by briefly scanning through the current
> > lib/glthread/lock.c.  As a hint, I'll just mention double-checked
> > locking and order of destruction.  And skipping destruction is of course
> > not correct in the general case either (unless I'm misreading
> > lib/glthread/cond.c, for example).
> 
> Can you give line numbers, please? lib/glthread/lock.c and lib/glthread/cond.c
> are large files. If you have found bugs there, *of course* I want to have
> them fixed.

I can understand both points.  However, I'd suggest that you first try
to find them (or that whoever maintains these files does); this is often
a good exercise, and helps make oneself sensitive for concurrency
problems.  If you don't want to do that first, I can give details.



Reply via email to