sync request - iperf fails to die when bind fails

Bug #331535 reported by Dinger
6
Affects Status Importance Assigned to Milestone
iperf (Debian)
Fix Released
Unknown
iperf (Ubuntu)
Medium
Dustin Kirkland 

Bug Description

Binary package hint: iperf

When iperf is ran as an normal user, and you tell it to listen on a privileged port (anything below 1024), it should die when the bind fails. Instead, it acts as if its still listening on the port you specified.

bwindle@wh-mon06:~$ apt-cache policy iperf
iperf:
  Installed: 2.0.4-1
  Candidate: 2.0.4-1
  Version table:
 *** 2.0.4-1 0
        500 http://us.archive.ubuntu.com intrepid/universe Packages
        100 /var/lib/dpkg/status

bwindle@wh-mon06:~$ iperf -s -p 500
bind failed: Permission denied
------------------------------------------------------------
Server listening on TCP port 500
TCP window size: 85.3 KByte (default)
------------------------------------------------------------

----
Sync request of iperf version 2.0.4-4 from Debian unstable.

 iperf (2.0.4-4) unstable; urgency=low

   * Die instead of warn on bind/connect fatal errors (thanks to Deny IP Any
     Any). (closes: #517239, #518033)
   * Add missing comma after ${shlibs:Depends} in the control file. Add
     Homepage in control file. (thanks to lintian).

 -- Roberto Lumbreras <email address hidden> Wed, 04 Mar 2009 12:46:12 +0100

Revision history for this message
Dinger (denyipanyany) wrote :

I've created a patch which fixes this bug.

According to a 4-year old post on the 'net (http://archive.ncsa.uiuc.edu/lists/iperf-users/jun05/msg00005.html), this has been around a while.

Revision history for this message
Dinger (denyipanyany) wrote :

woops, I had my diff backwards.. should be removing the WARN_errno and replacing it with a FAIL_errno.

Revision history for this message
Brian Murray (brian-murray) wrote :

I've tested this patch using iperf version 2.0.4-3 from Jaunty Jackalope and it works as advertised. The iperf package is actually synced from Debian so the patch should be forwarded to them.

Changed in iperf:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Dinger (denyipanyany) wrote :

This issue has already been fixed in Debian's iperf_2.0.4-4_i386.deb package as of 04 Mar 2009; bug IDs were 517239 and 518033.

Thank you for the follow-up.

Revision history for this message
Brian Murray (brian-murray) wrote :
Changed in iperf:
importance: Undecided → Unknown
status: New → Unknown
Changed in iperf:
status: Unknown → Fix Released
description: updated
description: updated
Revision history for this message
Iulian Udrea (iulian) wrote :

Acknowledged.

Changed in iperf:
status: Triaged → Confirmed
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

lp_archive@cocoplum:~/syncs$ sync-source.py -b iulian iperf
Getting binaries for jaunty...
[Updating] iperf (2.0.4-3 [Ubuntu] < 2.0.4-4 [Debian])
 * Trying to add iperf...
  - <iperf_2.0.4-4.dsc: downloading from http://ftp.debian.org/debian/>
  - <iperf_2.0.4-4.diff.gz: downloading from http://ftp.debian.org/debian/>
  - <iperf_2.0.4.orig.tar.gz: already in distro - downloading from librarian>
I: iperf [universe] -> iperf_2.0.4-3 [universe].

Changed in iperf (Ubuntu):
assignee: nobody → kirkland
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.