inspector: check if connected before waiting

Fixes: #10093 PR-URL: #10094 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>

Author: Eugene Ostroukhov

src/inspector_agent.cc

@@ -548,10 +548,12 @@ bool AgentImpl::IsStarted(){
 }
 
 voidAgentImpl::WaitForDisconnect(){
- shutting_down_ = true;
-fprintf(stderr, "Waiting for the debugger to disconnect...\n");
-fflush(stderr);
- inspector_->runMessageLoopOnPause(0);
+if (state_ == State::kConnected){
+ shutting_down_ = true;
+fprintf(stderr, "Waiting for the debugger to disconnect...\n");
+fflush(stderr);
+ inspector_->runMessageLoopOnPause(0);
+ }
 }
 
 #defineREADONLY_PROPERTY(obj, str, var) \

test/inspector/test-inspector-stops-no-file.js

@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+constspawn=require('child_process').spawn;
+
+constchild=spawn(process.execPath,
+['--inspect','no-such-script.js'],
+{'stdio': 'inherit'});
+
+functionsignalHandler(value){
+child.kill();
+process.exit(1);
+}
+
+process.on('SIGINT',signalHandler);
+process.on('SIGTERM',signalHandler);