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