https://gitlab.synchro.net/sbbs/sbbs/-/commit/98d4f0839217108314a838d5
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Allow forward-to-netmail option to be toggled off by user
Bug introduced in commit a2f5990b4db (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)
---
ï¿ Synchronet ï¿ Vertrauen ï¿ Home of Synchronet ï¿ [vert/cvs/bbs].synchro.net