[Webkit-unassigned] [Bug 262794] [WPE][GTK] Web process cache suspend/resume does not work, WebProcessProxy::processIdentifier is not the pid of the actual web process
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Dec 15 08:50:30 PST 2023
https://bugs.webkit.org/show_bug.cgi?id=262794
--- Comment #5 from Michael Catanzaro <mcatanzaro at redhat.com> ---
WebCore::processIdentifier is fine too.
I see a *lot* of things using WTF::ProcessID, but it's mostly used after calling getCurrentProcessID(), which is fine. The problem is specifically AuxiliaryProcessProxy::processID, which is where the bogus pids are coming from. Still, I doubt fixing this is realistic. We probably need to change flatpak. Is it really not possible for it to return results in the caller's pid namespace? :/
Until we are able to fix this, I think we should change ProcessLauncherGLib to set m_processID = 0 instead of setting the wrong pid. There are just too many possible bugs otherwise. This will either prevent or expose them all. E.g. the first one I found: ProcessLauncher::terminateProcess will currently only terminate the flatpak-spawn supervisor process and not the actual WebKit child process; presumably that will cause the WebKit process to leak, rather than terminate?
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20231215/58f45a06/attachment-0001.htm>
More information about the webkit-unassigned
mailing list