Looks good.

Reviewed by: Kirti Wankhede <kwankh...@nvidia.com>

Thanks,
Kirti

On 3/6/2018 5:04 PM, Gerd Hoffmann wrote:
> Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
> ---
>  hw/vfio/common.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/hw/vfio/common.c b/hw/vfio/common.c
> index f895e3c335..6a8203a532 100644
> --- a/hw/vfio/common.c
> +++ b/hw/vfio/common.c
> @@ -858,6 +858,13 @@ void vfio_region_finalize(VFIORegion *region)
>      g_free(region->mmaps);
>  
>      trace_vfio_region_finalize(region->vbasedev->name, region->nr);
> +
> +    region->mem = NULL;
> +    region->mmaps = NULL;
> +    region->nr_mmaps = 0;
> +    region->size = 0;
> +    region->flags = 0;
> +    region->nr = 0;
>  }
>  
>  void vfio_region_mmaps_set_enabled(VFIORegion *region, bool enabled)
> 

Reply via email to