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

Fix loop limit (system.stats.total_users != system.lastuser)

When looping to highest user number, use system.lastuser, not
system.stats.total_users.

This explains why not all userbases would be affected by the just-fixed
issue in js_user.c. If the total_users happens to equal lastuser, then
you would have hit the issue in js_user.c. If you had at least one
deleted or inactive user account, then system.stats.total_users would
be less than system.lastuser and your system would *not* have triggered
that bug.
parent 65e93897
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #3048 passed
......@@ -16,7 +16,7 @@ load("sbbsdefs.js");
function birthdays(month, day)
{
var u = new User;
var lastuser = system.stats.total_users;
var lastuser = system.lastuser;
var list = [];
month = parseInt(month, 10) + 1;
for(u.number = 1; u.number <= lastuser; u.number++) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment