Review Request: Do not use the port number from /etc/services in sftpProtocol::setHost
Andreas Schneider
asn at cryptomilk.org
Mon Apr 16 15:53:23 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104617/#review12512
-----------------------------------------------------------
Ship it!
Thanks for all the work you're doing here.
- Andreas Schneider
On April 16, 2012, 2:06 p.m., Dawit Alemayehu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104617/
> -----------------------------------------------------------
>
> (Updated April 16, 2012, 2:06 p.m.)
>
>
> Review request for KDE Runtime and Andreas Schneider.
>
>
> Description
> -------
>
> The attached patch does the following:
> - Removed code that set the port number to the one specified in /etc/services since it is completely the wrong thing to do for a client application.
> - Cleaned up header includes.
>
> Using the port number from /etc/services as a default is a common mistake that was fixed in all the other protocols a long while back. kio_sftp should not do that because if the port number is changed in /etc/services to something else other than port# 22, then it would be impossible for the user to login into another ssh server without explicitly specifying the port number as part of the url.
>
>
> Diffs
> -----
>
> kioslave/sftp/kio_sftp.h ca2b4ac
> kioslave/sftp/kio_sftp.cpp f8591f8
>
> Diff: http://git.reviewboard.kde.org/r/104617/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Dawit Alemayehu
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120416/5144c878/attachment.htm>
More information about the kde-core-devel
mailing list