mini-os: Fix crash on frontend shutdown failures
Do not free frontend resources if some error happened, since the backend
may not have finished properly restarting in such case.
Signed-off-by: Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>
diff -r adce8bc43fcc extras/mini-os/blkfront.c
--- a/extras/mini-os/blkfront.c Tue Apr 06 07:16:47 2010 +0100
+++ b/extras/mini-os/blkfront.c Tue Apr 06 22:54:33 2010 +0200
@@ -313,7 +313,8 @@
snprintf(path, sizeof(path), "%s/event-channel", nodename);
xenbus_rm(XBT_NIL, path);
- free_blkfront(dev);
+ if (!err)
+ free_blkfront(dev);
}
static void blkfront_wait_slot(struct blkfront_dev *dev)
diff -r adce8bc43fcc extras/mini-os/fbfront.c
--- a/extras/mini-os/fbfront.c Tue Apr 06 07:16:47 2010 +0100
+++ b/extras/mini-os/fbfront.c Tue Apr 06 22:54:33 2010 +0200
@@ -286,7 +288,8 @@
snprintf(path, sizeof(path), "%s/request-abs-pointer", nodename);
xenbus_rm(XBT_NIL, path);
- free_kbdfront(dev);
+ if (!err)
+ free_kbdfront(dev);
}
#ifdef HAVE_LIBC
@@ -676,7 +682,8 @@
snprintf(path, sizeof(path), "%s/feature-update", nodename);
xenbus_rm(XBT_NIL, path);
- free_fbfront(dev);
+ if (!err)
+ free_fbfront(dev);
}
#ifdef HAVE_LIBC
diff -r adce8bc43fcc extras/mini-os/netfront.c
--- a/extras/mini-os/netfront.c Tue Apr 06 07:16:47 2010 +0100
+++ b/extras/mini-os/netfront.c Tue Apr 06 22:54:33 2010 +0200
@@ -574,7 +574,8 @@
snprintf(path, sizeof(path), "%s/request-rx-copy", nodename);
xenbus_rm(XBT_NIL, path);
- free_netfront(dev);
+ if (!err)
+ free_netfront(dev);
}
diff -r adce8bc43fcc extras/mini-os/pcifront.c
--- a/extras/mini-os/pcifront.c Tue Apr 06 07:16:47 2010 +0100
+++ b/extras/mini-os/pcifront.c Tue Apr 06 22:54:33 2010 +0200
@@ -378,7 +378,8 @@
snprintf(path, sizeof(path), "%s/event-channel", nodename);
xenbus_rm(XBT_NIL, path);
- free_pcifront(dev);
+ if (!err)
+ free_pcifront(dev);
}
int pcifront_physical_to_virtual (struct pcifront_dev *dev,
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|