HACK: windows: Work around broken AssocQueryStringW() not returning actual string length

Wine has a bug where calling `AssocQueryStringW()` with a buffer doesn't
actually update `line_len` to the actual number of bytes returned,
resulting in `webbrowser` failing on seeing a bunch of unexpected `nul`
bytes in the array:

    Error { kind: InvalidInput, message: "nul byte found in provided data" }

This is reported and fixed upstream via:

https://bugs.winehq.org/show_bug.cgi?id=59402
https://gitlab.winehq.org/wine/wine/-/merge_requests/10072

I still need to write tests to get it merged, and even then it might
take a while to trickle down into releases and ultimately Steam Proton.
Hence have this ugly workaround to catch cases where `nul` characters
end up in the output and reduce the string length by hand.

Author: MarijnS95

src/windows.rs

@@ -1,6 +1,6 @@
 use crate::common::{for_each_token, run_command};
 use crate::{Browser, BrowserOptions, Error, ErrorKind, Result, TargetType};
-use log::trace;
+use log::{error, trace};
 use std::path::Path;
 use std::process::Command;
 
@@ -57,8 +57,19 @@ pub(super) fn open_browser_internal(
                     ));
                 }
 
+                let found_nul = cmdline_u16
+                    .iter()
+                    .position(|&c| c == 0)
+                    .unwrap_or(BUF_SIZE - 1);
+
+                let mut line_len = line_len as usize;
+                if found_nul < line_len {
+                    error!("Broken AssocQueryStringW(), {found_nul} < {line_len} == {BUF_SIZE}");
+                    line_len = found_nul;
+                }
+
                 use std::os::windows::ffi::OsStringExt;
-                std::ffi::OsString::from_wide(&cmdline_u16[..(line_len - 1) as usize])
+                std::ffi::OsString::from_wide(&cmdline_u16[..(line_len - 1)])
                     .into_string()
                     .map_err(|_err| {
                         Error::new(