don't dereference NULL pointer when there is nothing selected and pressing right
authorMikael Magnusson <mikachu@gmail.com>
Thu, 29 Apr 2010 20:11:17 +0000 (16:11 -0400)
committerMikael Magnusson <mikachu@gmail.com>
Fri, 14 May 2010 13:42:18 +0000 (15:42 +0200)
openbox/event.c

index f7fe030..1c6c043 100644 (file)
@@ -1725,7 +1725,9 @@ static gboolean event_handle_menu_keyboard(XEvent *ev)
 
         else if (ob_keycode_match(keycode, OB_KEY_RIGHT)) {
             /* Right goes to the selected submenu */
-            if (frame->selected->entry->type == OB_MENU_ENTRY_TYPE_SUBMENU) {
+            if (frame->selected &&
+                frame->selected->entry->type == OB_MENU_ENTRY_TYPE_SUBMENU)
+            {
                 /* make sure it is visible */
                 menu_frame_select(frame, frame->selected, TRUE);
                 menu_frame_select_next(frame->child);