Tunnel Manager

dynamic port doesn't work if name is given

Bug #851099 reported by Francesco Apollonio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Tunnel Manager
Fix Released
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.

Revision history for this message
Francesco Apollonio (lorddex) wrote :
Francesco Apollonio (lorddex)
Changed in tunnelmanager:
status: New → Confirmed
Revision history for this message
Brandon Williams (opensource-subakutty) wrote :

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)
Revision history for this message
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!

Brandon Williams (opensource-subakutty)
Changed in tunnelmanager:
status: Confirmed → Fix Committed
Revision history for this message
Brandon Williams (opensource-subakutty) wrote :

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.

Revision history for this message
Brandon Williams (opensource-subakutty) wrote :

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  
Everyone can see this information.
Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

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