Skip to content

fix: Windows String conversion in uninstall - #29

Closed
da-kami wants to merge 1 commit into
Rubensei:masterfrom
da-kami:fix-uninstall
Closed

da-kami wants to merge 1 commit into
Rubensei:masterfrom
da-kami:fix-uninstall

Conversation

@da-kami

@da-kami da-kami commented Jul 14, 2026

Copy link
Copy Markdown

Calling the function failed with:

The filename, directory name, or volume label syntax is incorrect. (0x8007007B)

"WinDivert" is a Rust &str literal. .as_ptr() gives a pointer to the raw UTF‑8 bytes of that literal — but Rust string literals are not guaranteed to be null-terminated. Ensure that the name is properly terminated.

Calling the function failed with:

```
The filename, directory name, or volume label syntax is incorrect. (0x8007007B)
```

"WinDivert" is a Rust &str literal. .as_ptr() gives a pointer to the raw UTF‑8 bytes of that literal — but Rust string literals are not guaranteed to be null-terminated.
Ensure that the name is properly terminated.
@dilluti0n

Copy link
Copy Markdown
Contributor

FYI, I've opened #30 which fixes this along with a dangling SERVICE_STATUS pointer bug in the same function. I had both fixes sitting in dpibreak's tree and decided to upstream them together. There's overlap on the string fix; no strong preference on which one lands, I'll rebase mine accordingly.

@Rubensei

Copy link
Copy Markdown
Owner

Closing this on favor of #30

@Rubensei Rubensei closed this Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants