dynamic port doesn't work if name is given

Bug #851099 reported by Francesco Apollonio on 2011-09-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Tunnel Manager
Undecided
Brandon Williams

Bug Description

If you start a tunnel with a dynamic port it will crash saying:
-----
Unexpected termination of tunnel: test

Pseudo-terminal will not be allocated because stdin is not a terminal.
bind: Cannot assign requested address
channel_setup_fwd_listener: cannot listen to port: 1233

Could not request local forwarding.
-----

The problem is that calling PortInfo.DynamicPort you have swapped the two arguments: name instead of IP.

Francesco Apollonio (lorddex) wrote :
Changed in tunnelmanager:
status: New → Confirmed

Thank you for the bug report.

The problem only occurs if you specify a name for the port definition. This is not required (which is why I haven't noticed it myself). As a work-around, simply remove the name from the port definition.

summary: - A tunnel with a dynamic port doesn't start
+ dynamic port doesn't work if name is given
Changed in tunnelmanager:
assignee: nobody → Brandon Williams (opensource-subakutty)
Francesco Apollonio (lorddex) wrote :

But in PortInfo.py the constructor is:
                def __init__(self, name, bindIP, localPort)
and in GUI.py the class is created using:
                newPort = PortInfo.DynamicPort(bindIPEntry.get_text(),
                                               nameEntry.get_text(),
so the two parameters are actually swapped.
Now i'm using a version patched with this edit and it works right!

Changed in tunnelmanager:
status: Confirmed → Fix Committed

fix committed in revision 66

I agree that the proposed work-around doesn't help. It's actually _both_ name and bindip that have to be left out. Neither one is required, and most users won't need them. I'll push new packages to my PPA some time soon, but this may help until then.

fix released in version 0.7.4

Changed in tunnelmanager:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments