nodejs
diff --git a/Collapse file
‎src/js_native_api_v8.cc‎
Copy file name to clipboardExpand all lines: src/js_native_api_v8.cc
+3-1Lines changed: 3 additions & 1 deletion b/Collapse file
‎src/js_native_api_v8.cc‎
Copy file name to clipboardExpand all lines: src/js_native_api_v8.cc
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/Collapse file
‎src/js_native_api_v8.h‎
Copy file name to clipboardExpand all lines: src/js_native_api_v8.h
+1Lines changed: 1 addition & 0 deletions b/Collapse file
‎src/js_native_api_v8.h‎
Copy file name to clipboardExpand all lines: src/js_native_api_v8.h
+1Lines changed: 1 addition & 0 deletions
diff --git a/Collapse file
‎test/js-native-api/common.h‎
Copy file name to clipboardExpand all lines: test/js-native-api/common.h
+4-3Lines changed: 4 additions & 3 deletions b/Collapse file
‎test/js-native-api/common.h‎
Copy file name to clipboardExpand all lines: test/js-native-api/common.h
+4-3Lines changed: 4 additions & 3 deletions
@@ -761,12 +761,14 @@ napi_status napi_get_last_error_info(napi_env env,
       NAPI_ARRAYSIZE(error_messages) == last_status + 1,
       "Count of error messages must match count of error values");
   CHECK_LE(env->last_error.error_code, last_status);
-
   // Wait until someone requests the last error information to fetch the error
   // message string
   env->last_error.error_message =
       error_messages[env->last_error.error_code];
 
+  if (env->last_error.error_code == napi_ok) {
+     napi_clear_last_error(env);
+  }
   *result = &(env->last_error);
   return napi_ok;
 }
 
@@ -159,6 +159,7 @@ static inline napi_status napi_clear_last_error(napi_env env) {
   // TODO(boingoing): Should this be a callback?
   env->last_error.engine_error_code = 0;
   env->last_error.engine_reserved = nullptr;
+  env->last_error.error_message = nullptr;
   return napi_ok;
 }
 
 
@@ -8,12 +8,13 @@
     const napi_extended_error_info *error_info;                          \
     napi_get_last_error_info((env), &error_info);                        \
     bool is_pending;                                                     \
+    const char* err_message = error_info->error_message;                  \
     napi_is_exception_pending((env), &is_pending);                       \
     /* If an exception is already pending, don't rethrow it */           \
     if (!is_pending) {                                                   \
-      const char* error_message = error_info->error_message != NULL ?    \
-        error_info->error_message :                                      \
-        "empty error message";                                           \
+      const char* error_message = err_message != NULL ?                  \
+                       err_message :                                     \
+                      "empty error message";                             \
       napi_throw_error((env), NULL, error_message);                      \
     }                                                                    \
   } while (0)
Original file line number	Diff line number	Diff line change
`@@ -159,6 +159,7 @@ static inline napi_status napi_clear_last_error(napi_env env) {`
`159`	`159`	`// TODO(boingoing): Should this be a callback?`
`160`	`160`	`env->last_error.engine_error_code = 0;`
`161`	`161`	`env->last_error.engine_reserved = nullptr;`
	`162`	`+ env->last_error.error_message = nullptr;`
`162`	`163`	`return napi_ok;`
`163`	`164`	`}`
`164`	`165`