Bug 3631 - xinit might send SIGTERM to wrong process
Summary: xinit might send SIGTERM to wrong process
Status: CLOSED WORKSFORME
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Client (show other bugs)
Version: pre-1.0
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.15.0
Assignee: Bugzilla mail exporter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-20 11:25 CEST by Pierre Ossman
Modified: 2022-08-30 16:29 CEST (History)
0 users

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2010-10-20 11:25:58 CEST
When we pull down the session, we do a kill() on all sub-processes. The problem is that we have done waitpid() on these earlier, so if they've died for other reasons then we could possibly send signals to another process.

We might want to think about abstracting the process monitoring a bit so that we get all of it in one place and possibly also hide the differences between platforms.
Comment 1 Pierre Ossman cendio 2022-08-30 16:29:19 CEST
This got fixed when we replaced xinit with our own code in ThinLinc 3.2.0 (bug 357).

Note You need to log in before you can comment on or make changes to this bug.