Review Request 109748: kioslave/sftp: Show correct port number when connecting to default port

Commit Hook null at
Mon Apr 1 17:38:48 BST 2013

This is an automatically generated e-mail. To reply, visit:

(Updated April 1, 2013, 4:38 p.m.)


This change has been marked as submitted.

Review request for KDE Runtime.


Currently, if you make a sftp connection through the kio slave, this slave shows port number 0 to the user in the status bar (and as debug output) instead of the ssh port number 22. This is purely an UI issue, as internally the port number is tested on >0 using the default port number as a fall back if the test fails.
The attached patch address this issue by testing for port number 0. If the port number is invalid, the default port number is shown instead.


  kioslave/sftp/kio_sftp.cpp f493477 




Thomas Fischer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list