Long environment strings are being truncated.

[rprichard]

Long environment strings are being truncated.

[rprichard]

Thanks for reporting this issue. I'll try to look at it this weekend when I get time.

[rprichard]

FWIW, the "unix adapter" in winpty passes NULL for the winpty_start_process function. It doesn't send anything over IPC. I see that https://github.com/chjj/pty.js/blob/master/src/win/pty.cc is passing a non-NULL env parameter, though.

I think there may be misunderstanding about that parameter. From the winpty.h header:

• Start a child process. Either (but not both) of appname and cmdline may
• be NULL. cwd and env may be NULL. env is a pointer to an environment
• block like that passed to CreateProcess.

From pty.js/src/win/pty.cc:

const wchar_t *env = ToWChar(String::Utf8Value(args[3]->ToString()));

AFAICT, the string returned by ToWChar will have no embedded NUL characters and will end with one NUL character. I was intending the winpty_start_process API to accept an environment block like that of CreateProcess -- a sequence of VAR=VAL strings, separated by NUL characters, terminated by two NULs (one for the last string and one for the block itself).

That's also the reason for the p++ on https://github.com/rprichard/winpty/blob/master/libwinpty/winpty.cc#L373. The p++ is skipping over the final NUL terminator of the environment block. The following envStr.assign(env, p); line then assigns the (p - env) characters to envStr, so that envStr is the complete double-NUL-terminated environment block.

[peters]

@rprichard Sorry about that, but you are looking at the wrong source code. Please see https://github.com/peters/pty.js/blob/master/src/win/pty.cc

[peters]

Fixed in peters/pty.js@e18e8f9