Skip to content
Snippets Groups Projects
Commit 4d8bb33e authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

Use XSAFECOPY in place of SAFECOPY to avoid NULL pointer checks

parent ea99ca80
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #3685 passed
...@@ -663,7 +663,7 @@ js_client_add(JSContext *cx, uintN argc, jsval *arglist) ...@@ -663,7 +663,7 @@ js_client_add(JSContext *cx, uintN argc, jsval *arglist)
if(argc>1) { if(argc>1) {
JSVALUE_TO_MSTRING(cx, argv[1], cstr, NULL); JSVALUE_TO_MSTRING(cx, argv[1], cstr, NULL);
HANDLE_PENDING(cx, cstr); HANDLE_PENDING(cx, cstr);
SAFECOPY(client.user, cstr); XSAFECOPY(client.user, cstr);
} }
if(argc>2) if(argc>2)
...@@ -717,7 +717,7 @@ js_client_update(JSContext *cx, uintN argc, jsval *arglist) ...@@ -717,7 +717,7 @@ js_client_update(JSContext *cx, uintN argc, jsval *arglist)
if(argc>1) { if(argc>1) {
JSVALUE_TO_MSTRING(cx, argv[1], cstr, NULL); JSVALUE_TO_MSTRING(cx, argv[1], cstr, NULL);
SAFECOPY(client.user, cstr); XSAFECOPY(client.user, cstr);
} }
if(argc>2) if(argc>2)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment