INN commit: trunk/scripts (simpleftp.in)

[INN Commit]

    Date: Saturday, April 16, 2011 @ 01:21:13
  Author: iulius
Revision: 9196

The simpleftp script does not switch to binary mode
before transferring files.

If it is used by actsyncd to get active.gz from ftp.isc.org,
the transferred archive is corrupted.

Thanks to Matthias Meyser for the bug report.

Modified:
  trunk/scripts/simpleftp.in

--------------+
 simpleftp.in |    5 +++++
 1 file changed, 5 insertions(+)

Modified: simpleftp.in
===================================================================
--- simpleftp.in	2011-04-16 08:09:42 UTC (rev 9195)
+++ simpleftp.in	2011-04-16 08:21:13 UTC (rev 9196)
@@ -2,6 +2,8 @@
 
 # simpleftp - Rudimentary FTP client.
 #
+# $Id$
+#
 # Author: David Lawrence <tale at isc.org>.
 #         Rewritten by Julien Elie to use Net::FTP.
 #
@@ -75,6 +77,9 @@
   $path =~ s([^/]+$)();
   $localfile =~ s(.*/)();
 
+  $ftp->binary
+    or next;
+
   $ftp->cwd($path)
     or next;
   $ftp->get($localfile)