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

More parameter validation for JS 'system' methods 

And corresponding tests
parent f300a32e
No related branches found
No related tags found
No related merge requests found
Pipeline #7890 failed
Stage: build
Stage: test
Stage: cleanup
Show whitespace changes
Inline Side-by-side
exec/tests/system/except.js 0 → 100644
+ 92
− 0
View file @ 849cb917
// Test system function exceptions
// Value (e.g. 0, 1) is number of non-null/undefined args required
var func_list = {
'alias': 1,
'check_filename': 0,
'check_name': 0,
'check_pid': 1,
'check_syspass': 0,
'datestr': 1,
'del_user': 1,
'exec': 1,
'filter_ip': 1,
'find_login_id': 0,
'findstr': 2,
'get_node': 1,
'get_node_message': 1,
'get_telegram': 1,
'hacklog': 1,
'illegal_filename': 0,
'matchuser': 0,
'matchuserdata': 2,
'new_user': 1,
'notify': 2,
'popen': 1,
'put_node_message': 2,
'put_telegram': 2,
'safest_filename': 0,
'secondstr': 1,
'spamlog': 1,
'terminate_pid': 1,
'text': 1,
'timestr': 1,
'trashcan': 2,
'username': 0,
'zonestr': 1,
};
var noargs_required = [
'datestr',
'secondstr',
'timestr',
'zonestr',
];
var prefix = "system.";
for (var func in func_list) {
if (system[func] === undefined)
throw new Error("Function " + prefix + func + " isn't defined");
if (noargs_required.indexOf(func) < 0) {
var exp = prefix + func + "()";
var success = false;
try {
eval(exp);
} catch (e) {
if(e instanceof Error)
success = true;
}
if (!success)
throw new Error("Invocation of '" + exp + "' did not throw the expected Error exception");
}
if (func_list[func] < 1)
continue;
var arglist = ["(null)", "(undefined)"];
for(var i in arglist) {
exp = prefix + func + arglist[i];
success = false;
try {
eval(exp);
} catch (e) {
if(e instanceof Error)
success = true;
}
if (!success)
throw new Error("Invocation of '" + exp + "' did not throw the expected Error exception");
}
if (func_list[func] < 2)
continue;
var arglist = ["(0)", "(0, undefined)"];
for(var i in arglist) {
exp = prefix + func + arglist[i];
success = false;
try {
eval(exp);
} catch (e) {
if(e instanceof Error)
success = true;
}
if (!success)
throw new Error("Invocation of '" + exp + "' did not throw the expected Error exception");
}
}
exec/tests/system/rtypes.js 0 → 100644
+ 44
− 0
View file @ 849cb917
// Test system method return types
// @format.tab-size 4, @format.use-tabs true
var type = {
'username("")' : 'string',
'alias("")' : 'string',
'find_login_id("")' : 'number',
'matchuser("")' : 'number',
'matchuserdata(1,"")' : 'number',
'trashcan("","")' : 'boolean',
'findstr("", "")' : 'boolean',
'zonestr()' : 'string',
'datestr()' : 'string',
'secondstr(0)' : 'string',
'get_node(0)' : 'object',
'get_node_message(0)' : 'string',
'get_telegram(0)' : 'string',
'exec(" ")' : 'number',
'check_syspass(null)' : 'boolean',
'check_name(null)' : 'boolean',
'check_filename(null)' : 'boolean',
'allowed_filename(null)': 'boolean',
'safest_filename(null)' : 'boolean',
'illegal_filename(null)': 'boolean',
'text(0)' : 'object', // null
'text(1)' : 'string',
'text("")' : 'object', // null
'text("Email")' : 'string',
};
var prefix = "system.";
for (var i in type) {
var result;
var exp = prefix + i;
try {
// writeln(exp);
result = eval(exp);
} catch(e) {
log("Error evaluating: " + exp);
throw e;
}
if (typeof result !== type[i])
throw new Error("typeof " + exp + " = '" + (typeof result) + "' instead of '" + type[i] + "'");
}
exec/tests/system/skipif 0 → 100644
+ 1
− 0
View file @ 849cb917
jsdoor_revision !== undefined
src/sbbs3/js_system.c
+ 174
− 120
View file @ 849cb917
...@@ -854,7 +854,10 @@ js_alias(JSContext *cx, uintN argc, jsval *arglist) ...@@ -854,7 +854,10 @@ js_alias(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -893,8 +896,12 @@ js_username(JSContext *cx, uintN argc, jsval *arglist) ...@@ -893,8 +896,12 @@ js_username(JSContext *cx, uintN argc, jsval *arglist)
char* cstr; char* cstr;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_SET_RVAL(cx, arglist, JS_GetEmptyStringValue(cx));
return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -922,25 +929,25 @@ js_matchuser(JSContext *cx, uintN argc, jsval *arglist) ...@@ -922,25 +929,25 @@ js_matchuser(JSContext *cx, uintN argc, jsval *arglist)
BOOL sysop_alias = TRUE; BOOL sysop_alias = TRUE;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_ZERO);
return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
if ((js_str = JS_ValueToString(cx, argv[0])) == NULL) { if ((js_str = JS_ValueToString(cx, argv[0])) == NULL)
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(0)); return JS_FALSE;
return JS_TRUE;
}
if (argc > 1) if (argc > 1)
JS_ValueToBoolean(cx, argv[1], &sysop_alias); JS_ValueToBoolean(cx, argv[1], &sysop_alias);
JSSTRING_TO_ASTRING(cx, js_str, p, (LEN_ALIAS > LEN_NAME) ? LEN_ALIAS + 2:LEN_NAME + 2, NULL); JSSTRING_TO_ASTRING(cx, js_str, p, (LEN_ALIAS > LEN_NAME) ? LEN_ALIAS + 2:LEN_NAME + 2, NULL);
if (p == NULL) { if (p == NULL)
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(0)); return JS_FALSE;
return JS_TRUE;
}
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(matchuser(sys->cfg, p, sysop_alias))); JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(matchuser(sys->cfg, p, sysop_alias)));
...@@ -1011,22 +1018,23 @@ js_find_login_id(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1011,22 +1018,23 @@ js_find_login_id(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_ZERO);
return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
if ((js_str = JS_ValueToString(cx, argv[0])) == NULL) { if ((js_str = JS_ValueToString(cx, argv[0])) == NULL)
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(0)); return JS_FALSE;
return JS_TRUE;
}
JSSTRING_TO_ASTRING(cx, js_str, p, (LEN_ALIAS > LEN_NAME) ? LEN_ALIAS + 2:LEN_NAME + 2, NULL); JSSTRING_TO_ASTRING(cx, js_str, p, (LEN_ALIAS > LEN_NAME) ? LEN_ALIAS + 2:LEN_NAME + 2, NULL);
if (p == NULL) { if (p == NULL)
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(0)); return JS_FALSE;
return JS_TRUE;
}
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(find_login_id(sys->cfg, p))); JS_SET_RVAL(cx, arglist, INT_TO_JSVAL(find_login_id(sys->cfg, p)));
...@@ -1048,26 +1056,29 @@ js_trashcan(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1048,26 +1056,29 @@ js_trashcan(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 2))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0) || js_argvIsNullOrVoid(cx, argv, 1))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
if ((js_can = JS_ValueToString(cx, argv[0])) == NULL) { if ((js_can = JS_ValueToString(cx, argv[0])) == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
if ((js_str = JS_ValueToString(cx, argv[1])) == NULL) { if ((js_str = JS_ValueToString(cx, argv[1])) == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
JSSTRING_TO_MSTRING(cx, js_can, can, NULL); JSSTRING_TO_MSTRING(cx, js_can, can, NULL);
HANDLE_PENDING(cx, can); HANDLE_PENDING(cx, can);
if (can == NULL) { if (can == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
...@@ -1079,7 +1090,7 @@ js_trashcan(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1079,7 +1090,7 @@ js_trashcan(JSContext *cx, uintN argc, jsval *arglist)
} }
if (str == NULL) { if (str == NULL) {
free(can); free(can);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
...@@ -1104,7 +1115,11 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1104,7 +1115,11 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist)
BOOL ret; BOOL ret;
str_list_t list = NULL; str_list_t list = NULL;
if (JSVAL_IS_OBJECT(argv[0]) && !JSVAL_IS_NULL(argv[0])) { if (js_argcIsInsufficient(cx, argc, 2))
return JS_FALSE;
if(js_argvIsNullOrVoid(cx, argv, 0) || js_argvIsNullOrVoid(cx, argv, 1))
return JS_FALSE;
if (JSVAL_IS_OBJECT(argv[0])) {
JSObject* array = JSVAL_TO_OBJECT(argv[0]); JSObject* array = JSVAL_TO_OBJECT(argv[0]);
if (array == NULL || !JS_IsArrayObject(cx, array)) if (array == NULL || !JS_IsArrayObject(cx, array))
return JS_TRUE; return JS_TRUE;
...@@ -1127,18 +1142,18 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1127,18 +1142,18 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist)
} }
else { else {
if ((js_fname = JS_ValueToString(cx, argv[0])) == NULL) { if ((js_fname = JS_ValueToString(cx, argv[0])) == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
JSSTRING_TO_MSTRING(cx, js_fname, fname, NULL); JSSTRING_TO_MSTRING(cx, js_fname, fname, NULL);
HANDLE_PENDING(cx, fname); HANDLE_PENDING(cx, fname);
if (fname == NULL) { if (fname == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
} }
if ((js_str = JS_ValueToString(cx, argv[1])) == NULL) { if ((js_str = JS_ValueToString(cx, argv[1])) == NULL) {
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
free(fname); free(fname);
return JS_TRUE; return JS_TRUE;
} }
...@@ -1150,7 +1165,7 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1150,7 +1165,7 @@ js_findstr(JSContext *cx, uintN argc, jsval *arglist)
} }
if (str == NULL) { if (str == NULL) {
free(fname); free(fname);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(JS_FALSE)); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE; return JS_TRUE;
} }
...@@ -1178,7 +1193,8 @@ js_zonestr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1178,7 +1193,8 @@ js_zonestr(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
char* cstr; char* cstr;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1212,7 +1228,8 @@ js_timestr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1212,7 +1228,8 @@ js_timestr(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1245,7 +1262,8 @@ js_datestr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1245,7 +1262,8 @@ js_datestr(JSContext *cx, uintN argc, jsval *arglist)
char * p; char * p;
enum date_fmt fmt; enum date_fmt fmt;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1278,15 +1296,11 @@ js_secondstr(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1278,15 +1296,11 @@ js_secondstr(JSContext *cx, uintN argc, jsval *arglist)
uint32_t t = 0; uint32_t t = 0;
JSString* js_str; JSString* js_str;
JS_SET_RVAL(cx, arglist, JSVAL_NULL);
if (js_argcIsInsufficient(cx, argc, 1)) if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE; return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_ReportError(cx, "Invalid argument");
return JS_FALSE; return JS_FALSE;
}
JS_ValueToECMAUint32(cx, argv[0], &t); JS_ValueToECMAUint32(cx, argv[0], &t);
sectostr(t, str); sectostr(t, str);
if ((js_str = JS_NewStringCopyZ(cx, str)) == NULL) if ((js_str = JS_NewStringCopyZ(cx, str)) == NULL)
...@@ -1314,7 +1328,10 @@ js_spamlog(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1314,7 +1328,10 @@ js_spamlog(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1379,7 +1396,10 @@ js_hacklog(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1379,7 +1396,10 @@ js_hacklog(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1444,7 +1464,10 @@ js_filter_ip(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1444,7 +1464,10 @@ js_filter_ip(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1504,7 +1527,10 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1504,7 +1527,10 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist)
int32 node_num; int32 node_num;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_NULL); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1514,7 +1540,7 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1514,7 +1540,7 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist)
node_num = cfg->node_num; node_num = cfg->node_num;
if (argc) { if (argc) {
if (!JS_ValueToInt32(cx, argv[0], &node_num)) if (!JS_ValueToInt32(cx, argv[0], &node_num))
return JS_TRUE; return JS_FALSE;
} }
if (node_num < 1) if (node_num < 1)
node_num = 1; node_num = 1;
...@@ -1526,11 +1552,11 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1526,11 +1552,11 @@ js_get_node(JSContext *cx, uintN argc, jsval *arglist)
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
if (retval != 0) { if (retval != 0) {
JS_ReportError(cx, "getnodat(%d) returned %d", node_num, retval); JS_ReportError(cx, "getnodat(%d) returned %d", node_num, retval);
return JS_TRUE; return JS_FALSE;
} }
if ((nodeobj = JS_NewObject(cx, NULL, NULL, obj)) == NULL) { if ((nodeobj = JS_NewObject(cx, NULL, NULL, obj)) == NULL) {
JS_ReportError(cx, "JS_NewObject failure"); JS_ReportError(cx, "JS_NewObject failure");
return JS_TRUE; return JS_FALSE;
} }
JS_DefineProperty(cx, nodeobj, "status", INT_TO_JSVAL((int)node.status), NULL, NULL, JSPROP_ENUMERATE); JS_DefineProperty(cx, nodeobj, "status", INT_TO_JSVAL((int)node.status), NULL, NULL, JSPROP_ENUMERATE);
JS_DefineProperty(cx, nodeobj, "vstatus", STRING_TO_JSVAL(JS_NewStringCopyZ(cx, node_vstatus(sys->cfg, &node, str, sizeof str))), NULL, NULL, JSPROP_ENUMERATE); JS_DefineProperty(cx, nodeobj, "vstatus", STRING_TO_JSVAL(JS_NewStringCopyZ(cx, node_vstatus(sys->cfg, &node, str, sizeof str))), NULL, NULL, JSPROP_ENUMERATE);
...@@ -1556,7 +1582,10 @@ js_get_node_message(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1556,7 +1582,10 @@ js_get_node_message(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_NULL); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1572,9 +1601,10 @@ js_get_node_message(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1572,9 +1601,10 @@ js_get_node_message(JSContext *cx, uintN argc, jsval *arglist)
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
buf = getnmsg(sys->cfg, node_num); buf = getnmsg(sys->cfg, node_num);
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
if (buf == NULL) if (buf == NULL) {
JS_SET_RVAL(cx, arglist, JS_GetEmptyStringValue(cx));
return JS_TRUE; return JS_TRUE;
}
js_str = JS_NewStringCopyZ(cx, buf); js_str = JS_NewStringCopyZ(cx, buf);
free(buf); free(buf);
...@@ -1595,7 +1625,10 @@ js_put_node_message(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1595,7 +1625,10 @@ js_put_node_message(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 2))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0) || js_argvIsNullOrVoid(cx, argv, 1))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1611,7 +1644,7 @@ js_put_node_message(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1611,7 +1644,7 @@ js_put_node_message(JSContext *cx, uintN argc, jsval *arglist)
JSSTRING_TO_MSTRING(cx, js_msg, msg, NULL); JSSTRING_TO_MSTRING(cx, js_msg, msg, NULL);
HANDLE_PENDING(cx, msg); HANDLE_PENDING(cx, msg);
if (msg == NULL) if (msg == NULL)
return JS_TRUE; return JS_FALSE;
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
ret = putnmsg(sys->cfg, node, msg) == 0; ret = putnmsg(sys->cfg, node, msg) == 0;
...@@ -1632,7 +1665,10 @@ js_get_telegram(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1632,7 +1665,10 @@ js_get_telegram(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_NULL); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1645,9 +1681,10 @@ js_get_telegram(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1645,9 +1681,10 @@ js_get_telegram(JSContext *cx, uintN argc, jsval *arglist)
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
buf = getsmsg(sys->cfg, usernumber); buf = getsmsg(sys->cfg, usernumber);
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
if (buf == NULL) if (buf == NULL) {
JS_SET_RVAL(cx, arglist, JS_GetEmptyStringValue(cx));
return JS_TRUE; return JS_TRUE;
}
js_str = JS_NewStringCopyZ(cx, buf); js_str = JS_NewStringCopyZ(cx, buf);
free(buf); free(buf);
...@@ -1668,7 +1705,10 @@ js_put_telegram(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1668,7 +1705,10 @@ js_put_telegram(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 2))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0) || js_argvIsNullOrVoid(cx, argv, 1))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1684,7 +1724,7 @@ js_put_telegram(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1684,7 +1724,7 @@ js_put_telegram(JSContext *cx, uintN argc, jsval *arglist)
JSSTRING_TO_MSTRING(cx, js_msg, msg, NULL); JSSTRING_TO_MSTRING(cx, js_msg, msg, NULL);
HANDLE_PENDING(cx, msg); HANDLE_PENDING(cx, msg);
if (msg == NULL) if (msg == NULL)
return JS_TRUE; return JS_FALSE;
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
ret = putsmsg(sys->cfg, usernumber, msg) == 0; ret = putsmsg(sys->cfg, usernumber, msg) == 0;
...@@ -1708,7 +1748,10 @@ js_notify(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1708,7 +1748,10 @@ js_notify(JSContext *cx, uintN argc, jsval *arglist)
jsrefcount rc; jsrefcount rc;
BOOL ret; BOOL ret;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 2))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0) || js_argvIsNullOrVoid(cx, argv, 1))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
...@@ -1764,20 +1807,16 @@ js_new_user(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1764,20 +1807,16 @@ js_new_user(JSContext *cx, uintN argc, jsval *arglist)
client_t* client = NULL; client_t* client = NULL;
jsval val; jsval val;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
scfg_t* cfg = sys->cfg; scfg_t* cfg = sys->cfg;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_ReportError(cx, "Invalid argument");
return JS_FALSE;
}
JSVALUE_TO_ASTRING(cx, argv[0], alias, LEN_ALIAS + 2, NULL); JSVALUE_TO_ASTRING(cx, argv[0], alias, LEN_ALIAS + 2, NULL);
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
...@@ -1816,7 +1855,7 @@ js_new_user(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1816,7 +1855,7 @@ js_new_user(JSContext *cx, uintN argc, jsval *arglist)
i = newuserdat(cfg, &user); i = newuserdat(cfg, &user);
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
if (i == 0) { if (i == USER_SUCCESS) {
userobj = js_CreateUserObject(cx, obj, NULL, &user, /* client: */ NULL, /* global_user: */ FALSE, (struct mqtt*)NULL); userobj = js_CreateUserObject(cx, obj, NULL, &user, /* client: */ NULL, /* global_user: */ FALSE, (struct mqtt*)NULL);
JS_SET_RVAL(cx, arglist, OBJECT_TO_JSVAL(userobj)); JS_SET_RVAL(cx, arglist, OBJECT_TO_JSVAL(userobj));
} else } else
...@@ -1834,6 +1873,11 @@ js_del_user(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1834,6 +1873,11 @@ js_del_user(JSContext *cx, uintN argc, jsval *arglist)
int32 n; int32 n;
user_t user; user_t user;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -1861,6 +1905,11 @@ js_sys_exec(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1861,6 +1905,11 @@ js_sys_exec(JSContext *cx, uintN argc, jsval *arglist)
char * cmd = NULL; char * cmd = NULL;
int ret; int ret;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
JSVALUE_TO_MSTRING(cx, argv[0], cmd, NULL); JSVALUE_TO_MSTRING(cx, argv[0], cmd, NULL);
HANDLE_PENDING(cx, cmd); HANDLE_PENDING(cx, cmd);
if (cmd == NULL) { if (cmd == NULL) {
...@@ -1894,15 +1943,11 @@ js_popen(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1894,15 +1943,11 @@ js_popen(JSContext *cx, uintN argc, jsval *arglist)
JSString* js_str; JSString* js_str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID);
if (js_argcIsInsufficient(cx, argc, 1)) if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE; return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_ReportError(cx, "Invalid argument");
return JS_FALSE; return JS_FALSE;
}
if ((array = JS_NewArrayObject(cx, 0, NULL)) == NULL) if ((array = JS_NewArrayObject(cx, 0, NULL)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -1950,8 +1995,12 @@ js_chksyspass(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1950,8 +1995,12 @@ js_chksyspass(JSContext *cx, uintN argc, jsval *arglist)
jsval * argv = JS_ARGV(cx, arglist); jsval * argv = JS_ARGV(cx, arglist);
char * pass; char * pass;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -1970,8 +2019,12 @@ js_chkname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1970,8 +2019,12 @@ js_chkname(JSContext *cx, uintN argc, jsval *arglist)
char* str; char* str;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_VOID); if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
return JS_TRUE;
}
JSVALUE_TO_ASTRING(cx, argv[0], str, (LEN_ALIAS > LEN_NAME)?LEN_ALIAS + 2:LEN_NAME + 2, NULL); JSVALUE_TO_ASTRING(cx, argv[0], str, (LEN_ALIAS > LEN_NAME)?LEN_ALIAS + 2:LEN_NAME + 2, NULL);
js_system_private_t* sys; js_system_private_t* sys;
...@@ -1993,11 +2046,12 @@ js_chkfname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -1993,11 +2046,12 @@ js_chkfname(JSContext *cx, uintN argc, jsval *arglist)
char* fname = NULL; char* fname = NULL;
jsrefcount rc; jsrefcount rc;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (!JSVAL_IS_STRING(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (argc < 1 || !JSVAL_IS_STRING(argv[0]))
return JS_TRUE; return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -2007,9 +2061,10 @@ js_chkfname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2007,9 +2061,10 @@ js_chkfname(JSContext *cx, uintN argc, jsval *arglist)
return JS_FALSE; return JS_FALSE;
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(!illegal_filename(fname) bool result = !illegal_filename(fname)
&& allowed_filename(sys->cfg, fname) && allowed_filename(sys->cfg, fname)
&& !trashcan(sys->cfg, fname, "file"))); && !trashcan(sys->cfg, fname, "file");
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(result));
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
free(fname); free(fname);
...@@ -2023,17 +2078,19 @@ js_safest_fname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2023,17 +2078,19 @@ js_safest_fname(JSContext *cx, uintN argc, jsval *arglist)
char* fname = NULL; char* fname = NULL;
jsrefcount rc; jsrefcount rc;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (!JSVAL_IS_STRING(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (argc < 1 || !JSVAL_IS_STRING(argv[0]))
return JS_TRUE; return JS_TRUE;
}
JSVALUE_TO_MSTRING(cx, argv[0], fname, NULL); JSVALUE_TO_MSTRING(cx, argv[0], fname, NULL);
if (fname == NULL) if (fname == NULL)
return JS_FALSE; return JS_FALSE;
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(safest_filename(fname))); bool result = safest_filename(fname);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(result));
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
free(fname); free(fname);
...@@ -2047,17 +2104,19 @@ js_illegal_fname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2047,17 +2104,19 @@ js_illegal_fname(JSContext *cx, uintN argc, jsval *arglist)
char* fname = NULL; char* fname = NULL;
jsrefcount rc; jsrefcount rc;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (!JSVAL_IS_STRING(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (argc < 1 || !JSVAL_IS_STRING(argv[0]))
return JS_TRUE; return JS_TRUE;
}
JSVALUE_TO_MSTRING(cx, argv[0], fname, NULL); JSVALUE_TO_MSTRING(cx, argv[0], fname, NULL);
if (fname == NULL) if (fname == NULL)
return JS_FALSE; return JS_FALSE;
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(illegal_filename(fname))); bool result = illegal_filename(fname);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(result));
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
free(fname); free(fname);
...@@ -2072,11 +2131,12 @@ js_allowed_fname(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2072,11 +2131,12 @@ js_allowed_fname(JSContext *cx, uintN argc, jsval *arglist)
char* fname = NULL; char* fname = NULL;
jsrefcount rc; jsrefcount rc;
if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE;
if (!JSVAL_IS_STRING(argv[0])) {
JS_SET_RVAL(cx, arglist, JSVAL_FALSE); JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (argc < 1 || !JSVAL_IS_STRING(argv[0]))
return JS_TRUE; return JS_TRUE;
}
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -2102,19 +2162,15 @@ js_chkpid(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2102,19 +2162,15 @@ js_chkpid(JSContext *cx, uintN argc, jsval *arglist)
int32 pid = 0; int32 pid = 0;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (js_argcIsInsufficient(cx, argc, 1)) if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE; return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_ReportError(cx, "Invalid argument");
return JS_FALSE; return JS_FALSE;
}
JS_ValueToInt32(cx, argv[0], &pid); JS_ValueToInt32(cx, argv[0], &pid);
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(check_pid(pid))); bool result = check_pid(pid);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(result));
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
return JS_TRUE; return JS_TRUE;
...@@ -2127,19 +2183,19 @@ js_killpid(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2127,19 +2183,19 @@ js_killpid(JSContext *cx, uintN argc, jsval *arglist)
int32 pid = 0; int32 pid = 0;
jsrefcount rc; jsrefcount rc;
JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
if (js_argcIsInsufficient(cx, argc, 1)) if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE; return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
return JS_FALSE;
if (JSVAL_NULL_OR_VOID(argv[0])) { if (js_argcIsInsufficient(cx, argc, 1))
JS_ReportError(cx, "Invalid argument");
return JS_FALSE; return JS_FALSE;
}
JS_ValueToInt32(cx, argv[0], &pid); JS_ValueToInt32(cx, argv[0], &pid);
rc = JS_SUSPENDREQUEST(cx); rc = JS_SUSPENDREQUEST(cx);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(terminate_pid(pid))); bool result = terminate_pid(pid);
JS_SET_RVAL(cx, arglist, BOOLEAN_TO_JSVAL(result));
JS_RESUMEREQUEST(cx, rc); JS_RESUMEREQUEST(cx, rc);
return JS_TRUE; return JS_TRUE;
...@@ -2151,15 +2207,13 @@ js_text(JSContext *cx, uintN argc, jsval *arglist) ...@@ -2151,15 +2207,13 @@ js_text(JSContext *cx, uintN argc, jsval *arglist)
JSObject* obj = JS_THIS_OBJECT(cx, arglist); JSObject* obj = JS_THIS_OBJECT(cx, arglist);
jsval* argv = JS_ARGV(cx, arglist); jsval* argv = JS_ARGV(cx, arglist);
uint32 i = 0; uint32 i = 0;
JS_SET_RVAL(cx, arglist, JSVAL_NULL);
if (js_argcIsInsufficient(cx, argc, 1)) if (js_argcIsInsufficient(cx, argc, 1))
return JS_FALSE; return JS_FALSE;
if (js_argvIsNullOrVoid(cx, argv, 0))
if (JSVAL_NULL_OR_VOID(argv[0])) {
JS_ReportError(cx, "Invalid argument");
return JS_FALSE; return JS_FALSE;
} JS_SET_RVAL(cx, arglist, JSVAL_NULL);
js_system_private_t* sys; js_system_private_t* sys;
if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL) if ((sys = (js_system_private_t*)js_GetClassPrivate(cx, obj, &js_system_class)) == NULL)
return JS_FALSE; return JS_FALSE;
...@@ -2264,7 +2318,7 @@ static jsSyncMethodSpec js_system_functions[] = { ...@@ -2264,7 +2318,7 @@ static jsSyncMethodSpec js_system_functions[] = {
{"newuser", js_new_user, 1, JSTYPE_ALIAS }, {"newuser", js_new_user, 1, JSTYPE_ALIAS },
{"new_user", js_new_user, 1, JSTYPE_OBJECT, JSDOCSTR("name/alias [,client object]") {"new_user", js_new_user, 1, JSTYPE_OBJECT, JSDOCSTR("name/alias [,client object]")
, JSDOCSTR("Create a new user record, returns a new <a href=#User_object>User object</a> representing the new user account, on success.<br>" , JSDOCSTR("Create a new user record, returns a new <a href=#User_object>User object</a> representing the new user account, on success.<br>"
"returns an numeric error code on failure") "returns a numeric error code on failure")
, 310}, , 310},
{"del_user", js_del_user, 1, JSTYPE_BOOLEAN, JSDOCSTR("user_number") {"del_user", js_del_user, 1, JSTYPE_BOOLEAN, JSDOCSTR("user_number")
, JSDOCSTR("Delete the specified user account") , JSDOCSTR("Delete the specified user account")
...@@ -2310,8 +2364,8 @@ static jsSyncMethodSpec js_system_functions[] = { ...@@ -2310,8 +2364,8 @@ static jsSyncMethodSpec js_system_functions[] = {
"returns <tt>true</tt> on success") "returns <tt>true</tt> on success")
, 315}, , 315},
{"text", js_text, 1, JSTYPE_STRING, JSDOCSTR("<i>number</i> index or <i>string</i> id") {"text", js_text, 1, JSTYPE_STRING, JSDOCSTR("<i>number</i> index or <i>string</i> id")
, JSDOCSTR("Return specified text string (see <tt>bbs.text()</tt> for details)<br>" , JSDOCSTR("Return specified text string (see <tt>bbs.text()</tt> for details) or <tt>null</tt> if invalid <i>index</i> or <i>id</i> specified.<br>"
"The <i>string</i> id support was added in v3.20.") "The <i>string id</i> support was added in v3.20.")
, 31802}, , 31802},
{0} {0}
}; };
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment