SDOCM00115768 Incorrect check for invalid pointer in Ipc_detach
authorRamsey Harris <ramsey@ti.com>
Thu, 16 Apr 2015 00:00:37 +0000 (17:00 -0700)
committerRobert Tivy <rtivy@ti.com>
Thu, 16 Apr 2015 23:05:16 +0000 (16:05 -0700)
When checking for invalid pointers in 'struct Reserved', use
SharedRegion_INVALIDSRPTR instead of implicit check for NULL.

packages/ti/sdo/ipc/Ipc.c

index 9e67d389913ff606a7c9f8f3c703a83d65b04d73..a424886719996ab56ce6e4504dd0833967552756 100644 (file)
@@ -518,7 +518,7 @@ Int Ipc_detach(UInt16 remoteProcId)
             return (Ipc_E_FAIL);
         }
 
             return (Ipc_E_FAIL);
         }
 
-        if (slave->transportSRPtr) {
+        if (slave->transportSRPtr != SharedRegion_invalidSRPtr()) {
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to MessageQ transport instance */
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to MessageQ transport instance */
@@ -544,7 +544,7 @@ Int Ipc_detach(UInt16 remoteProcId)
             return (Ipc_E_FAIL);
         }
 
             return (Ipc_E_FAIL);
         }
 
-        if (slave->nsrnSRPtr) {
+        if (slave->nsrnSRPtr != SharedRegion_invalidSRPtr()) {
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to NSRN instance */
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to NSRN instance */
@@ -570,7 +570,7 @@ Int Ipc_detach(UInt16 remoteProcId)
             return (Ipc_E_FAIL);
         }
 
             return (Ipc_E_FAIL);
         }
 
-        if (slave->notifySRPtr) {
+        if (slave->notifySRPtr != SharedRegion_invalidSRPtr()) {
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to Notify instance */
             /* free the memory if slave processor */
             if (MultiProc_self() < remoteProcId) {
                 /* get the pointer to Notify instance */