_dispose(): The mlist.Save() call should be in the try: block, since

not making it to the end of the try ought to be morally equivalent to
an aborted transaction.

1 files changed, 1 insertions, 1 deletions

diff --git a/Mailman/Queue/IncomingRunner.py b/Mailman/Queue/IncomingRunner.py
index ee1938784..0abf058c0 100644
--- a/Mailman/Queue/IncomingRunner.py
+++ b/Mailman/Queue/IncomingRunner.py
@@ -113,9 +113,9 @@ class IncomingRunner(Runner):
             status = self._dopipeline(mlist, msg, msgdata, pipeline)
             if status:
                 msgdata['pipeline'] = pipeline
+            mlist.Save()
             return status
         finally:
-            mlist.Save()
             mlist.Unlock()
 
     # Overridable