D6711: Check for HUP/ERR/NVAL conditions on stdin

Fabian Vogt noreply at phabricator.kde.org
Sat Jul 15 12:05:49 UTC 2017


fvogt created this revision.
Restricted Application added a project: Plasma.

REVISION SUMMARY
  While Qt's event loop checks for POLLHUP, it
  treats it exactly like POLLIN and it's not possible
  to retrieve that information from Qt. So in the
  Connection read handler it needs to be checked for
  again.

TEST PLAN
  Before this patch plasma-browser-integration-host
  consumed 100% CPU after exiting the browser, 100% reproducible.
  Now it quits gracefully.

REPOSITORY
  R856 Plasma Browser Integration

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D6711

AFFECTED FILES
  host/connection.cpp

To: fvogt, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170715/21443c8e/attachment.html>


More information about the Plasma-devel mailing list