Bug fix - do not access the bars on cleanup if the device wasn't present.

Longer term it should be nice not to allocate the bars at all

drivers/driver.cc

@@ -34,6 +34,11 @@ bool Driver::isPresent() {
     return _present;
 }
 
+Driver::~Driver() {
+    //todo - Better change the Bar allocation to live in the stack
+    for (int i=0;i<6;i++) if (_bars[i]) delete _bars[i];
+}
+
 void
 Driver::setPresent(u8 bus, u8 slot, u8 func) {
     _present = true;

drivers/driver.hh

@@ -40,7 +40,7 @@ class Driver {
 public:
     Driver(u16 vid, u16 id) :_id(id), _vid(vid), _present(false), _bus(0), _slot(0), _func(0)\
            {for (int i=0;i<6;i++) _bars[i] = nullptr;};
-    virtual ~Driver() {for (int i=0;i<6;i++) if (_bars[i]) delete _bars[i];}
+    virtual ~Driver();
 
     bool isPresent();
     void setPresent(u8 bus, u8 slot, u8 func);

drivers/virtio-net.cc

@@ -15,7 +15,6 @@ namespace virtio {
 
     virtio_net::~virtio_net()
     {
-        set_dev_status(0);
     }
 
     bool virtio_net::Init(Device *d)

drivers/virtio.cc

@@ -22,7 +22,6 @@ namespace virtio {
     virtio_driver::~virtio_driver()
     {
         reset_host_side();
-
         for (int i=0; i < max_virtqueues_nr; i++) {
             if (NULL != _queues[i]) {
                 delete (_queues[i]);
@@ -31,6 +30,8 @@ namespace virtio {
     }
 
     void virtio_driver::reset_host_side() {
+        if (!isPresent()) return;
+
         set_dev_status(0);
         pci_conf_write(VIRTIO_PCI_QUEUE_PFN,(u32)0);
     }

loader.cc

@@ -282,7 +282,7 @@ void* do_main_thread(void* pprog)
 
     DeviceFactory::Instance()->InitializeDrivers();
 
-    //DriverFactory::Instance()->Destroy();
+    DriverFactory::Instance()->Destroy();
 
     auto t1 = clock::get()->time();
     auto t2 = clock::get()->time();