commit b498945361d8c73ea7d183ed9ead2eb5d3d4249a
author Stephen Crane <cranes@google.com> Fri Feb 17 18:08:49 2023 +0000
committer Stephen Crane <cranes@google.com> Sat Feb 18 00:12:24 2023 +0000
tree 257b274510c07c8b8eca1152f5eeea7ed47cab4e
parent 23c90c3415b1ba8863dff5d304f8bb78f8f26293

storage: client_tipc: Checkpoint without active transaction

STORAGE_END_TRANSACTION with STORAGE_MSG_FLAG_TRANSACT_COMPLETE is
normally a no-op, but adding the STORAGE_MSG_FLAG_TRANSACT_CHECKPOINT
flag indicates that the client wants to create a checkpoint of the
current state. This change allows that combination to create a
checkpoint even when a checkpoint did not exist previously.

Test: manual
Bug: 244611675
Change-Id: I0422996279f958423fe85fb830fd2ab4cda818bf

client_tipc.c

diff --git a/client_tipc.c b/client_tipc.c
index 0c554e2..f5144ec 100644
--- a/client_tipc.c
+++ b/client_tipc.c
@@ -1246,6 +1246,11 @@
     /* abort transaction and clear sticky transaction error */
     if (msg->cmd == STORAGE_END_TRANSACTION) {
         if (msg->flags & STORAGE_MSG_FLAG_TRANSACT_COMPLETE) {
+            /* Allow checkpoint creation without an active transaction */
+            if (msg->flags & STORAGE_MSG_FLAG_TRANSACT_CHECKPOINT &&
+                !transaction_is_active(&session->tr)) {
+                transaction_activate(&session->tr);
+            }
             /* try to complete current transaction */
             if (transaction_is_active(&session->tr)) {
                 transaction_complete_etc(