Fix CopyFile2 to raise proper OSError subclass

youknowone · youknowone · commit ccd4c670719d · 2026-02-20T23:53:19.000+09:00
Use std::io::Error::from_raw_os_error instead of vm.new_os_error so
that winerror attribute is set and errno-to-exception mapping works
(e.g. ERROR_ACCESS_DENIED → PermissionError).
diff --git a/crates/vm/src/stdlib/_winapi.rs b/crates/vm/src/stdlib/_winapi.rs
@@ -1944,7 +1944,9 @@ mod _winapi {
             } else {
                 hr as u32
             };
-            return Err(vm.new_os_error(err as i32));
+            return Err(
+                std::io::Error::from_raw_os_error(err as i32).to_pyexception(vm),
+            );
         }
         Ok(())
     }

Original file line number	Diff line number	Diff line change
`@@ -1944,7 +1944,9 @@ mod _winapi {`
`1944`	`1944`	`} else {`
`1945`	`1945`	`hr as u32`
`1946`	`1946`	`};`
`1947`		`- return Err(vm.new_os_error(err as i32));`
	`1947`	`+ return Err(`
	`1948`	`+ std::io::Error::from_raw_os_error(err as i32).to_pyexception(vm),`
	`1949`	`+ );`
`1948`	`1950`	`}`
`1949`	`1951`	`Ok(())`
`1950`	`1952`	`}`