]> xenbits.xensource.com Git - people/aperard/xen-unstable.git/commitdiff
libxl: drop dead assignment to transaction variable from libxl__domain_make()
authorJan Beulich <jbeulich@suse.com>
Wed, 14 Jun 2023 10:32:10 +0000 (12:32 +0200)
committerJan Beulich <jbeulich@suse.com>
Wed, 14 Jun 2023 10:32:10 +0000 (12:32 +0200)
"t" is written first thing at the "retry_transaction" label.

Coverity ID: 1532321
Fixes: 1057300109ea ("libxl: fix error handling (xenstore transaction leak) in libxl__domain_make")
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Juergen Gross <jgross@suse.com>
Acked-by: Anthony PERARD <anthony.perard@citrix.com>
tools/libs/light/libxl_create.c

index d7e80d5df7b0b4f571cdc7e1ce6301ec46f349e6..393c5355797bf593eb3f4b452d2fccc341442b66 100644 (file)
@@ -909,10 +909,8 @@ retry_transaction:
              strlen(dom_type));
 
     if (!xs_transaction_end(ctx->xsh, t, 0)) {
-        if (errno == EAGAIN) {
-            t = 0;
+        if (errno == EAGAIN)
             goto retry_transaction;
-        }
         LOGED(ERROR, *domid, "domain creation ""xenstore transaction commit failed");
         rc = ERROR_FAIL;
         goto out;