This is an archive of the discontinued LLVM Phabricator instance.

Don't close psedo terminal master file descriptor on EOF
ClosedPublic

Authored by tberghammer on Mar 19 2015, 6:57 AM.

Details

Summary

Don't close pseudo terminal master file descriptor on EOF

Some application on Linux an all application on android close stdout and
stderr during the libc exit stage. Previously the master file descriptor
of the pseudo terminal used to communicate with the inferior was closed
on an EOF causing a race condition and a possible SIGHUP on process
exit. After this change the master file descriptor will be closed by the
destructor of the GDBRemoteCommunicationServerLLGS class.

Diff Detail

Repository
rL LLVM

Event Timeline

tberghammer retitled this revision from to Don't close psedo terminal master file descriptor on EOF.
tberghammer updated this object.
tberghammer edited the test plan for this revision. (Show Details)
tberghammer added reviewers: labath, ovyalov.
tberghammer added a subscriber: Unknown Object (MLST).
labath accepted this revision.Mar 19 2015, 7:33 AM
labath edited edge metadata.

lgtm

This revision is now accepted and ready to land.Mar 19 2015, 7:33 AM
This revision was automatically updated to reflect the committed changes.