Blame SOURCES/polkit-0.112-agent-leaks.patch

beed5f
A part of commit 7ecf29a9db86f7161e2ff48e7bb8ea46a90f954f
beed5f
Author: Miloslav Trmač <mitr@redhat.com>
beed5f
Date:   Wed Feb 8 22:57:21 2017 +0100
beed5f
beed5f
    Fix a memory leak in server_handle_authentication_agent_response{,2}
beed5f
    
beed5f
    Signed-off-by: Miloslav Trmač <mitr@redhat.com>
beed5f
beed5f
diff --git a/src/polkitbackend/polkitbackendauthority.c b/src/polkitbackend/polkitbackendauthority.c
beed5f
index 2bcad62..cad3f74 100644
beed5f
--- a/src/polkitbackend/polkitbackendauthority.c
beed5f
+++ b/src/polkitbackend/polkitbackendauthority.c
beed5f
@@ -1054,6 +1054,7 @@ server_handle_authentication_agent_response (Server                 *server,
beed5f
   g_dbus_method_invocation_return_value (invocation, g_variant_new ("()"));
beed5f
 
beed5f
  out:
beed5f
+  g_variant_unref (identity_gvariant);
beed5f
   if (identity != NULL)
beed5f
     g_object_unref (identity);
beed5f
 }
beed5f
commit d9efd2673d73214e7990e3e67cdddfa77c6a8226
beed5f
Author: Miloslav Trmač <mitr@redhat.com>
beed5f
Date:   Wed Feb 8 22:55:10 2017 +0100
beed5f
beed5f
    Fix a memory leak in server_handle_unregister_authentication_agent
beed5f
    
beed5f
    Signed-off-by: Miloslav Trmač <mitr@redhat.com>
beed5f
beed5f
diff --git a/src/polkitbackend/polkitbackendauthority.c b/src/polkitbackend/polkitbackendauthority.c
beed5f
index 7e08e57..2bcad62 100644
beed5f
--- a/src/polkitbackend/polkitbackendauthority.c
beed5f
+++ b/src/polkitbackend/polkitbackendauthority.c
beed5f
@@ -1003,6 +1003,7 @@ server_handle_unregister_authentication_agent (Server                 *server,
beed5f
   g_dbus_method_invocation_return_value (invocation, g_variant_new ("()"));
beed5f
 
beed5f
  out:
beed5f
+  g_variant_unref (subject_gvariant);
beed5f
   if (subject != NULL)
beed5f
     g_object_unref (subject);
beed5f
 }
beed5f
commit af4566e1a7e9031b9a05f49c7d27bf379d822016
beed5f
Author: Miloslav Trmač <mitr@redhat.com>
beed5f
Date:   Thu Feb 9 19:53:54 2017 +0100
beed5f
beed5f
    Fix a memory leak per agent authentication
beed5f
    
beed5f
    Signed-off-by: Miloslav Trmač <mitr@redhat.com>
beed5f
beed5f
diff --git a/src/polkitbackend/polkitbackendinteractiveauthority.c b/src/polkitbackend/polkitbackendinteractiveauthority.c
beed5f
index bf0ee48..b8096b3 100644
beed5f
--- a/src/polkitbackend/polkitbackendinteractiveauthority.c
beed5f
+++ b/src/polkitbackend/polkitbackendinteractiveauthority.c
beed5f
@@ -1906,15 +1906,15 @@ authentication_agent_begin_cb (GDBusProxy   *proxy,
beed5f
   AuthenticationSession *session = user_data;
beed5f
   gboolean gained_authorization;
beed5f
   gboolean was_dismissed;
beed5f
+  GVariant *result;
beed5f
   GError *error;
beed5f
 
beed5f
   was_dismissed = FALSE;
beed5f
   gained_authorization = FALSE;
beed5f
 
beed5f
   error = NULL;
beed5f
-  if (!g_dbus_proxy_call_finish (proxy,
beed5f
-                                 res,
beed5f
-                                 &error))
beed5f
+  result = g_dbus_proxy_call_finish (proxy, res, &error);
beed5f
+  if (result == NULL)
beed5f
     {
beed5f
       g_printerr ("Error performing authentication: %s (%s %d)\n",
beed5f
                   error->message,
beed5f
@@ -1926,6 +1926,7 @@ authentication_agent_begin_cb (GDBusProxy   *proxy,
beed5f
     }
beed5f
   else
beed5f
     {
beed5f
+      g_variant_unref (result);
beed5f
       gained_authorization = session->is_authenticated;
beed5f
       g_debug ("Authentication complete, is_authenticated = %d", session->is_authenticated);
beed5f
     }