[PATCH] Fix for plugins returning DONE from HELO/EHLO

[Hanno Hecker]

We have to return something true, else the client gets an additional
 451 Internal error - try again later - command 'helo' failed unexpectedly
after the plugin's ->respond() message.
---
 lib/Qpsmtpd/SMTP.pm |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/lib/Qpsmtpd/SMTP.pm b/lib/Qpsmtpd/SMTP.pm
index f669055..bf21771 100644
--- a/lib/Qpsmtpd/SMTP.pm
+++ b/lib/Qpsmtpd/SMTP.pm
@@ -162,7 +162,8 @@ sub helo_respond {
   my ($self, $rc, $msg, $args) = @_;
   my ($hello_host) = @$args;
   if ($rc == DONE) {
-    # do nothing
+    # do nothing:
+    1;
   } elsif ($rc == DENY) {
     $self->respond(550, @$msg);
   } elsif ($rc == DENYSOFT) {
@@ -198,7 +199,8 @@ sub ehlo_respond {
   my ($self, $rc, $msg, $args) = @_;
   my ($hello_host) = @$args;
   if ($rc == DONE) {
-    # do nothing
+    # do nothing:
+    1; 
   } elsif ($rc == DENY) {
     $self->respond(550, @$msg);
   } elsif ($rc == DENYSOFT) {
-- 
1.5.6.5