[taler-merchant] branch master updated: fix commit logic

[gnunet]

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository merchant.

The following commit(s) were added to refs/heads/master by this push:
     new 43d1009  fix commit logic
43d1009 is described below

commit 43d1009f4f333b9413866624eee3861d96da8a5c
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Sat Aug 29 01:39:01 2020 +0200

    fix commit logic
---
 src/backend/taler-merchant-httpd_post-orders-ID-pay.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/backend/taler-merchant-httpd_post-orders-ID-pay.c 
b/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
index 4626e3d..915f305 100644
--- a/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
+++ b/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
@@ -1409,12 +1409,11 @@ begin_transaction (struct PayContext *pc)
   }
 
   /* Now commit! */
-  if (0 <= qs)
-    qs = TMH_db->commit (TMH_db->cls);
-  else
-    TMH_db->rollback (TMH_db->cls);
+  qs = TMH_db->commit (TMH_db->cls);
   if (0 > qs)
   {
+    /* commit failed */
+    TMH_db->rollback (TMH_db->cls);
     if (GNUNET_DB_STATUS_SOFT_ERROR == qs)
     {
       begin_transaction (pc);

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.