Moved ifname release that was deleted too early

src/NetworkInterface.cpp

@@ -561,12 +561,6 @@ void NetworkInterface::deleteDataStructures() {
   }
 #endif
 
-  if(ifname) {
-    //ntop->getTrace()->traceEvent(TRACE_NORMAL, "Interface %s shutdown", ifname);
-    free(ifname);
-    ifname = NULL;
-  }
-
 #ifdef HAVE_EBPF
   if(ebpfEvents) {
     for(u_int16_t i=0; i<EBPF_QUEUE_LEN; i++)
@@ -592,6 +586,8 @@ NetworkInterface::~NetworkInterface() {
   }
 #endif
 
+  if(db) db->shutdown();
+
   if(getNumPackets() > 0) {
     ntop->getTrace()->traceEvent(TRACE_NORMAL,
 				 "Flushing host contacts for interface %s",
@@ -644,6 +640,8 @@ NetworkInterface::~NetworkInterface() {
   if(tsExporter)            delete tsExporter;
   if(ts_ring)               delete ts_ring;
   if(mdns)                  delete mdns; /* Leave it at the end so the mdns resolved has time to initialize */
+
+  if(ifname)                free(ifname);
 }
 
 /* **************************************************** */

src/Ntop.cpp

@@ -1388,7 +1388,6 @@ bool Ntop::checkGuiUserPassword(struct mg_connection *conn,
   char *remote_ip, ipbuf[64], key[128], val[16];
   int cur_attempts = 0;
   bool rv;
-  struct mg_request_info *request_info = mg_get_request_info(conn);
   IpAddress client_addr;
 
   client_addr.set(mg_get_client_address(conn));