Bug 1560 - client sometimes warns about broken pipe
Summary: client sometimes warns about broken pipe
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: VNC (show other bugs)
Version: trunk
Hardware: PC Linux
: P2 Normal
Target Milestone: 4.14.0
Assignee: William Sjöblom
URL:
Keywords: prosaic, samuel_tester
Depends on: 7634
Blocks:
  Show dependency treegraph
 
Reported: 2005-10-07 16:47 CEST by Pierre Ossman
Modified: 2021-11-25 14:08 CET (History)
2 users (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2005-10-07 16:47:48 CEST
When Xvnc gets terminated it kills of the VNC connection with complete disregard
for the protocol. With some bad timing this causes the client to detect a
problem with the protocol stream and displays an error message to the user.

We should try and fix Xvnc so it shuts down more gracefully and terminates the
VNC connection without causing protocol errors.
Comment 1 Peter Åstrand cendio 2009-01-22 08:48:14 CET
In revision 16364, we have disabled the error dialogs. 
Comment 2 Pierre Ossman cendio 2014-10-29 13:40:54 CET
Got a dialog with broken pipe from the Linux client today. This is in the log:

2014-10-29T13:38:51: vncviewer[E]: Wed Oct 29 13:38:51 2014
2014-10-29T13:38:51: vncviewer[E]:  Viewport:    write: Broken pipe (32)
2014-10-29T13:38:51: vncviewer[E]:  CConn:       write: Broken pipe (32)
Comment 3 Pierre Ossman cendio 2015-04-23 11:08:40 CEST
I was strace:ing Xvnc and noticed that it does not do shutdown() or close() on the VNC sockets. So that might be a good start and see if it solves the issue.
Comment 4 Pierre Ossman cendio 2018-12-11 13:37:26 CET
I don't think this is a server issue really. The client might be doing writes at any time, regardless of what it receives. So I think this might be a client issue.

We need more investigation here. The key issue is that we are getting nasty errors that we show to the user.
Comment 6 William Sjöblom cendio 2021-11-24 08:51:07 CET
This error dialog should now be suppressed as of the fixes for bug 289. Marking as resolved.
Comment 8 Samuel Mannehed cendio 2021-11-25 14:08:48 CET
Tested as part of bug 289.

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