Ubuntu
iperf package

sync request - iperf fails to die when bind fails

Bug #331535 reported by Dinger on 2009-02-19

Affects		Status	Importance	Assigned to	Milestone
	iperf (Debian)	Fix Released	Unknown	debbugs #517239
	iperf (Ubuntu)	Fix Released	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

See original description

Revision history for this message

Dinger (denyipanyany) wrote on 2009-02-23:

Patch which fixes the issue, at least for this bug. Edit (476 bytes, text/plain)

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 on 2009-02-23:

corrected patch Edit (486 bytes, text/plain)

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 on 2009-03-12:

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 on 2009-03-12:

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 on 2009-03-12:

iperf_2.0.4-3ubuntu1.debdiff Edit (1.6 KiB, text/plain)

Brian Murray (brian-murray) on 2009-03-12

Changed in iperf:
importance:	Undecided → Unknown
status:	New → Unknown

Bug Watch Updater (bug-watch-updater) on 2009-03-12

Changed in iperf:
status:	Unknown → Fix Released

Brian Murray (brian-murray) on 2009-03-16

description:	updated
description:	updated

Revision history for this message

Iulian Udrea (iulian) wrote on 2009-03-16:

Acknowledged.

Changed in iperf:
status:	Triaged → Confirmed

Revision history for this message

Dustin Kirkland  (kirkland) wrote on 2009-03-18:

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].