Fix debugserver stop

Summary:
For some reason the debugserver stop command called status by mistake! This means that `debugserver stop` wouldn't actually kill the debugserver.

Now it does.

Reviewed By: jamescoggan

Differential Revision: D33479256

fbshipit-source-id: e4f6b4b85eae800d2ac8a6b275c47d68bbb398e6

FBDeviceControl/Management/FBDeviceDebugServer.m

@@ -168,6 +168,7 @@ - (void)socketServer:(FBSocketServer *)server clientConnected:(struct in6_addr)a
     [self.logger log:@"Client Disconnected"];
     self.twistedPair = nil;
   }];
+  [self.teardown resolveFromFuture:completed];
   self.twistedPair = twistedPair;
 }
 

idb_companion/Server/FBIDBCommandExecutor.m

@@ -359,10 +359,13 @@ - (instancetype)initWithTarget:(id<FBiOSTarget>)target storageManager:(FBIDBStor
       failFuture];
   }
 
-  return [[self
+  return [[[self
     debugserver_prepare:bundleID]
     onQueue:self.target.workQueue fmap:^(FBBundleDescriptor *application) {
       return [commands launchDebugServerForHostApplication:application port:self.ports.debugserverPort];
+    }]
+    onQueue:self.target.workQueue doOnResolved:^(id<FBDebugServer> debugServer) {
+      self.debugServer = debugServer;
     }];
 }
 

idb_companion/Server/FBIDBServiceHandler.mm

@@ -1507,7 +1507,7 @@ static void populate_companion_info(idb::CompanionInfo *info, id<FBEventReporter
       return Status::OK;
     }
     case idb::DebugServerRequest::ControlCase::kStop: {
-      id<FBDebugServer> debugServer = [[_commandExecutor debugserver_status] block:&error];
+      id<FBDebugServer> debugServer = [[_commandExecutor debugserver_stop] block:&error];
       if (!debugServer) {
         return Status(grpc::StatusCode::INTERNAL, error.localizedDescription.UTF8String);
       }