byterazor/qpsmtpd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Need to cover situation where there are *no* hooks. Should we actually return

Browse Source 
OK from run_hooks() in the case where no hooks are there to fire?

git-svn-id: https://svn.perl.org/qpsmtpd/branches/0.3x@627 958fd67b-6ff1-0310-b445-bb7760255be9
This commit is contained in:
John Peacock 2006-03-01 17:25:51 +00:00
parent 82a32ed558
commit bd19ded5a2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/Qpsmtpd/SMTP.pm
View File

@ -597,7 +597,7 @@ sub queue {
if ($rc == DONE) {
return 1;
}
elsif ($rc != OK and $rc != DECLINED) {
elsif ($rc != OK and $rc != DECLINED and $rc != 0 ) {
return $self->log(LOGERROR, "pre plugin returned illegal value");
return 0;
}
@ -624,7 +624,7 @@ sub queue {
# And finally run any queue_post hooks
($rc, $msg) = $self->run_hooks("queue_post");
$self->log(LOGERROR, $msg) unless $rc == OK;
$self->log(LOGERROR, $msg) unless ($rc == OK or $rc == 0);
}