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