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

Add comment about race condition 

Fixed typo in another comment
parent 14940b8a
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #3005 passed
Stage: build
Hide whitespace changes
Inline Side-by-side
Showing
with 2 additions and 2 deletions
src/sbbs3/logout.cpp
+ 2
− 2
View file @ b75114da
...@@ -49,7 +49,7 @@ void sbbs_t::logout() ...@@ -49,7 +49,7 @@ void sbbs_t::logout()
return; return;
} }
lprintf(LOG_INFO, "logout initiated"); lprintf(LOG_INFO, "logout initiated");
SAFECOPY(lastuseron,useron.alias); SAFECOPY(lastuseron,useron.alias); // TODO: race condition here
if(!online && getnodedat(cfg.node_num, &node, /* lock: */true) == 0) { if(!online && getnodedat(cfg.node_num, &node, /* lock: */true) == 0) {
node.status = NODE_LOGOUT; node.status = NODE_LOGOUT;
putnodedat(cfg.node_num, &node); putnodedat(cfg.node_num, &node);
...@@ -89,7 +89,7 @@ void sbbs_t::logout() ...@@ -89,7 +89,7 @@ void sbbs_t::logout()
remove(path); remove(path);
delfiles(cfg.temp_dir,ALLFILES); delfiles(cfg.temp_dir,ALLFILES);
if(sys_status&SS_USERON) { // Insures the useron actually when through logon()/getmsgptrs() first if(sys_status&SS_USERON) { // Insures the useron actually went through logon()/getmsgptrs() first
putmsgptrs(); putmsgptrs();
} }
if(!REALSYSOP) if(!REALSYSOP)
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment