We knew "peventbuf" was a valid pointer and "peventbuf + 2" is also
non-NULL.  I have removed the check.

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/drivers/staging/rtl8712/rtl8712_cmd.c 
b/drivers/staging/rtl8712/rtl8712_cmd.c
index 088647c..53f247b 100644
--- a/drivers/staging/rtl8712/rtl8712_cmd.c
+++ b/drivers/staging/rtl8712/rtl8712_cmd.c
@@ -471,11 +471,9 @@ void r8712_event_handle(struct _adapter *padapter, uint 
*peventbuf)
        if (pevt_priv->event_seq > 127)
                pevt_priv->event_seq = 0;
        peventbuf = peventbuf + 2; /* move to event content, 8 bytes alignment 
*/
-       if (peventbuf) {
-               event_callback = wlanevents[evt_code].event_callback;
-               if (event_callback)
-                       event_callback(padapter, (u8 *)peventbuf);
-       }
+       event_callback = wlanevents[evt_code].event_callback;
+       if (event_callback)
+               event_callback(padapter, (u8 *)peventbuf);
        pevt_priv->evt_done_cnt++;
 _abort_event_:
        return;
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to