ftpserver.pl: Corrected misaligned indentation in POP3 handlers
authorSteve Holme <steve_holme@hotmail.com>
Sat, 7 Sep 2013 22:14:54 +0000 (23:14 +0100)
committerSteve Holme <steve_holme@hotmail.com>
Sun, 8 Sep 2013 01:48:33 +0000 (02:48 +0100)
Fixed incorrect indentation used in both the RETR_pop3 and LIST_pop3
functions which was 5 and 9 characters rather than 4 and 8.

tests/ftpserver.pl

index f9031bb..990414b 100755 (executable)
@@ -1231,69 +1231,68 @@ sub AUTH_pop3 {
 }
 
 sub RETR_pop3 {
-     my ($testno) = @_;
-     my @data;
-
-     if($testno =~ /^verifiedserver$/) {
-         # this is the secret command that verifies that this actually is
-         # the curl test server
-         my $response = "WE ROOLZ: $$\r\n";
-         if($verbose) {
-             print STDERR "FTPD: We returned proof we are the test server\n";
-         }
-         $data[0] = $response;
-         logmsg "return proof we are we\n";
-     }
-     else {
-         logmsg "retrieve a mail\n";
-
-         $testno =~ s/^([^0-9]*)//;
-         my $testpart = "";
-         if ($testno > 10000) {
-             $testpart = $testno % 10000;
-             $testno = int($testno / 10000);
-         }
-
-         # send mail content
-         loadtest("$srcdir/data/test$testno");
-
-         @data = getpart("reply", "data$testpart");
-     }
-
-     sendcontrol "+OK Mail transfer starts\r\n";
-
-     for my $d (@data) {
-         sendcontrol $d;
-     }
-
-     # end with the magic 3-byte end of mail marker, assumes that the
-     # mail body ends with a CRLF!
-     sendcontrol ".\r\n";
-
-     return 0;
+    my ($testno) = @_;
+    my @data;
+
+    if($testno =~ /^verifiedserver$/) {
+        # this is the secret command that verifies that this actually is
+        # the curl test server
+        my $response = "WE ROOLZ: $$\r\n";
+        if($verbose) {
+            print STDERR "FTPD: We returned proof we are the test server\n";
+        }
+        $data[0] = $response;
+        logmsg "return proof we are we\n";
+    }
+    else {
+        logmsg "retrieve a mail\n";
+
+        $testno =~ s/^([^0-9]*)//;
+        my $testpart = "";
+        if ($testno > 10000) {
+            $testpart = $testno % 10000;
+            $testno = int($testno / 10000);
+        }
+
+        # send mail content
+        loadtest("$srcdir/data/test$testno");
+
+        @data = getpart("reply", "data$testpart");
+    }
+
+    sendcontrol "+OK Mail transfer starts\r\n";
+
+    for my $d (@data) {
+        sendcontrol $d;
+    }
+
+    # end with the magic 3-byte end of mail marker, assumes that the
+    # mail body ends with a CRLF!
+    sendcontrol ".\r\n";
+
+    return 0;
 }
 
 sub LIST_pop3 {
+    # This is a built-in fake-message list
+    my @data = (
+        "1 100\r\n",
+        "2 4294967400\r\n",    # > 4 GB
+        "4 200\r\n", # Note that message 3 is a simulated "deleted" message
+    );
 
-# this is a built-in fake-message list
-my @pop3list=(
-"1 100\r\n",
-"2 4294967400\r\n",    # > 4 GB
-"4 200\r\n", # Note that message 3 is a simulated "deleted" message
-);
-
-     logmsg "retrieve a message list\n";
+    logmsg "retrieve a message list\n";
 
-     sendcontrol "+OK Listing starts\r\n";
+    sendcontrol "+OK Listing starts\r\n";
 
-     for my $d (@pop3list) {
-         sendcontrol $d;
-     }
+    for my $d (@data) {
+        sendcontrol $d;
+    }
 
-     # end with the magic 3-byte end of listing marker
-     sendcontrol ".\r\n";
+    # End with the magic 3-byte end of listing marker
+    sendcontrol ".\r\n";
 
-     return 0;
+    return 0;
 }
 
 sub DELE_pop3 {