Move shutdown logging to just before deleting the SQLitePool, which i…

…s where unmapping happens

BedrockServer.cpp

@@ -3,6 +3,7 @@
 
 #include <arpa/inet.h>
 #include <cstring>
+#include <fstream>
 #include <iomanip>
 #include <sys/resource.h>
 #include <sys/time.h>
@@ -621,6 +622,15 @@ void BedrockServer::sync()
     // until they return.
     atomic_store(&_syncNode, shared_ptr<SQLiteNode>(nullptr));
 
+    // If we're not detaching, save that we're shutting down.
+    if (!_detach) {
+        ofstream file("/var/log/bedrock_shutdown", std::ios::app);
+        if (file) {
+            file << "shutdown " << getpid() << " " << SComposeTime("%Y-%m-%dT%H:%M:%S", STimeNow()) << endl;
+            file.close();
+        }
+    }
+
     // Release the current DB pool, and zero out our pointer. If any socket threads hold a handle to `_syncNode`, they will keep this in existence
     // until they release it.
     _dbPool = nullptr;

main.cpp

@@ -4,7 +4,6 @@
 ///
 #include <dlfcn.h>
 #include <iostream>
-#include <fstream>
 #include <signal.h>
 #include <sys/resource.h>
 #include <sys/stat.h>
@@ -401,12 +400,6 @@ int main(int argc, char* argv[]) {
     // All done
     SINFO("Graceful process shutdown complete");
 
-    // Save that we're shutting down.
-    ofstream file("/var/log/bedrock_shutdown", std::ios::app);
-    if (file) {
-        file << "shutdown " << getpid() << " " << SComposeTime("%Y-%m-%dT%H:%M:%S", STimeNow()) << endl;
-        file.close();
-    }
 
     return 0;
 }