Fix OSError.__reduce__ to preserve winerror when filename is None

youknowone · youknowone · commit f10bc82f43d2 · 2026-02-23T02:18:01.000+09:00
When filename is None, __reduce__ was reconstructing a 2-element
(errno, msg) tuple, dropping the winerror at position 3 in the
original args. Use the original args tuple instead, matching CPython.
diff --git a/crates/vm/src/exceptions.rs b/crates/vm/src/exceptions.rs
@@ -2084,10 +2084,12 @@ pub(super) mod types {
                         }
                         result.push(args_reduced.into_pytuple(vm).into());
                     } else {
-                        result.push(vm.new_tuple((errno, msg)).into());
+                        // filename is None - use original args as-is
+                        // (may contain winerror at position 3)
+                        result.push(args.into());
                     }
                 } else {
-                    result.push(vm.new_tuple((errno, msg)).into());
+                    result.push(args.into());
                 }
             } else {
                 result.push(args.into());

Original file line number	Diff line number	Diff line change
`@@ -2084,10 +2084,12 @@ pub(super) mod types {`
`2084`	`2084`	`}`
`2085`	`2085`	`result.push(args_reduced.into_pytuple(vm).into());`
`2086`	`2086`	`} else {`
`2087`		`- result.push(vm.new_tuple((errno, msg)).into());`
	`2087`	`+ // filename is None - use original args as-is`
	`2088`	`+ // (may contain winerror at position 3)`
	`2089`	`+ result.push(args.into());`
`2088`	`2090`	`}`
`2089`	`2091`	`} else {`
`2090`		`- result.push(vm.new_tuple((errno, msg)).into());`
	`2092`	`+ result.push(args.into());`
`2091`	`2093`	`}`
`2092`	`2094`	`} else {`
`2093`	`2095`	`result.push(args.into());`