commit 99b7c20caa1ad6f67b027f9fceecabd6e7b76bba
author Treehugger Robot <treehugger-gerrit@google.com> Mon Mar 20 23:24:32 2017 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Mar 20 23:24:33 2017 +0000
tree 4a60bc50de1453b473fddbc6791291c339a29a9b
parent b84c43f056231838b5cedacd4ad756135ef8dd04
parent cbb65ae216a47a080c2ff301b593f86d7ac2c1a2

Merge "jdwp: switch to SOCK_SEQPACKET."

runtime/jdwp/jdwp_adb.cc

diff --git a/runtime/jdwp/jdwp_adb.cc b/runtime/jdwp/jdwp_adb.cc
index b13d565..0aa04c1 100644
--- a/runtime/jdwp/jdwp_adb.cc
+++ b/runtime/jdwp/jdwp_adb.cc
@@ -227,7 +227,7 @@
     const int  sleep_max_ms = 2*1000;
     char       buff[5];
 
-    int sock = socket(PF_UNIX, SOCK_STREAM, 0);
+    int sock = socket(AF_UNIX, SOCK_SEQPACKET, 0);
     if (sock < 0) {
       PLOG(ERROR) << "Could not create ADB control socket";
       return false;
@@ -264,7 +264,7 @@
        * up after a few minutes in case somebody ships an app with
        * the debuggable flag set.
        */
-      int  ret = connect(ControlSock(), &control_addr_.controlAddrPlain, control_addr_len_);
+      int ret = connect(ControlSock(), &control_addr_.controlAddrPlain, control_addr_len_);
       if (!ret) {
         int control_sock = ControlSock();
 #ifdef ART_TARGET_ANDROID
@@ -278,7 +278,7 @@
 
         /* now try to send our pid to the ADB daemon */
         ret = TEMP_FAILURE_RETRY(send(control_sock, buff, 4, 0));
-        if (ret >= 0) {
+        if (ret == 4) {
           VLOG(jdwp) << StringPrintf("PID sent as '%.*s' to ADB", 4, buff);
           break;
         }