2010-10-28 Marcus Brinkmann <marcus.brinkmann@ruhr-uni-bochum.de>
authorMarcus Brinkmann <mb@g10code.com>
Thu, 28 Oct 2010 14:00:14 +0000 (14:00 +0000)
committerMarcus Brinkmann <mb@g10code.com>
Thu, 28 Oct 2010 14:00:14 +0000 (14:00 +0000)
* opassuan.c (gpgme_op_assuan_transact_ext): Fix uninitialized
value use.  Reported by Marc Mutz.

src/ChangeLog
src/opassuan.c

index 50b0b68..a868d6d 100644 (file)
@@ -1,3 +1,8 @@
+2010-10-28  Marcus Brinkmann  <marcus.brinkmann@ruhr-uni-bochum.de>
+
+       * opassuan.c (gpgme_op_assuan_transact_ext): Fix uninitialized
+       value use.  Reported by Marc Mutz.
+
 2010-10-07  Werner Koch  <wk@g10code.com>
 
        * gpgme-tool.c (ARGP_ERR_UNKNOWN): Use EDEADLK if available.
index 639a366..bd50089 100644 (file)
@@ -126,9 +126,10 @@ gpgme_op_assuan_transact_ext (gpgme_ctx_t ctx,
                         data_cb, data_cb_value,
                         inq_cb, inq_cb_value,
                         status_cb, status_cb_value);
-  if (!err)
-    err = _gpgme_wait_one_ext (ctx, &op_err);
+  if (err)
+    goto out;
 
+  err = _gpgme_wait_one_ext (ctx, &op_err);
   if (op_err)
     {
       TRACE_LOG2 ("op_err = %s <%s>", gpgme_strerror (op_err),
@@ -140,7 +141,8 @@ gpgme_op_assuan_transact_ext (gpgme_ctx_t ctx,
     }
   if (op_err_p)
     *op_err_p = op_err;
-
+  
+ out:
   return TRACE_ERR (err);
 }