While moving the local user logic in it's own function I missed to
change the "secure" string macro. We copy only the first 3 bytes
of the username. Guess why I didn't notice it :)
if (u->ut_time > recent) {
recent = u->ut_time;
if (u->ut_time > recent) {
recent = u->ut_time;
- strfieldcpy(user, u->ut_user);
+ strnfieldcpy(user, u->ut_user, OWNER_SIZE);
dbg("local user is '%s'", user);
break;
}
dbg("local user is '%s'", user);
break;
}