Race condition in tdb_runtime_check_for_robust_mutexes()

Jeremy Allison jra at samba.org
Wed Mar 23 16:57:08 UTC 2016


On Wed, Mar 23, 2016 at 01:02:48PM +0200, Uri Simchoni wrote:
> Here's a patch to fix the issue. I don't have confirmation that it
> fixes the issue but would like to continue the discussion on:
> a. Whether this is the right direction or should we eliminate the
> signal handler as Ralph suggested.
> b. The use of sigprocmask without #ifdefs, as opposed to installing
> and uninstalling the signal handler - it's hard as it is to get
> those things right, and I wonder whether we target envs which don't
> support sigprocmask. fork() hasn't #ifdefs around it ... :)

sigprocmask is used without #ifdefs in tevent, so I think
this is ok.

I'll try and get to review this once I'm done with
other stuff.

I won't push anything until I've gotten agreements
with you and Ralph that this is the right way to go.

Cheers,

	Jeremy.



More information about the samba-technical mailing list