Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 19 Feb 2014 10:27:01 GMT
From:      Matthew Seaman <matthew@FreeBSD.org>
To:        FreeBSD-gnats-submit@freebsd.org
Subject:   ports/186889: ftp/proftpd -- fix for inoperative scp mode
Message-ID:  <201402191027.s1JAR1u2072504@stingray.adestra.com>
Resent-Message-ID: <201402191100.s1JB00bS084295@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         186889
>Category:       ports
>Synopsis:       ftp/proftpd -- fix for inoperative scp mode
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Feb 19 11:00:00 UTC 2014
>Closed-Date:
>Last-Modified:
>Originator:     Matthew Seaman
>Release:        FreeBSD 9.2-RELEASE-p3 amd64
>Organization:
Adestra Ltd
>Environment:
System: FreeBSD stingray.adestra.com 9.2-RELEASE-p3 FreeBSD 9.2-RELEASE-p3 #0: Sat Jan 11 03:25:02 UTC 2014 root@amd64-builder.daemonology.net:/usr/obj/usr/src/sys/GENERIC amd64


	
>Description:

We had some customer complaints that any attempt to download a file
via SCP, while apparently successful, resulted in a zero length file
locally.

This patch fixes the problem.  It is derived from the proftpd
bugzilla:

http://bugs.proftpd.org/show_bug.cgi?id=3954

>How-To-Repeat:
	
>Fix:

	

--- proftpd.diff begins here ---
Index: files/patch-contrib_mod__sftp_scp.c
===================================================================
--- files/patch-contrib_mod__sftp_scp.c	(revision 0)
+++ files/patch-contrib_mod__sftp_scp.c	(working copy)
@@ -0,0 +1,20 @@
+
+$FreeBSD$
+
+--- contrib/mod_sftp/scp.c.orig
++++ contrib/mod_sftp/scp.c
+@@ -2077,10 +2077,11 @@
+      * want to return 1 here, since it will be us, not the client, which needs
+      * to close the connection.
+      */
+-    if (paths[scp_session->path_idx-1]->wrote_errors == TRUE) {
+-      return 1;
++    if (res == 1) {
++      if (paths[scp_session->path_idx-1]->wrote_errors == TRUE) {
++        return 1;
++      }
+     }
+-
+     return 0;
+ 
+   } else if (scp_opts & SFTP_SCP_OPT_ISDST) {

Property changes on: files/patch-contrib_mod__sftp_scp.c
___________________________________________________________________
Added: svn:mime-type
## -0,0 +1 ##
+text/plain
\ No newline at end of property
Added: fbsd:nokeywords
## -0,0 +1 ##
+yes
\ No newline at end of property
Added: svn:eol-style
## -0,0 +1 ##
+native
\ No newline at end of property
--- proftpd.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201402191027.s1JAR1u2072504>