[libvirt] [PATCH 1/2] Revert "network: Check for QOS before blindly using it"

Jiri Denemark jdenemar at redhat.com
Mon Nov 25 08:19:32 UTC 2019


On Fri, Nov 22, 2019 at 17:08:59 +0100, Erik Skultety wrote:
> This reverts commit f4db846c32c0a1e99a0f62b340273e48f8a98ed3.
> 
> This patch results in the following error when trying to start
> essentially any VM with default network:
> 
> unsupported configuration: QOS must be defined for network 'default'
> 
> Coverity didn't see that the bandwidth == NULL it complained about in
> virNetDevBandwidthPlug was already checked properly in
> networkCheckBandwidth, thus causing networkPlugBandwidth to return 0
> and finish before a call to virNetDevBandwidthPlug would have been even
> made.
> 
> Signed-off-by: Erik Skultety <eskultet at redhat.com>
> ---
>  src/network/bridge_driver.c | 14 --------------
>  1 file changed, 14 deletions(-)

Reviewed-by: Jiri Denemark <jdenemar at redhat.com>




More information about the libvir-list mailing list