[SCM] Git repository for devscripts branch, squeeze, updated. v2.10.69+squeeze3-5-gab3e4a4

James McCoy jamessan at debian.org
Sat Sep 15 19:42:43 UTC 2012


The following commit has been merged in the squeeze branch:
commit 252a42d225f489e398f3c0402c1f7d1e9a4451c0
Author: Nikolaus Schulz <schulz at macnetix.de>
Date:   Thu Aug 23 21:46:14 2012 +0200

    debdiff: fix regression in exit code (Closes: #686247)
    
    Commit 9cbe605 "debdiff: fix CVE-2012-2012 [...]" broke the exit code of
    debdiff, at it made wdiff_control_files return 1 if wdiff found no
    differences in the control files, which in turn made debdiff exit with an
    exit code of 1.
    
    Signed-off-by: Raphael Geissert <geissert at debian.org>
    Signed-off-by: James McCoy <jamessan at debian.org>

diff --git a/scripts/debdiff.pl b/scripts/debdiff.pl
index d6f6130..6da8162 100755
--- a/scripts/debdiff.pl
+++ b/scripts/debdiff.pl
@@ -1047,6 +1047,7 @@ sub wdiff_control_files($$$$$)
 		my $msg = ucfirst($cf) . " files$usepkgname: wdiff output";
 		print $msg, "\n", '-' x length $msg, "\n";
 		print $wdiff;
+		$status = 1;
 	    } else {
 		my @output;
 		@output = split /\n/, $wdiff;
@@ -1054,8 +1055,8 @@ sub wdiff_control_files($$$$$)
 		my $msg = ucfirst($cf) . " files$usepkgname: lines which differ (wdiff format)";
 		print "\n", $msg, "\n", '-' x length $msg, "\n";
                 print join("\n", at output), "\n";
+		$status = 1;
 	    }
-	    $status = 1;
 	}
     }
 

-- 
Git repository for devscripts



More information about the devscripts-devel mailing list