On Thu, Feb 23, 2017 at 01:15:50PM +0100, Martin Kletzander wrote:
> On Thu, Feb 23, 2017 at 10:46:14AM +, Daniel P. Berrange wrote:
> > On Thu, Feb 23, 2017 at 11:38:40AM +0100, Martin Kletzander wrote:
> > > On Thu, Feb 23, 2017 at 09:48:48AM +, Daniel P. Berrange wrote:
> > > > On Wed, F
On Thu, Feb 23, 2017 at 10:46:14AM +, Daniel P. Berrange wrote:
On Thu, Feb 23, 2017 at 11:38:40AM +0100, Martin Kletzander wrote:
On Thu, Feb 23, 2017 at 09:48:48AM +, Daniel P. Berrange wrote:
> On Wed, Feb 22, 2017 at 09:19:15PM +0100, Martin Kletzander wrote:
> > On Wed, Feb 22, 2017
On Thu, Feb 23, 2017 at 11:38:40AM +0100, Martin Kletzander wrote:
> On Thu, Feb 23, 2017 at 09:48:48AM +, Daniel P. Berrange wrote:
> > On Wed, Feb 22, 2017 at 09:19:15PM +0100, Martin Kletzander wrote:
> > > On Wed, Feb 22, 2017 at 02:44:01PM -0500, Laine Stump wrote:
> > > > On 02/22/2017 12
On Thu, Feb 23, 2017 at 09:48:48AM +, Daniel P. Berrange wrote:
On Wed, Feb 22, 2017 at 09:19:15PM +0100, Martin Kletzander wrote:
On Wed, Feb 22, 2017 at 02:44:01PM -0500, Laine Stump wrote:
> On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
> > One of the conditions in qemuDomainDeviceCal
On Wed, Feb 22, 2017 at 09:19:15PM +0100, Martin Kletzander wrote:
> On Wed, Feb 22, 2017 at 02:44:01PM -0500, Laine Stump wrote:
> > On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
> > > One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
> > > was missing a break that could result
On Wed, Feb 22, 2017 at 02:52:08PM -0600, Eric Blake wrote:
> On 02/22/2017 11:52 AM, Daniel P. Berrange wrote:
> > One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
> > was missing a break that could result it in falling through to
> > an incorrect codepath.
> >
> > Signed-off-by:
On Wed, Feb 22, 2017 at 03:00:41PM -0600, Eric Blake wrote:
> On 02/22/2017 01:44 PM, Laine Stump wrote:
> > On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
> >> One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
> >> was missing a break that could result it in falling through to
>
On 02/22/2017 01:44 PM, Laine Stump wrote:
> On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
>> One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
>> was missing a break that could result it in falling through to
>> an incorrect codepath.
>
> Actually that's not true. Every codepa
On 02/22/2017 11:52 AM, Daniel P. Berrange wrote:
> One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
> was missing a break that could result it in falling through to
> an incorrect codepath.
>
> Signed-off-by: Daniel P. Berrange
> ---
> src/qemu/qemu_domain_address.c | 1 +
> 1
On Wed, Feb 22, 2017 at 02:44:01PM -0500, Laine Stump wrote:
On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
was missing a break that could result it in falling through to
an incorrect codepath.
Actually that's not true. Every
On 02/22/2017 12:52 PM, Daniel P. Berrange wrote:
One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
was missing a break that could result it in falling through to
an incorrect codepath.
Actually that's not true. Every codepath of the preceding case ends with
a "return blah". T
On Wed, 2017-02-22 at 17:52 +, Daniel P. Berrange wrote:
> @@ -553,6 +553,7 @@
> qemuDomainDeviceCalculatePCIConnectFlags(virDomainDeviceDefPtr dev,
> return pciFlags;
> }
> }
> +break;
>
> case VIR_DOMAIN_DEVICE_FS:
> /* the only type of file
One of the conditions in qemuDomainDeviceCalculatePCIConnectFlags
was missing a break that could result it in falling through to
an incorrect codepath.
Signed-off-by: Daniel P. Berrange
---
src/qemu/qemu_domain_address.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/qemu/qemu_domain_ad
13 matches
Mail list logo
