Re: [PATCH 21/23] Switch away from virHashFree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 30, 2021 at 03:32:05PM +0100, Peter Krempa wrote:
> Use 'g_clear_pointer(&ptr, g_hash_table_unref)' instead.
> 
> In few instances it allows us to also remove explicit clearing of
> pointers.
> 
> Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
> ---
>  src/conf/domain_addr.c                 |  2 +-
>  src/conf/domain_conf.c                 |  2 +-
>  src/conf/nwfilter_conf.c               |  2 +-
>  src/conf/nwfilter_ipaddrmap.c          |  3 +--
>  src/conf/virchrdev.c                   |  2 +-
>  src/conf/virdomainmomentobjlist.c      |  2 +-
>  src/conf/virdomainobjlist.c            |  4 ++--
>  src/conf/virinterfaceobj.c             |  2 +-
>  src/conf/virnetworkobj.c               |  4 ++--
>  src/conf/virnodedeviceobj.c            |  2 +-
>  src/conf/virnwfilterbindingdef.c       |  2 +-
>  src/conf/virnwfilterbindingobjlist.c   |  2 +-
>  src/conf/virsecretobj.c                |  2 +-
>  src/conf/virstorageobj.c               | 10 +++++-----
>  src/hyperv/hyperv_wmi.c                |  2 +-
>  src/hypervisor/virclosecallbacks.c     |  2 +-
>  src/libxl/libxl_logger.c               |  2 +-
>  src/locking/lock_daemon.c              |  2 +-
>  src/nwfilter/nwfilter_dhcpsnoop.c      |  6 +++---
>  src/nwfilter/nwfilter_gentech_driver.c |  2 +-
>  src/nwfilter/nwfilter_learnipaddr.c    |  7 ++-----
>  src/qemu/qemu_domain.c                 |  2 +-
>  src/qemu/qemu_driver.c                 |  2 +-
>  src/rpc/virnetdaemon.c                 |  2 +-
>  src/security/security_selinux.c        |  6 +++---
>  src/util/virfilecache.c                |  2 +-
>  src/util/virhash.c                     |  2 +-
>  src/util/virlockspace.c                |  2 +-
>  src/util/virmacmap.c                   |  2 +-
>  src/util/virsystemd.c                  |  2 +-
>  tests/nwfilterxml2firewalltest.c       |  2 +-
>  tests/qemusecuritymock.c               |  7 +++----
>  32 files changed, 45 insertions(+), 50 deletions(-)
> 
> diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c
> index a06721c35d..49745ba881 100644
> --- a/src/conf/domain_addr.c
> +++ b/src/conf/domain_addr.c
> @@ -1377,7 +1377,7 @@ void virDomainCCWAddressSetFree(virDomainCCWAddressSet *addrs)
>      if (!addrs)
>          return;
> 
> -    virHashFree(addrs->defined);
> +    g_clear_pointer(&addrs->defined, g_hash_table_unref);

This is in a Free() method, so we should just be calling
g_hash_table_unref directly.  Likewise for all the other
Free and Dispose methods. There's only a couple of places
needing a g_clear_pointer because they're touching global
variables.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux