From: ho.namkoong <ho.namkoong@samsung.com>
Date: Thu, 28 Nov 2013 02:47:51 +0000 (+0900)
Subject: SDB: fix sdb error message
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9b86722d51c72db687326641bae2982d52768cca;p=sdk%2Ftools%2Fsdb.git

SDB: fix sdb error message
when there are more than one device, error happend by -d option is modified

Change-Id: Ibdf5fea4e9c8e42752edbeaba530ebf7d1e22105
Signed-off-by: ho.namkoong <ho.namkoong@samsung.com>
---

diff --git a/src/sdb_constants.c b/src/sdb_constants.c
index bc31029..db8f227 100644
--- a/src/sdb_constants.c
+++ b/src/sdb_constants.c
@@ -273,10 +273,9 @@
     const char* STATE_UNKNOWN = "unknown";
     const char* STATE_SUSPENDED = "suspended";
 
-    const char* TRANSPORT_ERR_MORE_THAN_ONE_TARGET = "more than one target";
-    const char* TRANSPORT_ERR_MORE_THAN_ONE_EMUL = "more than one emulator";
-    const char* TRANSPORT_ERR_MORE_THAN_ONE_CONNECT = "more than one connected emulator";
-    const char* TRANSPORT_ERR_MORE_THAN_ONE_DEV = "more than one device";
+    const char* TRANSPORT_ERR_MORE_THAN_ONE_TARGET = "There are more than one target. Please specify the target using -s option.";
+    const char* TRANSPORT_ERR_MORE_THAN_ONE_EMUL = "There are more than one emulator. Please specify the emulator using -s option.";
+    const char* TRANSPORT_ERR_MORE_THAN_ONE_DEV = "There are more than one device. Please specify the device using -s option.";
     const char* TRANSPORT_ERR_TARGET_OFFLINE = "target offline";
     const char* TRANSPORT_ERR_TARGET_SUSPENDED = "emulator is in suspended mode";
     const char* TRANSPORT_ERR_TARGET_NOT_FOUND = "target not found";
diff --git a/src/transport.c b/src/transport.c
index 8c444ae..cc0fc8f 100755
--- a/src/transport.c
+++ b/src/transport.c
@@ -540,7 +540,7 @@ TRANSPORT *acquire_one_transport(transport_type ttype, const char* serial, char*
                 if (result) {
                     *error_out = (char*)TRANSPORT_ERR_MORE_THAN_ONE_TARGET;
                     result = NULL;
-                    break;
+                    goto exit;
                 }
                 result = transport_;
             }
@@ -554,19 +554,19 @@ TRANSPORT *acquire_one_transport(transport_type ttype, const char* serial, char*
                         *error_out = (char*)TRANSPORT_ERR_MORE_THAN_ONE_EMUL;
                     }
                     result = NULL;
-                    break;
+                    goto exit;
                 }
                 result = transport_;
             }
         }
     }
 
-    sdb_mutex_unlock(&transport_lock, "transport acquire_one_transport");
-
     if (result == NULL ) {
         *error_out = (char*)TRANSPORT_ERR_TARGET_NOT_FOUND;
     }
 
+exit:
+    sdb_mutex_unlock(&transport_lock, "transport acquire_one_transport");
     return result;
 }