[at-spi2-core: 1/2] Move va_end from _atspi_dbus_call_partial_va to _atspi_dbus_call_partial




commit 5c497d0fb512d6ef432e1e5e87e1052691ca32ef
Author: Samuel Thibault <samuel thibault ens-lyon org>
Date:   Sun Mar 13 23:50:40 2022 +0100

    Move va_end from _atspi_dbus_call_partial_va to _atspi_dbus_call_partial
    
    The C standard says that va_end has to be called in the same function as
    va_start.

 atspi/atspi-misc.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
---
diff --git a/atspi/atspi-misc.c b/atspi/atspi-misc.c
index d896afbd..8a6d2636 100644
--- a/atspi/atspi-misc.c
+++ b/atspi/atspi-misc.c
@@ -1136,10 +1136,14 @@ _atspi_dbus_call_partial (gpointer obj,
                           GError **error,
                           const char *type, ...)
 {
+  DBusMessage *ret;
   va_list args;
 
   va_start (args, type);
-  return _atspi_dbus_call_partial_va (obj, interface, method, error, type, args);
+  ret = _atspi_dbus_call_partial_va (obj, interface, method, error, type, args);
+  va_end (args);
+
+  return ret;
 }
 
 
@@ -1174,7 +1178,6 @@ _atspi_dbus_call_partial_va (gpointer obj,
   reply = dbind_send_and_allow_reentry (aobj->app->bus, msg, &err);
   check_for_hang (reply, &err, aobj->app->bus, aobj->app->bus_name);
 out:
-  va_end (args);
   if (msg)
     dbus_message_unref (msg);
   process_deferred_messages ();


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]