GitLab

Bug introduced in commit a2f5990b (Sept-11):
By calling putuserrec() before modifying user->misc and then calling
noyes(), we're giving an opportunity for the low-level node sync code to
read the modified "useron" back from the database, thus losing the change
we just made to user->misc. Instead, move the putuserrec() call to the end
of the case statement. Another option would have been to turn off the NETMAIL
flag before the first call to putuserrec().

Bug reported by Nugax (BYTEXCHG)