vm_start_mutex use in IcedTeaNPPlugin.cc looks fishy

Deepak Bhole dbhole at redhat.com
Wed May 23 08:58:28 PDT 2012


Hi Martin,

* Martin Olsson <mnemo at minimum.se> [2012-05-23 11:44]:
> The usage of "vm_start_mutex" in IcedTeaNPPlugin.cc looks buggy to me.
> Won't each thread just create and lock it's own mutex, thereby voiding the
> raison d'être of the mutex?
>

Yep. I just noticed this a couple of days ago myself when trying to
investigate zombie VMs someone was seeing (though it was unrelated).

Haven't gotten around to creating a patch for it, but please feel free
to. If not, it is still on my list (though not critical since I have
never seen a bug due to it).

Cheers,
Deepak



More information about the distro-pkg-dev mailing list