Re: [PATCH 3/3] virDomainHostdevDefClear: Fix and shorten comment

There's more stuff than device info to clear nowadays. Drop the misleading comment. Shorten the comment saying that device info is freed elsewhere when 'parentnet' is present. Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- src/conf/domain_conf.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 7ad4ff26ab..f8067be49a 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -2608,13 +2608,7 @@ virDomainHostdevDefClear(virDomainHostdevDef *def) if (!def) return; - /* Free all resources in the hostdevdef. Currently the only - * such resource is the virDomainDeviceInfo. - */ - - /* If there is a parentnet device object, it will handle freeing - * def->info. - */ + /* Device info is freed with 'parentnet' if present. */

if (!def->parentnet) virDomainDeviceInfoFree(def->info); -- 2.39.2