[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: