severity 530058 minor
tags 530058 + patch
quit
Hi,
Raphael Geissert wrote:
> checkbashisms' output:
>> possible bashism in ./usr/lib/courier/filterctl line 52 (unsafe echo with
>> backslash):
>> echo "This filter is not installed in $filterbindir\n" >&2
This has pretty low impact: that message is intended for humans, and
this just puts some literal '\' and 'n' characters at the end of it.
Patch follows.
diff --git a/courier/filters/filterctl.in b/courier/filters/filterctl.in
index b1db7ce..2bb6b21 100644
--- a/courier/filters/filterctl.in
+++ b/courier/filters/filterctl.in
@@ -49,7 +49,8 @@ esac
if test "$filter" = ""
then
- echo "This filter is not installed in $filterbindir\n" >&2
+ echo "This filter is not installed in $filterbindir" >&2
+ echo >&2
exit 1
fi
--
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
