I hate segfaults

src/App.cpp

@@ -41,7 +41,11 @@ void App::run() {
         for(auto it = modules.begin(); it != modules.end(); it++) {
             (*it)->run();
         }
-        closeModules();
+        
+        for(auto it = toClose.begin(); it != modules.end(); it++) {
+            unload(it);
+        }
+        toClose.clear();
     }
     
 }
@@ -77,23 +81,17 @@ void App::load(std::string lib) {
 }
         
 void App::unload(decltype(Module::self) it) {
-    toClose.push_back((*it)->getHandle());
+    void* h = (*it)->getHandle();
-    std::cout << "marked handle for closure\n";
+    std::cout << "retrieved handle\nAttempting to delete module: " << (*it)->getName() << std::endl;
-    std::cout << "Attempting to delete module: " << (*it)->getName() << std::endl;
+    delete *it;
-    Module* m = *it;
-    *it = nullptr;    
-    modules.erase(it);
-    std::cout << "erased iterator\n";
-    delete m;
     std::cout << "deleted module\n";
-
-}
-
-void App::closeModules() {
-    for(auto h : toClose) {
     dlclose(h);
+    std::cout << "closed handle\n";
 }
-    toClose.clear();
+
+void App::stopModule(decltype(Module::self) it) {
+    toClose.push_back(*it);
+    modules.erase(it);
 }
 
 void App::handleArgs(const int& argc, char* argv[]) {

src/App.h

@@ -13,7 +13,7 @@ class App {
         bool done = false;
 
         std::list<Module*> modules;
-        std::list<void*> toClose;
+        std::list<Module*> toClose;
 
     public:
         App(const int&, char*[]);
@@ -27,7 +27,7 @@ class App {
 
         void unload(decltype(Module::self));
 
-        void closeModules();
+        void stopModule(decltype(Module::self));
 
         void handleArgs(const int&, char*[]);