[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.2.10-84-gf11638a



The following commit has been merged in the master branch:
commit 5d6635850b14ae51ccbedbae1280e3a84505f92e
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Mon Jun 1 19:29:07 2009 +0100

    Fix script_is_evil_and_wrong's return value for backgrounded tasks
    
    * checks/scripts:
      + [ADB] Fix a syntax error in the shellscript wrapper detection that
        lead to it returning a part of the wrapper script rather than a
        simple success value.

diff --git a/checks/scripts b/checks/scripts
index 8440d2b..7c86dde 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -1095,7 +1095,7 @@ sub script_is_evil_and_wrong {
 	    (^\s*|\beval\s*[\'\"]|(;|&&|\b(then|else))\s*)
 	    exec\s+true(\s|\Z)~x) {
 
-	    $ret = $1;
+	    $ret = 1;
 	    last;
 	}
     }
diff --git a/debian/changelog b/debian/changelog
index 7793553..a110d8d 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -68,6 +68,9 @@ lintian (2.2.11) UNRELEASED; urgency=low
       alternative.  Thanks, Guillaume Delacour.  (Closes: #529457)
     + [ADB] Detect shellscript wrappers which embed a call to "exec" in a
       conditional statement.  Thanks, Raphael Geissert.
+    + [ADB] Fix a syntax error in the shellscript wrapper detection that lead
+      to it returning a part of the wrapper script rather than a simple
+      success value.
   * checks/shared-libs.desc:
     + [RA] Downgrade certainty of tags for which we recommend overrides.
       Patch from Raphael Geissert.

-- 
Debian package checker


Reply to: