Write output corruption when spawning pwsh about pywinpty HOT 4 CLOSED

Yep. Right here: https://github.com/andfoy/winpty-rs/blob/3b08c65244630fb5c1b7772b0295f317e92e98b9/src/pty/base.rs#L444-L454

encode_wide does not add a null terminator, but you are calling WideCharToMultiByte with -1 which tells it to treat it as a null terminated string. Undefined Behavior! as the Rust books like to proclaim. 😝

EDIT: But you did manually add a null terminator, so that's not the issue... time to keep digging...

EDIT 2: So actually caused by writing the nul byte through to the PTY. I will send a PR.

@andfoy

from pywinpty.

The fix was merged. Thanks @andfoy. But winpty-rs and this module still need to be released for the fix to be picked up. (Just an FYI in case anyone hits this).

P.S. This module seems to have not committed the Cargo.lock which means it will build with whatever happens to be the latest matching versions of winpty-rs and other libraries. I think this is a case where you should commit the Cargo.lock for reproducible builds (As this is a cdylib case), just remember to update it when needed.

from pywinpty.

@andfoy is the maintainer of this project as well (as you can see in the commit history).

from pywinpty.

@andfoy is the maintainer of this project as well (as you can see in the commit history).

Yeah. I know. It's up to him when to make a new release of this.

from pywinpty.