[libvirt] [PATCH] conf: Resolve corner case on fc_host deletion

Michal Privoznik mprivozn at redhat.com
Wed May 24 13:11:56 UTC 2017


On 04/29/2017 05:36 PM, John Ferlan wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1420740
> 
> Testing found an inventive way to cause an error at shutdown by providing the
> parent name for the fc host creation using the "same name" as the HBA. Since
> the code thus assumed the parent host name provided was the parent HBA and
> just extracted out the host number and sent that along to the vport_destroy
> this avoided checks made for equality.
> 
> So just add the equality check to that path to resolve.
> 
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
>  src/conf/node_device_conf.c | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)

ACK.

Sorry for the delay.

Michal




More information about the libvir-list mailing list