SDOCM00114745 Ipc_transportConfig has race window

[ipc/ipcdev.git] / linux / src / api / MultiProc.c

diff --git a/linux/src/api/MultiProc.c b/linux/src/api/MultiProc.c
index d40cc76db8abe8be5997ca596dacd23b5e81ef88..0a8435f1d1bacf67e4406ef911fe5746ac634137 100644 (file)
--- a/linux/src/api/MultiProc.c
+++ b/linux/src/api/MultiProc.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013-2014, Texas Instruments Incorporated
+ * Copyright (c) 2013-2015 Texas Instruments Incorporated - http://www.ti.com
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -30,9 +30,9 @@
  * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 /*
- *  @file      MultiProcQ.c
+ *  @file       MultiProcQ.c
  *
- *  @brief     MultiProc Linux implementation
+ *  @brief      MultiProc Linux implementation
  */
 
 /* Standard IPC header */
@@ -72,32 +72,31 @@ Void MultiProc_getConfig (MultiProc_Config * cfg)
     assert (cfg != NULL);
 
     handle = LAD_findHandle();
-    if (handle == LAD_MAXNUMCLIENTS) {
-       PRINTVERBOSE1(
-         "MultiProc_getConfig: can't find connection to daemon for pid %d\n",
-          getpid())
 
-       return;
+    if (handle == LAD_MAXNUMCLIENTS) {
+        PRINTVERBOSE1("MultiProc_getConfig: can't find connection to daemon "
+                "for pid %d\n", getpid())
+        return;
     }
 
     cmd.cmd = LAD_MULTIPROC_GETCONFIG;
     cmd.clientId = handle;
 
     if ((status = LAD_putCommand(&cmd)) != LAD_SUCCESS) {
-       PRINTVERBOSE1(
-         "MultiProc_getConfig: sending LAD command failed, status=%d\n", status)
-       return;
+        PRINTVERBOSE1("MultiProc_getConfig: sending LAD command failed, "
+                "status=%d\n", status)
+        return;
     }
 
     if ((status = LAD_getResponse(handle, &rsp)) != LAD_SUCCESS) {
-       PRINTVERBOSE1("MultiProc_getConfig: no LAD response, status=%d\n", status)
-       return;
+        PRINTVERBOSE1("MultiProc_getConfig: no LAD response, status=%d\n",
+                status)
+        return;
     }
     status = rsp.multiprocGetConfig.status;
 
-    PRINTVERBOSE2(
-      "MultiProc_getConfig: got LAD response for client %d, status=%d\n",
-      handle, status)
+    PRINTVERBOSE2("MultiProc_getConfig: got LAD response for client %d, "
+            "status=%d\n", handle, status)
 
     memcpy(cfg, &rsp.multiprocGetConfig.cfg, sizeof(*cfg));