On a Thursday in 2023, Peter Krempa wrote:
>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@redhat.com>
>---
> 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. */
s/freed/freed elswehere/ maybe?
> if (!def->parentnet)
> virDomainDeviceInfoFree(def->info);
>
>--
>2.39.2
>