commit 0e42408b91f9581f0177ce86fd1074fb825bad2b
author liuchao <liuchao741@huawei.com> Wed Jan 04 10:46:06 2017 +0800
committer Marie Janssen <jamuraa@google.com> Wed Jan 04 18:59:44 2017 +0000
tree 8bcf006b2e865c1330a2da8d031675ad3dac10de
parent 2a399b3745502d9a09761ad0cd404bcaddf129f1

Fix an inappropriate sscanf return value judgment

This fixes an inappropriate sscanf return value judgment that
res check here is not needed while res2 is the one that need to be checked

Test: mm -j 8
Change-Id: I3e332286c9434d5b9d01421f5f421039f84ad6c7

bta/hf_client/bta_hf_client_at.cc

diff --git a/bta/hf_client/bta_hf_client_at.cc b/bta/hf_client/bta_hf_client_at.cc
index e6a7923..8bd55ed 100644
--- a/bta/hf_client/bta_hf_client_at.cc
+++ b/bta/hf_client/bta_hf_client_at.cc
@@ -1246,7 +1246,7 @@
 
     if (res2 == 0) {
       res2 = sscanf(buffer, ",\"\",%hu%n", &type, &offset);
-      if (res < 0) return NULL;
+      if (res2 < 0) return NULL;
 
       /* numstr is not matched in second attempt, correct this */
       res2++;