Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix child_process.kill oddities
* When the process is already dead, but the `exit` signal wasn't raised
  yet, the ESRCH error should be ignored.

* When an invalid signal is specified, kill() should throw.

* Like process.kill(), child_process.kill() now preserves a `0` signal
  which can be used to check the liveliness of the child process.

* process.kill() and child_process.kill() will now return true if the
  signal was actually delivered, and false otherwise.

* When an `exec`-ed process is automatically killed because a time or
  buffer limit is exceeded, and the kill() fails, this error should be
  reported through the `exec` callback.

Fixes: #3409
  • Loading branch information
piscisaureus committed Jun 12, 2012
1 parent b53b8b8 commit 10f85fa
Show file tree
Hide file tree
Showing 2 changed files with 43 additions and 10 deletions.
51 changes: 41 additions & 10 deletions lib/child_process.js
Expand Up @@ -536,12 +536,24 @@ exports.execFile = function(file /* args, options, callback */) {
}
}

function errorhandler(e) {
err = e;
child.stdout.destroy();
child.stderr.destroy();
exithandler();
}

function kill() {
child.stdout.destroy();
child.stderr.destroy();

killed = true;
child.kill(options.killSignal);
process.nextTick(function() {
exithandler(null, options.killSignal);
});
try {
child.kill(options.killSignal);
} catch (e) {
err = e;
exithandler();
}
}

if (options.timeout > 0) {
Expand Down Expand Up @@ -571,6 +583,7 @@ exports.execFile = function(file /* args, options, callback */) {
});

child.addListener('close', exithandler);
child.addListener('error', errorhandler);

return child;
};
Expand Down Expand Up @@ -822,25 +835,43 @@ function errnoException(errorno, syscall, errmsg) {


ChildProcess.prototype.kill = function(sig) {
var signal;

if (!constants) {
constants = process.binding('constants');
}

sig = sig || 'SIGTERM';
var signal = constants[sig];
if (sig === 0) {
signal = 0;
} else if (!sig) {
signal = constants['SIGTERM'];
} else {
signal = constants[sig];
}

if (!signal) {
if (signal === undefined) {
throw new Error('Unknown signal: ' + sig);
}

if (this._handle) {
this.killed = true;
var r = this._handle.kill(signal);
if (r === -1) {
if (r == 0) {
/* Success. */
this.killed = true;
return true;
} else if (errno == 'ESRCH') {
/* Already dead. */
} else if (errno == 'EINVAL' || errno == 'ENOSYS') {
/* The underlying platform doesn't support this signal. */
throw errnoException(errno, 'kill');
} else {
/* Other error, almost certainly EPERM. */
this.emit('error', errnoException(errno, 'kill'));
return;
}
}

/* Kill didn't succeed. */
return false;
};


Expand Down
2 changes: 2 additions & 0 deletions src/node.js
Expand Up @@ -430,6 +430,8 @@
if (r) {
throw errnoException(errno, 'kill');
}

return true;
};
};

Expand Down

0 comments on commit 10f85fa

Please sign in to comment.