On 10/20/11 03:44, Dave Airlie wrote:
From: Dave Airlie<[email protected]>

pointed out by coverity.

Signed-off-by: Dave Airlie<[email protected]>
---
  hw/xnest/Keyboard.c |    3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/hw/xnest/Keyboard.c b/hw/xnest/Keyboard.c
index ec629dc..c138438 100644
--- a/hw/xnest/Keyboard.c
+++ b/hw/xnest/Keyboard.c
@@ -207,7 +207,7 @@ void
  xnestUpdateModifierState(unsigned int state)
  {
    DeviceIntPtr pDev = xnestKeyboardDevice;
-  KeyClassPtr keyc = pDev->key;
+  KeyClassPtr keyc;
    int i;
    CARD8 mask;
    int xkb_state;
@@ -215,6 +215,7 @@ xnestUpdateModifierState(unsigned int state)
    if (!pDev)
        return;

+  keyc = pDev->key;
    xkb_state = XkbStateFieldFromRec(&pDev->key->xkbInfo->state);
    state = state&  0xff;


Reviewed-by: Alan Coopersmith <[email protected]>

--
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to