More trapping.

SVN revision: 45682

src/modules/illume2/e_kbd.c

@@ -78,8 +78,6 @@ e_kbd_shutdown(void)
    Ecore_Event_Handler *handler;
    E_Border_Hook *hook;
 
-   e_kbd_dbus_shutdown();
-
    EINA_LIST_FREE(handlers, handler)
      ecore_event_handler_del(handler);
 
@@ -89,6 +87,8 @@ e_kbd_shutdown(void)
    EINA_LIST_FREE(kbds, kbd)
      e_object_del(E_OBJECT(kbd));
 
+   e_kbd_dbus_shutdown();
+
    return 1;
 }
 
@@ -264,7 +264,8 @@ _e_kbd_cb_animate(void *data)
    E_Kbd *kbd;
    double t, v;
 
-   kbd = data;
+   if (!(kbd = data)) return 0;
+
    t = ecore_loop_time_get() - kbd->start;
    if (t > kbd->len) t = kbd->len;
    if (kbd->len > 0.0)
@@ -305,7 +306,7 @@ _e_kbd_by_border_get(E_Border *bd)
    E_Border *over;
    E_Kbd *kbd;
 
-   if (!bd->stolen) return NULL;
+   if ((!bd) || (!bd->stolen)) return NULL;
    EINA_LIST_FOREACH(kbds, l, kbd) 
      {
         if (kbd->border == bd) return kbd;
@@ -320,6 +321,7 @@ _e_kbd_layout_send(E_Kbd *kbd)
 {
    Ecore_X_Virtual_Keyboard_State type;
 
+   if (!kbd) return;
    if ((kbd->visible) && (!kbd->disabled))
      {
 	type = ECORE_X_VIRTUAL_KEYBOARD_STATE_ON;
@@ -373,6 +375,8 @@ _e_kbd_border_hide(E_Border *bd)
 static void 
 _e_kbd_border_adopt(E_Kbd *kbd, E_Border *bd) 
 {
+   if ((!kbd) || (!bd)) return;
+
    kbd->border = bd;
    bd->sticky = 1;
    if (kbd->fullscreen)
@@ -390,6 +394,7 @@ _e_kbd_border_adopt(E_Kbd *kbd, E_Border *bd)
 static int
 _e_kbd_border_is_keyboard(E_Border *bd)
 {
+   if (!bd) return 0;
    if ((bd->client.vkbd.vkbd) || /* explicit hint that its a virtual keyboard */
        /* legacy */
        ( /* trap the matchbox qwerty and multitap kbd's */
@@ -404,6 +409,7 @@ _e_kbd_border_is_keyboard(E_Border *bd)
 static void 
 _e_kbd_hide(E_Kbd *kbd) 
 {
+   if (!kbd) return;
    if (kbd->timer) ecore_timer_del(kbd->timer);
    kbd->timer = NULL;
    if ((!kbd->visible) || (kbd->disabled)) return;
@@ -424,6 +430,7 @@ _e_kbd_hide(E_Kbd *kbd)
 static void 
 _e_kbd_slide(E_Kbd *kbd, int visible, double len) 
 {
+   if (!kbd) return;
    kbd->start = ecore_loop_time_get();
    kbd->len = len;
    kbd->adjust_start = kbd->adjust;
@@ -502,13 +509,18 @@ _e_kbd_cb_border_remove(void *data, int type, void *event)
    E_Kbd *kbd;
 
    ev = event;
+   if (focused_border) 
+     {
         if (ev->border == focused_border) 
           {
              focused_border = NULL;
              focused_vkbd_state = 0;
              return 1;
           }
+     }
    if (!(kbd = _e_kbd_by_border_get(ev->border))) return 1;
+   if (kbd->border) 
+     {
         if (kbd->border == ev->border) 
           {
              kbd->border = NULL;
@@ -527,6 +539,7 @@ _e_kbd_cb_border_remove(void *data, int type, void *event)
                   e_kbd_hide(kbd);
                }
           }
+     }
    else
      kbd->waiting_borders = eina_list_remove(kbd->waiting_borders, ev->border);
    return 1;
@@ -589,8 +602,11 @@ _e_kbd_cb_border_property(void *data, int type, void *event)
    ev = event;
    if (!ev->border->focused) return 1;
    if (_e_kbd_by_border_get(ev->border)) return 1;
+   if (focused_border) 
+     {
         if ((ev->border == focused_border) && 
             (ev->border->client.vkbd.state == focused_vkbd_state)) return 1;
+     }
    focused_vkbd_state = ev->border->client.vkbd.state;
    if (focused_vkbd_state == 0) return 1;