[Bug] Fix possible client count overflow

The number of existing clients already running was being incremented
and then checked for maximum clients allowed.
However in the error scenario i.e. when maximum allowed clients are already
running and the new incoming client connection request was rejected the
client count was incremented unconditionally and not rolled back to
previous value thus resulting in possible overflow error.

This patch fixes this erroneous case.

Change-Id: I3e2b2bebfd025ba6a99b7ef838537d9716154ca4
Signed-off-by: Pradeep Venkatasubbarao <pradeepv@ti.com>

1 files changed, 6 insertions, 4 deletions

diff --git a/libdce.c b/libdce.c
index 349b6ab..5cb0ec5 100644
--- a/libdce.c
+++ b/libdce.c
@@ -164,13 +164,15 @@ static int dce_ipc_init(int core)
 
     DEBUG(" >> dce_ipc_init\n");
 
-    /* Create remote server insance */
-    __ClientCount[core]++;
-
-    if(__ClientCount[core] > MAX_INSTANCES) {
+        /*First check if maximum clients are already using ipc*/
+    if(__ClientCount[core] >= MAX_INSTANCES) {
         eError = DCE_EXDM_UNSUPPORTED;
         return (eError);
     }
+
+    /* Create remote server insance */
+    __ClientCount[core]++;
+
     if(__ClientCount[core] > 1) {
          goto EXIT;
     }