BIOS: NameServerRemoteNotify callback may have uninitialized var
authorArnie Reynoso <arnier@ti.com>
Fri, 14 Mar 2014 17:04:10 +0000 (10:04 -0700)
committerChris Ring <cring@ti.com>
Mon, 17 Mar 2014 19:38:49 +0000 (12:38 -0700)
The callback function for NameServerRemoteNotify may end up with an
uninitialized swiHandle.  Though this case should never happen, the handle
should be initialized and an assert must be issued if it remains NULL.
This also fixes a Coverity error with the variable.

packages/ti/sdo/ipc/nsremote/NameServerRemoteNotify.c

index 409325061dd7a006a6c042976dd41b1c563b1788..7cb60be807d1114a6a7006d421533442a979414a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012-2013, Texas Instruments Incorporated
+ * Copyright (c) 2012-2014, Texas Instruments Incorporated
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -268,7 +268,7 @@ Void NameServerRemoteNotify_cbFxn(UInt16 procId, UInt16 lineId, UInt32 eventId,
         UArg arg, UInt32 payload)
 {
     NameServerRemoteNotify_Object *obj;
-    Swi_Handle swiHandle;
+    Swi_Handle swiHandle = NULL;
 
     obj = (NameServerRemoteNotify_Object *)arg;
 
@@ -288,6 +288,10 @@ Void NameServerRemoteNotify_cbFxn(UInt16 procId, UInt16 lineId, UInt32 eventId,
             /* set object state (used by ROV) */
             obj->localState = NameServerRemoteNotify_RECEIVE_RESPONSE;
             break;
+
+        default:
+            /* We should never get here but assert if we do */
+            Assert_isTrue(swiHandle != NULL, Ipc_A_internal);
     }
 
     Swi_post(swiHandle);