Teach AbortOutOfAnyTransaction to clean up partially-started transactions.

AbortOutOfAnyTransaction failed to do anything if the state it saw on
entry corresponded to failing partway through StartTransaction.  I fixed
AbortCurrentTransaction to cope with that case way back in commit
60b2444cc3ba037630c9b940c3c9ef01b954b87b, but evidently overlooked that
AbortOutOfAnyTransaction should do likewise.

Back-patch to all supported branches.  It's not clear that this omission
has any more-than-cosmetic consequences, but it's also not clear that it
doesn't, so back-patching seems the least risky choice.

diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
index d1cebeb86db9f4f977a26d4dddeb23756016f00c..511e17c2e455b3763e6ae43624b4d193c8d8b82e 100644 (file)
--- a/src/backend/access/transam/xact.c
+++ b/src/backend/access/transam/xact.c
@@ -3748,7 +3748,24 @@ AbortOutOfAnyTransaction(void)
        switch (s->blockState)
        {
            case TBLOCK_DEFAULT:
-               /* Not in a transaction, do nothing */
+               if (s->state == TRANS_DEFAULT)
+               {
+                   /* Not in a transaction, do nothing */
+               }
+               else
+               {
+                   /*
+                    * We can get here after an error during transaction start
+                    * (state will be TRANS_START).  Need to clean up the
+                    * incompletely started transaction.  First, adjust the
+                    * low-level state to suppress warning message from
+                    * AbortTransaction.
+                    */
+                   if (s->state == TRANS_START)
+                       s->state = TRANS_INPROGRESS;
+                   AbortTransaction();
+                   CleanupTransaction();
+               }
                break;
            case TBLOCK_STARTED:
            case TBLOCK_BEGIN: