From: Francis De Brabandere <francisdb@gmail.com> Set on interp_int / interp_double / interp_date and strip at every boundary that's not the comparison consumer. var_cmp dispatches on the flag so BSTR vs literal numeric goes through numeric coercion (error 13 on parse fail) while BSTR vs non-literal numeric uses string compare. Const references emit OP_strip_reserved after inlining so they behave like variable reads. --- dlls/vbscript/compile.c | 18 ++++-- dlls/vbscript/global.c | 9 +++ dlls/vbscript/interp.c | 108 ++++++++++++++++++++++++----------- dlls/vbscript/tests/lang.vbs | 46 ++++++++------- dlls/vbscript/vbdisp.c | 5 ++ dlls/vbscript/vbscript.h | 1 + 6 files changed, 129 insertions(+), 58 deletions(-) diff --git a/dlls/vbscript/compile.c b/dlls/vbscript/compile.c index 16e661c98ff..5ae679d7828 100644 --- a/dlls/vbscript/compile.c +++ b/dlls/vbscript/compile.c @@ -510,8 +510,13 @@ static HRESULT compile_member_call_expression(compile_ctx_t *ctx, member_express expression_t *const_expr; const_expr = lookup_const_decls(ctx, expr->identifier, TRUE); - if(const_expr) - return compile_expression(ctx, const_expr); + if(const_expr) { + HRESULT hres; + hres = compile_expression(ctx, const_expr); + if(FAILED(hres)) + return hres; + return push_instr(ctx, OP_strip_reserved) ? S_OK : E_OUTOFMEMORY; + } } if(expr->obj_expr) { @@ -536,8 +541,13 @@ static HRESULT compile_member_expression(compile_ctx_t *ctx, member_expression_t if (!lookup_dim_decls(ctx, expr->identifier) && !lookup_args_name(ctx, expr->identifier)) { const_expr = lookup_const_decls(ctx, expr->identifier, TRUE); - if(const_expr) - return compile_expression(ctx, const_expr); + if(const_expr) { + HRESULT hres; + hres = compile_expression(ctx, const_expr); + if(FAILED(hres)) + return hres; + return push_instr(ctx, OP_strip_reserved) ? S_OK : E_OUTOFMEMORY; + } } return push_instr_bstr(ctx, OP_ident, expr->identifier); } diff --git a/dlls/vbscript/global.c b/dlls/vbscript/global.c index 7c0041a76ce..2904dc009a6 100644 --- a/dlls/vbscript/global.c +++ b/dlls/vbscript/global.c @@ -259,6 +259,9 @@ static HRESULT WINAPI Builtin_Invoke(IDispatch *iface, DISPID id, REFIID riid, L args[i] = *V_VARIANTREF(dp->rgvarg+dp->cArgs-i-1); else args[i] = dp->rgvarg[dp->cArgs-i-1]; + /* Strip VT_RESERVED — vbscript-internal flag must not reach + * VariantChangeType / VarAdd / etc. inside builtin proc bodies. */ + V_VT(&args[i]) &= ~VT_RESERVED; } hres = prop->proc(This, args, dp->cArgs, res); @@ -416,6 +419,9 @@ HRESULT to_int(VARIANT *v, int *ret) VARIANT r; HRESULT hres; + /* VariantChangeType rejects any flags including VT_RESERVED, so strip + * before calling it. The flag is purely an internal vbscript marker. */ + V_VT(v) &= ~VT_RESERVED; V_VT(&r) = VT_EMPTY; hres = VariantChangeType(&r, v, 0, VT_I4); if(FAILED(hres)) @@ -430,6 +436,7 @@ HRESULT to_double(VARIANT *v, double *ret) VARIANT dst; HRESULT hres; + V_VT(v) &= ~VT_RESERVED; V_VT(&dst) = VT_EMPTY; hres = VariantChangeType(&dst, v, 0, VT_R8); if(FAILED(hres)) @@ -444,6 +451,7 @@ static HRESULT to_float(VARIANT *v, float *ret) VARIANT dst; HRESULT hres; + V_VT(v) &= ~VT_RESERVED; V_VT(&dst) = VT_EMPTY; hres = VariantChangeType(&dst, v, 0, VT_R4); if(FAILED(hres)) @@ -458,6 +466,7 @@ static HRESULT to_string(VARIANT *v, BSTR *ret) VARIANT dst; HRESULT hres; + V_VT(v) &= ~VT_RESERVED; V_VT(&dst) = VT_EMPTY; hres = VariantChangeType(&dst, v, VARIANT_LOCALBOOL, VT_BSTR); if(FAILED(hres)) diff --git a/dlls/vbscript/interp.c b/dlls/vbscript/interp.c index 401e0d25c5a..8e1e057ad92 100644 --- a/dlls/vbscript/interp.c +++ b/dlls/vbscript/interp.c @@ -436,6 +436,14 @@ static HRESULT stack_pop_val(exec_ctx_t *ctx, variant_val_t *r) r->v = &r->store; } + /* VT_RESERVED is set on numeric literal pushes (interp_int / interp_double + * / interp_date) to flag the value for the literal-vs-BSTR comparison + * rule. The flag confuses Wine's VarAdd / VariantChangeType / etc. (they + * reject extra flags), so strip it once we leave the stack. cmp_oper / + * interp_case capture the flag from stack_top before calling this. */ + if(r->owned) + V_VT(r->v) &= ~VT_RESERVED; + return S_OK; } @@ -576,6 +584,14 @@ static void vbstack_to_dp(exec_ctx_t *ctx, unsigned arg_cnt, BOOL is_propput, DI }else { dp->rgvarg = is_propput ? ctx->stack+ctx->top-1 : NULL; } + + /* Strip VT_RESERVED — once a literal value flows into a function arg or + * property put it loses its literal-marker. */ + if(dp->rgvarg) { + unsigned i; + for(i = 0; i < dp->cArgs; i++) + V_VT(&dp->rgvarg[i]) &= ~VT_RESERVED; + } } HRESULT array_access(SAFEARRAY *array, DISPPARAMS *dp, VARIANT **ret) @@ -1142,6 +1158,9 @@ static HRESULT interp_const(exec_ctx_t *ctx) return hres; *v = *stack_pop(ctx); + /* Strip VT_RESERVED — internal flag from numeric-literal pushes; once a + * value is stored as a Const variable it must not carry it. */ + V_VT(v) &= ~VT_RESERVED; return S_OK; } @@ -1886,6 +1905,16 @@ static HRESULT interp_string(exec_ctx_t *ctx) return stack_push(ctx, &v); } +/* Clears VT_RESERVED on the top-of-stack variant. Emitted after a Const + * reference is inlined into a literal-emitting expression: the inlined value + * must not be treated as a bare literal by the comparison dispatcher. */ +static HRESULT interp_strip_reserved(exec_ctx_t *ctx) +{ + TRACE("\n"); + V_VT(stack_top(ctx, 0)) &= ~VT_RESERVED; + return S_OK; +} + static HRESULT interp_date(exec_ctx_t *ctx) { const DATE *d = ctx->instr->arg1.date; @@ -1893,7 +1922,10 @@ static HRESULT interp_date(exec_ctx_t *ctx) TRACE("%lf\n",*d); - V_VT(&v) = VT_DATE; + /* VT_RESERVED marks bare numeric literals so that the comparison + * dispatcher in var_cmp can apply the literal-vs-non-literal rule. + * Stripped at any boundary other than the immediate comparison. */ + V_VT(&v) = VT_DATE | VT_RESERVED; V_DATE(&v) = *d; return stack_push(ctx, &v); @@ -1907,10 +1939,10 @@ static HRESULT interp_int(exec_ctx_t *ctx) TRACE("%ld\n", arg); if(arg == (INT16)arg) { - V_VT(&v) = VT_I2; + V_VT(&v) = VT_I2 | VT_RESERVED; V_I2(&v) = arg; }else { - V_VT(&v) = VT_I4; + V_VT(&v) = VT_I4 | VT_RESERVED; V_I4(&v) = arg; } return stack_push(ctx, &v); @@ -1923,7 +1955,7 @@ static HRESULT interp_double(exec_ctx_t *ctx) TRACE("%lf\n", *arg); - V_VT(&v) = VT_R8; + V_VT(&v) = VT_R8 | VT_RESERVED; V_R8(&v) = *arg; return stack_push(ctx, &v); } @@ -2107,36 +2139,28 @@ static HRESULT interp_imp(exec_ctx_t *ctx) return stack_push(ctx, &v); } -/* Numeric types that take the "BSTR coerces to number, parse failure raises - * type-mismatch" path on Windows. The default integer types VT_I2/VT_I4 fit - * here; VT_R4/VT_R8/VT_DATE conceptually fit too, but only as bare literals - * (Windows distinguishes literal vs non-literal), and we have no way to track - * literal-ness without invasive plumbing — so they stay in the string-compare - * path which is correct for the common non-literal case. */ +/* Any numeric variant type — together with the VT_RESERVED literal marker + * (set on interp_int / interp_double / interp_date) this drives the + * literal-vs-non-literal dispatch in var_cmp. */ static inline BOOL is_numeric_vt(VARTYPE vt) { - return vt == VT_I2 || vt == VT_I4; -} - -/* Numeric types that take the "convert numeric side to CStr and string-compare" - * path on Windows. Non-numeric BSTRs return False with no error and relational - * operators use binary lex order. VT_BOOL behaves the same way but with - * hardcoded "True"/"False" since VarBstrFromBool yields "-1"/"0". */ -static inline BOOL is_string_compared_vt(VARTYPE vt) -{ - return vt == VT_UI1 || vt == VT_CY || vt == VT_R4 || vt == VT_R8 || vt == VT_DATE; + return vt == VT_I2 || vt == VT_I4 || vt == VT_R4 || vt == VT_R8 + || vt == VT_DATE || vt == VT_CY || vt == VT_UI1 || vt == VT_DECIMAL; } -static HRESULT var_cmp(exec_ctx_t *ctx, VARIANT *l, VARIANT *r) +static HRESULT var_cmp(exec_ctx_t *ctx, VARIANT *l, VARIANT *r, BOOL l_lit, BOOL r_lit) { VARTYPE lvt = V_VT(l) & VT_TYPEMASK; VARTYPE rvt = V_VT(r) & VT_TYPEMASK; TRACE("%s %s\n", debugstr_variant(l), debugstr_variant(r)); - /* VarCmp would use string comparison; VBScript converts the string to a number. */ - if((lvt == VT_BSTR && is_numeric_vt(rvt)) || - (rvt == VT_BSTR && is_numeric_vt(lvt))) { + /* BSTR vs numeric LITERAL: coerce BSTR to a number and numeric-compare; + * parse failure raises type-mismatch (error 13). VT_RESERVED is set by + * interp_int / interp_double / interp_date and stripped at any boundary + * other than the immediate comparison consumer. */ + if((lvt == VT_BSTR && is_numeric_vt(rvt) && r_lit) || + (rvt == VT_BSTR && is_numeric_vt(lvt) && l_lit)) { double dl, dr; HRESULT hres; @@ -2153,12 +2177,13 @@ static HRESULT var_cmp(exec_ctx_t *ctx, VARIANT *l, VARIANT *r) return VARCMP_EQ; } - /* VarCmp returns "BSTR always greater" for BSTR vs VT_BOOL/UI1/CY; VBScript - * instead coerces the numeric side to its CStr form and string-compares. - * (VarBstrFromBool yields "-1"/"0" rather than "True"/"False", so the bool - * strings are hardcoded.) */ - if((lvt == VT_BSTR && (rvt == VT_BOOL || is_string_compared_vt(rvt))) || - (rvt == VT_BSTR && (lvt == VT_BOOL || is_string_compared_vt(lvt)))) { + /* BSTR vs numeric (function-call result, variable, etc.) or VT_BOOL: + * coerce the numeric/bool side to its CStr form and string-compare. No + * error on unparseable BSTR; relational uses binary lex order. + * VarBstrFromBool yields "-1"/"0" rather than VBScript's "True"/"False", + * so bool strings are hardcoded. */ + if((lvt == VT_BSTR && (is_numeric_vt(rvt) || rvt == VT_BOOL)) || + (rvt == VT_BSTR && (is_numeric_vt(lvt) || lvt == VT_BOOL))) { VARIANT *num = lvt == VT_BSTR ? r : l; VARIANT *str = lvt == VT_BSTR ? l : r; VARIANT num_str; @@ -2187,15 +2212,21 @@ static HRESULT var_cmp(exec_ctx_t *ctx, VARIANT *l, VARIANT *r) static HRESULT cmp_oper(exec_ctx_t *ctx) { variant_val_t l, r; + BOOL l_lit, r_lit; HRESULT hres; + /* Capture VT_RESERVED ("this side is a numeric literal") before + * stack_pop_val strips it. */ + r_lit = (V_VT(stack_top(ctx, 0)) & VT_RESERVED) != 0; + l_lit = (V_VT(stack_top(ctx, 1)) & VT_RESERVED) != 0; + hres = stack_pop_val(ctx, &r); if(FAILED(hres)) return hres; hres = stack_pop_val(ctx, &l); if(SUCCEEDED(hres)) { - hres = var_cmp(ctx, l.v, r.v); + hres = var_cmp(ctx, l.v, r.v, l_lit, r_lit); release_val(&l); } @@ -2315,15 +2346,20 @@ static HRESULT interp_case(exec_ctx_t *ctx) { const unsigned arg = ctx->instr->arg1.uint; variant_val_t v; + BOOL test_lit, case_lit; HRESULT hres; TRACE("%d\n", arg); + /* Capture VT_RESERVED before stack_pop_val strips it. */ + case_lit = (V_VT(stack_top(ctx, 0)) & VT_RESERVED) != 0; + test_lit = (V_VT(stack_top(ctx, 1)) & VT_RESERVED) != 0; + hres = stack_pop_val(ctx, &v); if(FAILED(hres)) return hres; - hres = var_cmp(ctx, stack_top(ctx, 0), v.v); + hres = var_cmp(ctx, stack_top(ctx, 0), v.v, test_lit, case_lit); release_val(&v); if(FAILED(hres)) { if(hres == DISP_E_TYPEMISMATCH) { @@ -2624,6 +2660,11 @@ static HRESULT interp_incc(exec_ctx_t *ctx) return E_FAIL; } + /* Strip VT_RESERVED from the increment operand on the stack — VarAdd + * rejects extra flags. The literal value here ("1" for default For step) + * came from interp_int with VT_RESERVED set. */ + V_VT(stack_top(ctx, 0)) &= ~VT_RESERVED; + hres = VarAdd(stack_top(ctx, 0), ref.u.v, &v); if(FAILED(hres)) return hres; @@ -2927,6 +2968,9 @@ HRESULT exec_script(script_ctx_t *ctx, BOOL extern_caller, function_t *func, vbd if(SUCCEEDED(hres) && res) { *res = exec.ret_val; V_VT(&exec.ret_val) = VT_EMPTY; + /* Strip VT_RESERVED — the flag is internal to the comparison + * dispatcher; callers (e.g. ParseScriptText) shouldn't see it. */ + V_VT(res) &= ~VT_RESERVED; } ctx->current_named_item = prev_named_item; diff --git a/dlls/vbscript/tests/lang.vbs b/dlls/vbscript/tests/lang.vbs index 3b03a4c06e8..2558bc66ac9 100644 --- a/dlls/vbscript/tests/lang.vbs +++ b/dlls/vbscript/tests/lang.vbs @@ -235,15 +235,15 @@ Dim saved_err on error resume next err.clear x = ("abc" = 5.5) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 13, "literal R8: ""abc"" = 5.5 should error 13 (got " & saved_err & "; needs literal tracking)") +Call ok(saved_err = 13, "literal R8: ""abc"" = 5.5 should error 13 (got " & saved_err & "; needs literal tracking)") x = ("abc" = 1e2) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 13, "literal sci: ""abc"" = 1e2 should error 13 (got " & saved_err & "; needs literal tracking)") +Call ok(saved_err = 13, "literal sci: ""abc"" = 1e2 should error 13 (got " & saved_err & "; needs literal tracking)") x = ("abc" = &hff) : saved_err = err.number : err.clear Call ok(saved_err = 13, "literal hex: ""abc"" = &hff should error 13 (got " & saved_err & ")") x = ("abc" = 100000) : saved_err = err.number : err.clear Call ok(saved_err = 13, "literal I4: ""abc"" = 100000 should error 13 (got " & saved_err & ")") x = ("abc" = #1/15/2024#): saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 13, "literal date: ""abc"" = #1/15/2024# should error 13 (got " & saved_err & "; needs literal tracking)") +Call ok(saved_err = 13, "literal date: ""abc"" = #1/15/2024# should error 13 (got " & saved_err & "; needs literal tracking)") on error goto 0 ' --- Variable holding a numeric / arithmetic / unary / function return --- @@ -258,38 +258,38 @@ Dim n10 : n10 = 10 on error resume next err.clear x = ("abc" = n5) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 0, "var: ""abc"" = (n=5) should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "var: ""abc"" = (n=5) should not raise (got " & saved_err & ")") err.clear -x = ("abc" = n5) : Call todo_wine_ok(err.number = 0 and not x, "var: ""abc"" = (n=5) should be false") +x = ("abc" = n5) : Call ok(err.number = 0 and not x, "var: ""abc"" = (n=5) should be false") err.clear -x = ("010" = n10) : Call todo_wine_ok(err.number = 0 and not x, "var: ""010"" = (n=10) should be false (string)") +x = ("010" = n10) : Call ok(err.number = 0 and not x, "var: ""010"" = (n=10) should be false (string)") err.clear x = ("abc" = (5+0)) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 0, "arith: ""abc"" = (5+0) should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "arith: ""abc"" = (5+0) should not raise (got " & saved_err & ")") err.clear -x = ("010" = (5+5)) : Call todo_wine_ok(err.number = 0 and not x, "arith: ""010"" = (5+5) should be false") +x = ("010" = (5+5)) : Call ok(err.number = 0 and not x, "arith: ""010"" = (5+5) should be false") err.clear -x = ("010" = (10*1)) : Call todo_wine_ok(err.number = 0 and not x, "arith: ""010"" = (10*1) should be false") +x = ("010" = (10*1)) : Call ok(err.number = 0 and not x, "arith: ""010"" = (10*1) should be false") err.clear x = ("abc" = -5) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 0, "neg: ""abc"" = -5 should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "neg: ""abc"" = -5 should not raise (got " & saved_err & ")") on error goto 0 ' --- C-coercion functions return non-literal values. --- ' VT_I2/I4 from CInt/CLng still go through the numeric path on Wine (no literal ' tracking), so these remain todo_wine. -Call todo_wine_ok(not ("010" = CInt(10)), "CInt: ""010"" = CInt(10) should be false") -Call todo_wine_ok(not ("010" = CLng(10)), "CLng: ""010"" = CLng(10) should be false") +Call ok(not ("010" = CInt(10)), "CInt: ""010"" = CInt(10) should be false") +Call ok(not ("010" = CLng(10)), "CLng: ""010"" = CLng(10) should be false") ' VT_R4/VT_R8 from CSng/CDbl take the string-compare path now. Call ok(not ("010" = CSng(10)), "CSng: ""010"" = CSng(10) should be false") Call ok(not ("010" = CDbl(10)), "CDbl: ""010"" = CDbl(10) should be false") on error resume next err.clear x = ("abc" = CInt(5)) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 0, "CInt: ""abc"" = CInt(5) should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "CInt: ""abc"" = CInt(5) should not raise (got " & saved_err & ")") err.clear x = ("abc" = CLng(5)) : saved_err = err.number : err.clear -Call todo_wine_ok(saved_err = 0, "CLng: ""abc"" = CLng(5) should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "CLng: ""abc"" = CLng(5) should not raise (got " & saved_err & ")") err.clear x = ("abc" = CSng(5)) : saved_err = err.number : err.clear Call ok(saved_err = 0, "CSng: ""abc"" = CSng(5) should not raise (got " & saved_err & ")") @@ -319,7 +319,7 @@ on error resume next err.clear x = ("abc" = GetFiveLit()) : saved_err = err.number : err.clear on error goto 0 -Call todo_wine_ok(saved_err = 0, "fn return: ""abc"" = GetFiveLit() should not raise (got " & saved_err & ")") +Call ok(saved_err = 0, "fn return: ""abc"" = GetFiveLit() should not raise (got " & saved_err & ")") Sub TestByValStripsLit(ByVal v) Dim local_err @@ -327,7 +327,7 @@ Sub TestByValStripsLit(ByVal v) err.clear x = ("abc" = v) : local_err = err.number : err.clear on error goto 0 - Call todo_wine_ok(local_err = 0, "ByVal: ""abc"" = v should not raise (got " & local_err & ")") + Call ok(local_err = 0, "ByVal: ""abc"" = v should not raise (got " & local_err & ")") End Sub TestByValStripsLit 5 @@ -337,20 +337,22 @@ Sub TestByRefStripsLit(ByRef v) err.clear x = ("abc" = v) : local_err = err.number : err.clear on error goto 0 - Call todo_wine_ok(local_err = 0, "ByRef: ""abc"" = v should not raise (got " & local_err & ")") + Call ok(local_err = 0, "ByRef: ""abc"" = v should not raise (got " & local_err & ")") End Sub Dim litvar : litvar = 5 TestByRefStripsLit litvar -' --- Const inlines at compile-time in Wine, so the value is treated as a -' literal and raises error 13; on Windows Const acts like a variable. -' Tracked separately from the var_cmp fix; remains todo_wine for now. --- +' Const reads through normal variable lookup at runtime — the stored value +' has VT_RESERVED stripped, so "abc" = FIVE goes through the BSTR-vs-numeric +' string-compare path and returns False with no error. Const FIVE_C = 5 on error resume next err.clear -x = ("abc" = FIVE_C) : saved_err = err.number : err.clear +x = ("abc" = FIVE_C) +saved_err = err.number +err.clear on error goto 0 -Call todo_wine_ok(saved_err = 0, "Const: ""abc"" = FIVE should not raise (got " & saved_err & "; needs compile-time fix)") +Call ok(saved_err = 0, "Const: ""abc"" = FIVE should not raise (got " & saved_err & ")") Call ok(getVT(false) = "VT_BOOL", "getVT(false) is not VT_BOOL") Call ok(getVT(true) = "VT_BOOL", "getVT(true) is not VT_BOOL") diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index 9f6cf2e446a..e0080d38f73 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -188,6 +188,10 @@ static HRESULT invoke_variant_prop(script_ctx_t *ctx, VARIANT *v, WORD flags, DI if(res) V_VT(res) = VT_EMPTY; + /* Strip VT_RESERVED — internal vbscript flag, must not be stored on + * variables (VariantCopyInd would later reject it). */ + V_VT(&put_val) &= ~VT_RESERVED; + if(own_val) *v = put_val; else @@ -1859,6 +1863,7 @@ HRESULT disp_call(script_ctx_t *ctx, IDispatch *disp, DISPID id, DISPPARAMS *dp, ctx->ei = ei; hres = SCRIPT_E_RECORDED; } + FIXME("DEBUG disp_call END disp=%p id=%ld hres=%#lx retv_VT=%#x\n", disp, id, hres, retv ? V_VT(retv) : 0xDEADBEEF); return hres; } diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h index 6ff6e8ff8eb..0f12ea355e8 100644 --- a/dlls/vbscript/vbscript.h +++ b/dlls/vbscript/vbscript.h @@ -310,6 +310,7 @@ typedef enum { X(step, 0, ARG_ADDR, ARG_BSTR) \ X(stop, 1, 0, 0) \ X(string, 1, ARG_STR, 0) \ + X(strip_reserved, 1, 0, 0) \ X(sub, 1, 0, 0) \ X(val, 1, 0, 0) \ X(vcall, 1, ARG_UINT, 0) \ -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/10766