From e4363145bad7fe1bd63fbb2d41101de38477e7fb Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Thu, 25 Jul 2013 12:24:40 +0200 Subject: [PATCH] test: fix simple/test-setproctitle The title shouldn't be too long; libuv's uv_set_process_title() out of security considerations no longer overwrites envp, only argv, so the maximum title length is possibly quite short. Fixes #5908. --- test/simple/test-setproctitle.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/simple/test-setproctitle.js b/test/simple/test-setproctitle.js index 778e20012ed..6373af4cd18 100644 --- a/test/simple/test-setproctitle.js +++ b/test/simple/test-setproctitle.js @@ -31,7 +31,10 @@ var common = require('../common'); var assert = require('assert'); var exec = require('child_process').exec; -var title = 'testTestTESTtest123123123123123123HiHaiJo'; +// The title shouldn't be too long; libuv's uv_set_process_title() out of +// security considerations no longer overwrites envp, only argv, so the +// maximum title length is possibly quite short. +var title = 'testme'; assert.notEqual(process.title, title); process.title = title;