If the user number is 0, don't open the user file (user.dat) - the read

of the user record is going to fail anyway. *this* explains a lot of
instances of the user.dat file being open concurrently, at least one per
active thread with a JS context.

src/sbbs3/js_user.c

@@ -128,7 +128,7 @@ enum {
 
 static void js_getuserdat(scfg_t* scfg, private_t* p)
 {
-	if(!p->cached) {
+	if(p->user->number != 0 && !p->cached) {
 		if(p->file < 1)
 			p->file = openuserdat(scfg, /* for_modify: */FALSE);
 		if(fgetuserdat(scfg, p->user, p->file)==0)