fix review idea

Signed-off-by: Manjusaka <me@manjusaka.me>

Author: Zheaoli

Doc/library/sys.rst

@@ -1933,16 +1933,16 @@ always available. Unless explicitly noted otherwise, all variables are read-only
    interpreter is pre-release (alpha, beta, or release candidate) then the
    local and remote interpreters must be the same exact version.
 
-   .. audit-event:: remote_exec pid script_path
+   .. audit-event:: sys.remote_exec pid script_path
 
       When the code is executed in the remote process, an :ref:`auditing event <auditing>`
-      ``remote_exec`` is raised with the *pid* and the path to the script file.
+      ``sys.remote_exec`` is raised with the *pid* and the path to the script file.
 
-   .. audit-event:: remote_debugger_script script_path
+   .. audit-event:: cpython.remote_debugger_script script_path
 
       When the script is executed in the remote process, an
       :ref:`auditing event <auditing>`
-      ``sys.remote_debugger_script`` is raised
+      ``cpython.remote_debugger_script`` is raised
       with the path in the remote process.
 
    .. availability:: Unix, Windows.

Lib/test/audit-tests.py

@@ -650,7 +650,7 @@ def test_sys_remote_exec():
     event_pid = -1
     event_script_path = ""
     def hook(event, args):
-        if event != "remote_exec": return
+        if event != "sys.remote_exec": return
         nonlocal event_pid
         event_pid = args[0]
         nonlocal event_script_path

Lib/test/test_sys.py

@@ -2102,7 +2102,7 @@ def audit_hook(event, arg):
         returncode, stdout, stderr = self._run_remote_exec_test(script, prologue=prologue)
         self.assertEqual(returncode, 0)
         self.assertIn(b"Remote script executed successfully!", stdout)
-        self.assertIn(b"Audit event: remote_debugger_script, arg: ", stdout)
+        self.assertIn(b"Audit event: cpython.remote_debugger_script, arg: ", stdout)
         self.assertEqual(stderr, b"")
 
     def test_remote_exec_with_exception(self):

Python/ceval_gil.c

@@ -1220,7 +1220,7 @@ static inline int run_remote_debugger_source(PyObject *source)
 // that would be an easy target for a ROP gadget.
 static inline void run_remote_debugger_script(PyObject *path)
 {
-    if (0 != PySys_Audit("remote_debugger_script", "O", path)) {
+    if (0 != PySys_Audit("cpython.remote_debugger_script", "O", path)) {
         PyErr_FormatUnraisable(
             "Audit hook failed for remote debugger script %U", path);
         return;

Python/sysmodule.c

@@ -2485,13 +2485,14 @@ sys_remote_exec_impl(PyObject *module, int pid, PyObject *script)
     PyObject *path;
     const char *debugger_script_path;
 
-    if (PySys_Audit("remote_exec", "iO", pid, script) < 0) {
+    if (PyUnicode_FSConverter(script, &path) == 0) {
         return NULL;
     }
 
-    if (PyUnicode_FSConverter(script, &path) == 0) {
+    if (PySys_Audit("sys.remote_exec", "iO", pid, script) < 0) {
         return NULL;
     }
+
     debugger_script_path = PyBytes_AS_STRING(path);
 #ifdef MS_WINDOWS
     PyObject *unicode_path;