X-Git-Url: https://git.jankratochvil.net/?p=PerlMail.git;a=blobdiff_plain;f=perlmail-accept;h=9a8584bf82d2e5ed741897d806f7a0e2fdea5034;hp=c4fa61ebc8cd022e29ad9187b5aaf48b44ad63f3;hb=851ec8b7b3dc1252952d9cd95ab6fba89ce50ba4;hpb=8dbc021592be6992b33afbee6b9d91d1990766bf

diff --git a/perlmail-accept b/perlmail-accept
index c4fa61e..9a8584b 100755
--- a/perlmail-accept
+++ b/perlmail-accept
@@ -130,8 +130,8 @@ sub inetd
 		$length==($_=read STDIN,$message,$length) or confess "Got $_ out of required $length bytes";
 		$length==length $message or confess "False read return ".length($message)." instead of $length";
 		{
-			local *STDOUT;
-			local *STDERR;
+			local *STDOUT;	# FIXME: fd's inherited by spawned processes are not closed this way!
+			local *STDERR;	# FIXME: fd's inherited by spawned processes are not closed this way!
 			local $DoBell=0;
 			process $message;
 			if ($DoBell) {
@@ -489,12 +489,16 @@ my($message)=@_;
 # return: true (error-message or "1") if is spam
 sub spamassassin
 {
+my($cmd)=@_;
+
+	$cmd||="spamassassin --exit-code";
 	# spamassassin has exit code 1 if IS spam, code 0 if NOT spam
 	local *CHILD;
 	local $SIG{"PIPE"}=sub { warn "spamassassin gave me SIGPIPE: broken pipe"; };
 	# prevent Razor2's: Can't call method "log" on unblessed reference at Razor2/Client/Agent.pm line 212.
 	local $ENV{"HOME"}=$HOME;
-	open CHILD,'|spamassassin --exit-code --mbox >/dev/null'
+	# 2>/dev/null to prevent error messages to corrupt inetd() output of perlmail-accept(1)
+	open CHILD,"|$cmd --mbox >/dev/null 2>/dev/null"
 			or return 0;
 	print CHILD $Message;
 	my $return=close CHILD;