xen-changelog

[Top] [All Lists]

[Xen-changelog] [xen-unstable] Add missing 'break' statement.

from [Xen patchbot-unstable]

[Permanent Link][Original]

To:	xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject:	[Xen-changelog] [xen-unstable] Add missing 'break' statement.
From:	Xen patchbot-unstable <patchbot@xxxxxxx>
Date:	Tue, 30 Aug 2011 20:22:12 +0100
Delivery-date:	Tue, 30 Aug 2011 12:23:07 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
List-help:	<mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id:	BK change log <xen-changelog.lists.xensource.com>
List-post:	<mailto:xen-changelog@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to:	xen-devel@xxxxxxxxxxxxxxxxxxx
Sender:	xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx

# HG changeset patch
# User Kaushik Kumar Ram <kaushik@xxxxxxxx>
# Date 1314367121 -3600
# Node ID 469aa1fbd843d0797345ebf1e3d6c6df27ce744b
# Parent  1ec46033135e3a9f8b24a3e67d560a7ff2a826d8
Add missing 'break' statement.

Without the 'break', assigning a pci device to a PV guest results in an abort,
since the code always falls through to the default abort case in the switch
statement.

Signed-off-by: Kaushik Kumar Ram <kaushik@xxxxxxxx>
Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Committed-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---


diff -r 1ec46033135e -r 469aa1fbd843 tools/libxl/libxl_pci.c
--- a/tools/libxl/libxl_pci.c   Fri Aug 26 13:06:39 2011 +0100
+++ b/tools/libxl/libxl_pci.c   Fri Aug 26 14:58:41 2011 +0100
@@ -696,6 +696,7 @@
             }
         }
         fclose(f);
+        break;
     }
     default:
         abort();

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-changelog] [xen-unstable] Add missing 'break' statement., Xen patchbot-unstable <=

Previous by Date:	[Xen-changelog] [xen-unstable] Config.mk: Include optional .config file first rather than last, Xen patchbot-unstable
Next by Date:	[Xen-changelog] [xen-unstable] Merge, Xen patchbot-unstable
Previous by Thread:	[Xen-changelog] [xen-unstable] Config.mk: Include optional .config file first rather than last, Xen patchbot-unstable
Next by Thread:	[Xen-changelog] [xen-unstable] Merge, Xen patchbot-unstable
Indexes:	[Date] [Thread] [Top] [All Lists]