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

Commit

Permalink
test: fix simple/test-setproctitle
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
bnoordhuis committed Jul 25, 2013
1 parent 180f987 commit e436314
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion test/simple/test-setproctitle.js
Expand Up @@ -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;
Expand Down

0 comments on commit e436314

Please sign in to comment.