From: Dana Jansens <danakj@orodu.net>
Date: Mon, 27 Oct 2003 19:52:49 +0000 (+0000)
Subject: was setting NULL incorrectly to cancel actions and then not checking for NULL before... 
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=7c80db2552c97be9145ce88dd40f80b0873f56ef;p=chaz%2Fopenbox

was setting NULL incorrectly to cancel actions and then not checking for NULL before using it
---

diff --git a/openbox/action.c b/openbox/action.c
index 4ea50fe5..17d951cf 100644
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -395,7 +395,7 @@ void setup_action_showmenu(ObAction **a, ObUserAction uact)
        a time! */
     if (uact == OB_USER_ACTION_MENU_SELECTION) {
         action_unref(*a);
-        a = NULL;
+        *a = NULL;
     }
 }
 
@@ -817,7 +817,8 @@ ObAction *action_from_string(const gchar *name, ObUserAction uact)
             a = action_new(actionstrings[i].func);
             if (actionstrings[i].setup)
                 actionstrings[i].setup(&a, uact);
-            INTERACTIVE_LIMIT(a, uact);
+            if (a)
+                INTERACTIVE_LIMIT(a, uact);
             break;
         }
     if (!exist)