diff options
author | Ben Noordhuis <info@bnoordhuis.nl> | 2013-03-14 16:59:07 +0100 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2013-03-25 13:23:17 +0100 |
commit | 44843a60629ab5951a9c54ac6bf74208d607e3e0 (patch) | |
tree | ac3b8cd6b353b017c7eb73d379976db750c7baa1 | |
parent | 05bd6b7bcd5e1e962413afe66564f2b60b0178e6 (diff) | |
download | node-44843a60629ab5951a9c54ac6bf74208d607e3e0.tar.gz |
child_process: fix sending utf-8 to child process
In process#send() and child_process.ChildProcess#send(), use 'utf8' as
the encoding instead of 'ascii' because 'ascii' mutilates non-ASCII
input. Correctly handle partial character sequences by introducing
a StringDecoder.
Sending over UTF-8 no longer works in v0.10 because the high bit of
each byte is now cleared when converting a Buffer to ASCII. See
commit 96a314b for details.
Fixes #4999 and #5011.
-rw-r--r-- | lib/child_process.js | 4 | ||||
-rw-r--r-- | test/simple/test-child-process-send-utf8.js | 34 |
2 files changed, 37 insertions, 1 deletions
diff --git a/lib/child_process.js b/lib/child_process.js index 80735d4d0..52435913e 100644 --- a/lib/child_process.js +++ b/lib/child_process.js @@ -19,6 +19,7 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. +var StringDecoder = require('string_decoder').StringDecoder; var EventEmitter = require('events').EventEmitter; var net = require('net'); var dgram = require('dgram'); @@ -321,11 +322,12 @@ function handleMessage(target, message, handle) { function setupChannel(target, channel) { target._channel = channel; + var decoder = new StringDecoder('utf8'); var jsonBuffer = ''; channel.buffering = false; channel.onread = function(pool, offset, length, recvHandle) { if (pool) { - jsonBuffer += pool.toString('ascii', offset, offset + length); + jsonBuffer += decoder.write(pool.slice(offset, offset + length)); var i, start = 0; diff --git a/test/simple/test-child-process-send-utf8.js b/test/simple/test-child-process-send-utf8.js new file mode 100644 index 000000000..b3ce24390 --- /dev/null +++ b/test/simple/test-child-process-send-utf8.js @@ -0,0 +1,34 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); +var fork = require('child_process').fork; + +var expected = Array(1e5).join('ßßßß'); +if (process.argv[2] === 'child') { + process.send(expected); +} else { + var child = fork(process.argv[1], ['child']); + child.on('message', common.mustCall(function(actual) { + assert.equal(actual, expected); + })); +} |