remote/client: fix forwarding of remote ports

'labgrid-client forward --remote' accidentally uses the variable
'localport', which is either not set at all (leading to a
UnboundLocalError) if no --local/-L is set or is set to the LOCAL part
of --local/-L, which is wrong. The LOCAL part of --remote/-R should be
used instead.

Fix that by using the correct variable 'local'.

Note that the informational print used the correct variable already.

Fixes: 51f3fe5 ("client: Add port forwarding")
Signed-off-by: Bastian Krause <[email protected]>

Author: Bastian-Krause

CHANGES.rst

@@ -9,6 +9,9 @@ Bug fixes in 0.4.1
 
 - Fixed a bug where using ``labgrid-client io get`` always returned ``low``
   when reading a ``sysfsgpio``.
+- Fixed ``labgrid-client forward --remote``/``-R``, which used either the LOCAL
+  part of ``--local``/``-L`` accidentally (if specified) or raised an
+  UnboundLocalError.
 
 Known issues in 0.4.1
 ~~~~~~~~~~~~~~~~~~~~~~~~~

labgrid/remote/client.py

@@ -1116,7 +1116,7 @@ def forward(self):
                 print(f"Forwarding local port {localport:d} to remote port {remote:d}")
 
             for local, remote in self.args.remote:
-                stack.enter_context(drv.forward_remote_port(remote, localport))
+                stack.enter_context(drv.forward_remote_port(remote, local))
                 print(f"Forwarding remote port {remote:d} to local port {local:d}")
 
             try: