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

failed test: simple/test-setproctitle.js #5908

Closed
jbergstroem opened this issue Jul 26, 2013 · 2 comments
Closed

failed test: simple/test-setproctitle.js #5908

jbergstroem opened this issue Jul 26, 2013 · 2 comments
Labels

Comments

@jbergstroem
Copy link
Member

I consistently get a failed test running this. Below is output from running test suite on head (7d654be). The same applies to node v0.10.4.

== release test-setproctitle ===
Path: simple/test-setproctitle
assert.js:92
  throw new assert.AssertionError({
        ^
AssertionError: {"killed":false,"code":1,"signal":null} == null
    at /home/jbergstroem/node/test/simple/test-setproctitle.js:41:10
    at ChildProcess.exithandler (child_process.js:642:7)
    at ChildProcess.EventEmitter.emit (events.js:102:17)
    at maybeClose (child_process.js:736:16)
    at Socket.<anonymous> (child_process.js:949:11)
    at Socket.EventEmitter.emit (events.js:99:17)
    at Pipe.close (net.js:460:12)
Command: out/Release/node /home/jbergstroem/node/test/simple/test-setproctitle.js

If I invoke the test specifically, I get this:

 # ./node test/simple/test-setproctitle.js

assert.js:92
  throw new assert.AssertionError({
        ^
AssertionError: "testTestTESTtest123123123123123123HiHai" == "testTestTESTtest123123123123123123HiHaiJo"
    at Object.<anonymous> (/home/jbergstroem/node/test/simple/test-setproctitle.js:38:8)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)
    at Module.load (module.js:349:32)
    at Function.Module._load (module.js:305:12)
    at Function.Module.runMain (module.js:490:10)
    at startup (node.js:119:16)
    at node.js:819:3

This is Linux g2-x86_64 3.9.0-gentoo #1 SMP Tue Apr 30 16:35:08 EST 2013 x86_64 QEMU Virtual CPU version 0.14.1 GenuineIntel GNU/Linux running glibc 2.17, gcc 4.7.3

bnoordhuis added a commit that referenced this issue Jul 26, 2013
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.
@bnoordhuis
Copy link
Member

Thanks for the report, should be fixed in e436314.

@jbergstroem
Copy link
Member Author

FYI, the reason I got the other assertionError was because busybox ps doesn't accept -p. I suggest ignoring that :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants