Attachment 624219 Details for Bug 864613 – add debugging statements to clarify behavior of polkit_backend_local_authorization_store_ensure function

[patch] add debugging statements to clarify behavior of polkit_backend_local_authorization_store_ensure function

polkit-0.96-add-debug.patch (text/plain), 3.33 KB, created by jared jennings on 2012-10-09 18:32:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: jared jennings

Created: 2012-10-09 18:32:24 UTC

Size: 3.33 KB

patch

obsolete

>diff -ru -x '*.so' -x '*.o' -x '*.la' -x '*.lai' -x .libs -x 'config.*' -x 'Make*' -x '*.pc' -x '*.xml' -x '*.html' -x libtool -x cancel -x pk-example-frobnicate -x polkit-agent-helper-1 -x polkitd -x pkaction -x pkcheck -x pkexec polkit-0.96/src/polkitbackend/polkitbackendlocalauthorizationstore.c polkit-0.96-add-debug/src/polkitbackend/polkitbackendlocalauthorizationstore.c
>--- polkit-0.96/src/polkitbackend/polkitbackendlocalauthorizationstore.c	2012-10-04 14:07:37.713544396 -0500
>+++ polkit-0.96-add-debug/src/polkitbackend/polkitbackendlocalauthorizationstore.c	2012-10-09 12:49:53.218534994 -0500
>@@ -539,6 +539,7 @@
>   GError *error;
>   GList *files;
>   GList *l;
>+  GList *l2;
> 
>   files = NULL;
> 
>@@ -592,6 +593,8 @@
> 
>       filename = g_file_get_path (file);
> 
>+      g_debug("reading file `%s'", filename);
>+
>       key_file = g_key_file_new ();
> 
>       error = NULL;
>@@ -627,20 +630,41 @@
>                 }
>               else
>                 {
>+                  g_debug("prepending authorization %s", authorization->id);
>                   store->priv->authorizations = g_list_prepend (store->priv->authorizations,
>                                                                 authorization);
>                 }
>             }
>           g_strfreev (groups);
> 
>+          g_debug("reversing authorization list");
>           store->priv->authorizations = g_list_reverse (store->priv->authorizations);
> 
>+          if(store->priv->authorizations) {
>+              g_debug("             after reverse, authorization list is:");
>+              for (l2 = store->priv->authorizations; l2 != NULL; l2 = l2->next)
>+                {
>+                  LocalAuthorization *authorization = l2->data;
>+                  g_debug("             %s", authorization->id);
>+                }
>+          }
>+             
>           g_key_file_free (key_file);
>         }
> 
>       g_free (filename);
>     }
> 
>+  if(store->priv->authorizations) {
>+      g_debug("p_b_l_a_s_e: after all reads, authorization list is:");
>+      for (l = store->priv->authorizations; l != NULL; l = l->next)
>+        {
>+          LocalAuthorization *authorization = l->data;
>+          g_debug("             %s", authorization->id);
>+        }
>+  }
>+      
>+
>   store->priv->has_data = TRUE;
> 
>  out:
>@@ -691,10 +715,16 @@
> 
>   polkit_backend_local_authorization_store_ensure (store);
> 
>+  if(store->priv->authorizations) {
>+  g_debug("p_b_l_a_s_l: looking up in store with dir %s:", g_file_get_uri (store->priv->directory));
>+  g_debug("             can %s do %s?", polkit_identity_to_string(identity), action_id);
>+  }
>+
>   for (l = store->priv->authorizations; l != NULL; l = l->next)
>     {
>       LocalAuthorization *authorization = l->data;
> 
>+      g_debug("-- checking authorization %s", authorization->id);
>       /* first match the action */
>       for (ll = authorization->action_specs; ll != NULL; ll = ll->next)
>         {
>@@ -739,6 +769,15 @@
>                action_id,
>                polkit_identity_to_string (identity));
>     }
>+  
>+  if(store->priv->authorizations) {
>+  g_debug ("** finished checking; did any match? %d", ret);
>+  if(ret) {
>+      g_debug ("                      any %d", *out_result_any);
>+      g_debug ("                      active %d", *out_result_active);
>+      g_debug ("                      inactive %d", *out_result_inactive);
>+    }
>+  }
> 
>   g_free (identity_string);
>

Actions: View | Diff

Attachments on bug 864613: 624219 | 624255 | 624256 | 744858