From: Dana Jansens <danakj@orodu.net>
Date: Wed, 9 May 2007 20:27:05 +0000 (+0000)
Subject: fix using uninit'd memory. don't use 0 keycode
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=481e68112177d1a1c0eb920f37c372ea34eeb371;p=chaz%2Fopenbox

fix using uninit'd memory. don't use 0 keycode
---

diff --git a/openbox/modkeys.c b/openbox/modkeys.c
index 4eb836e5..70605511 100644
--- a/openbox/modkeys.c
+++ b/openbox/modkeys.c
@@ -64,12 +64,17 @@ void modkeys_startup(gboolean reconfigure)
             KeySym sym;
             /* get a keycode that is bound to the mask (i) */
             KeyCode keycode = modmap->modifiermap[i*modmap->max_keypermod + j];
-            /* go through each keysym bound to the given keycode */
-            for (k = 0; k < keysyms_per_keycode; ++k) {
-                sym = keymap[(keycode-min_keycode) * keysyms_per_keycode + k];
-                if (sym != NoSymbol) {
-                    /* bind the key to the mask (e.g. Alt_L => Mod1Mask) */
-                    set_modkey_mask(nth_mask(i), sym);
+            g_print("keycode %d for mask %d\n", keycode, i);
+            if (keycode) {
+                /* go through each keysym bound to the given keycode */
+                for (k = 0; k < keysyms_per_keycode; ++k) {
+                    sym = keymap[(keycode-min_keycode) * keysyms_per_keycode +
+                                 k];
+                    g_print("k %d sym %s \n", k, XKeysymToString(sym));
+                    if (sym != NoSymbol) {
+                        /* bind the key to the mask (e.g. Alt_L => Mod1Mask) */
+                        set_modkey_mask(nth_mask(i), sym);
+                    }
                 }
             }
         }