From: Victor Stinner <vstinner@python.org>
Date: Sat, 17 May 2025 22:24:40 +0000 (-0400)
Subject: gh-134064: Fix sys.remote_exec() error checking (#134067)
X-Git-Tag: v3.15.0a1~1688
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=009e7b36981fd07f7cca1fdcfcf172ce1584fac7;p=thirdparty%2FPython%2Fcpython.git

gh-134064: Fix sys.remote_exec() error checking (#134067)
---

diff --git a/Lib/test/test_sys.py b/Lib/test/test_sys.py
index 8af2e3488b48..fb1c8492a64d 100644
--- a/Lib/test/test_sys.py
+++ b/Lib/test/test_sys.py
@@ -2176,6 +2176,13 @@ raise Exception("Remote script exception")
         with self.assertRaises(OSError):
             sys.remote_exec(99999, "print('should not run')")
 
+    def test_remote_exec_invalid_script(self):
+        """Test remote exec with invalid script type"""
+        with self.assertRaises(TypeError):
+            sys.remote_exec(0, None)
+        with self.assertRaises(TypeError):
+            sys.remote_exec(0, 123)
+
     def test_remote_exec_syntax_error(self):
         """Test remote exec with syntax error in script"""
         script = '''
diff --git a/Python/sysmodule.c b/Python/sysmodule.c
index dd048205757e..4ed045e3297b 100644
--- a/Python/sysmodule.c
+++ b/Python/sysmodule.c
@@ -2485,7 +2485,7 @@ sys_remote_exec_impl(PyObject *module, int pid, PyObject *script)
     PyObject *path;
     const char *debugger_script_path;
 
-    if (PyUnicode_FSConverter(script, &path) < 0) {
+    if (PyUnicode_FSConverter(script, &path) == 0) {
         return NULL;
     }
     debugger_script_path = PyBytes_AS_STRING(path);