Fix return return value when methods auto-open a message base (e.g. save_msg())

js_open() sets the rval for the context (e.g. to JSVAL_TRUE), so we need to set
it back to JSVAL_FALSE for failure conditions. Otherwise, the methods always
return true even upon (save-msg) failure.

src/sbbs3/js_msgbase.c

@@ -2578,6 +2578,7 @@ js_save_msg(JSContext *cx, uintN argc, jsval *arglist)
 			return JS_FALSE;
 		if(JS_RVAL(cx, arglist) == JSVAL_FALSE)
 			return JS_TRUE;
+		JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
 	}
 
 	memset(&msg,0,sizeof(msg));
@@ -2727,6 +2728,7 @@ js_vote_msg(JSContext *cx, uintN argc, jsval *arglist)
 			return JS_FALSE;
 		if(JS_RVAL(cx, arglist) == JSVAL_FALSE)
 			return JS_TRUE;
+		JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
 	}
 
 	memset(&msg, 0, sizeof(msg));
@@ -2794,6 +2796,7 @@ js_add_poll(JSContext *cx, uintN argc, jsval *arglist)
 			return JS_FALSE;
 		if(JS_RVAL(cx, arglist) == JSVAL_FALSE)
 			return JS_TRUE;
+		JS_SET_RVAL(cx, arglist, JSVAL_FALSE);
 	}
 
 	memset(&msg, 0, sizeof(msg));