Bug 115692 - net/rdesktop(1) hangs in a tight loop upon sound device close
Summary: net/rdesktop(1) hangs in a tight loop upon sound device close
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: David E. O'Brien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-21 18:30 UTC by Eugene M. Kim
Modified: 2007-08-24 07:11 UTC (History)
0 users

See Also:


Attachments
patch-rdpsnd_oss.c (1.31 KB, text/plain)
2007-08-21 18:30 UTC, Eugene M. Kim
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene M. Kim 2007-08-21 18:30:01 UTC
Upon server-initiated closure of the sound device, rdesktop(1) hangs in
a tight loop if the sound packet buffer is not empty.  This usually
occurs after a long playback, and is indicated by an endless stream of
"ERROR: select: Bad file descriptor" messages on stderr.

Fix: Apply the following patch, which forcibly empties the sound packet buffer
after closing the sound device.
How-To-Repeat: Run rdesktop with -r sound:local, and play an MP3 file for about 30
seconds then stop playback.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2007-08-22 01:08:40 UTC
Responsible Changed
From-To: freebsd-ports-bugs->obrien

Over to maintainer
Comment 2 David E. O'Brien freebsd_committer freebsd_triage 2007-08-23 21:42:07 UTC
State Changed
From-To: open->feedback

Hi Eugene, 
Have you submitted this patch to the rdesktop project on SourceForge? 
If so, I'll commit it to the FreeBSD port.  Otherwise, could you please 
report the bug?  I'm a little worried your patch is large enough that 
it could make the next upgrade harder. 
-- David
Comment 3 Eugene M. Kim 2007-08-23 23:35:48 UTC
Yes, there was already a bug filed for this, to which I submitted the 
same patch.

https://sourceforge.net/tracker/?func=detail&atid=381347&aid=1695146&group_id=24366

is the bug report.

Thank you,
Eugene
Comment 4 dfilter service freebsd_committer freebsd_triage 2007-08-24 00:54:04 UTC
obrien      2007-08-23 23:53:59 UTC

  FreeBSD ports repository

  Added files:
    net/rdesktop/files   patch-rdpsnd_oss.c 
  Log:
  Upon server-initiated closure of the sound device, rdesktop(1) hangs in
  a tight loop if the sound packet buffer is not empty.  This usually
  occurs after a long playback, and is indicated by an endless stream of
  "ERROR: select: Bad file descriptor" messages on stderr.
  
  Fix: forcibly empty the sound packet buffer after closing the sound device.
  
  PR:             115692
  Submitted by:   "Eugene M. Kim" <freebsd.org@ab.ote.we.lv>
  
  Revision  Changes    Path
  1.1       +57 -0     ports/net/rdesktop/files/patch-rdpsnd_oss.c (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 5 David E. O'Brien freebsd_committer freebsd_triage 2007-08-24 07:11:11 UTC
State Changed
From-To: feedback->closed

Submitter sent patch up-stream, and I committed to /usr/ports.