[PATCH V2 0/9] use g_autofree

Jiang Jiacheng posted 9 patches 1 year, 3 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20230106091839.1263372-1-jiangjiacheng@huawei.com
src/conf/domain_audit.c                | 121 ++++++++-----------------
src/conf/node_device_util.c            |  49 ++++------
src/conf/nwfilter_conf.c               |  55 ++++-------
src/conf/virnetworkobj.c               |  85 ++++++-----------
src/conf/virnetworkportdef.c           |  12 +--
src/conf/virnwfilterbindingobj.c       |  22 ++---
src/conf/virnwfilterobj.c              |  19 +---
src/conf/virstorageobj.c               |  23 ++---
src/interface/interface_backend_udev.c |  22 ++---
src/locking/lock_driver_lockd.c        |  36 +++-----
src/locking/lock_driver_sanlock.c      |  46 ++++------
src/remote/remote_driver.c             |  22 ++---
src/rpc/virnetclientprogram.c          |   6 +-
src/rpc/virnettlscontext.c             |  16 +---
src/security/security_apparmor.c       |  94 +++++++------------
src/util/virconf.c                     |  27 ++----
src/vmware/vmware_conf.c               |  28 ++----
src/vz/vz_utils.c                      |  18 ++--
18 files changed, 239 insertions(+), 462 deletions(-)
[PATCH V2 0/9] use g_autofree
Posted by Jiang Jiacheng 1 year, 3 months ago
Use g_autofree more often and remove unnecessary cleanup/error
labels.

diff to v1:
  * use g_autofree for uuid in 'virNWFilterDefParseXML'
  * remove unused rc in security_apparmor.c

Jiang Jiacheng (9):
  conf: use g_autofree and remove unnecessary label
  interface: use g_autofree and remove unnecessary label
  locking: use g_autofree and remove unnecessary label
  remote: use g_autofree and remove unnecessary label
  rpc: use g_autofree and remove unnecessary label
  security: use g_autofree and remove unnecessary label
  util: use g_autofree and remove unnecessary label
  vmware: use g_autofree and remove unnecessary label
  vz: use g_autofree and remove unnecessary label

 src/conf/domain_audit.c                | 121 ++++++++-----------------
 src/conf/node_device_util.c            |  49 ++++------
 src/conf/nwfilter_conf.c               |  55 ++++-------
 src/conf/virnetworkobj.c               |  85 ++++++-----------
 src/conf/virnetworkportdef.c           |  12 +--
 src/conf/virnwfilterbindingobj.c       |  22 ++---
 src/conf/virnwfilterobj.c              |  19 +---
 src/conf/virstorageobj.c               |  23 ++---
 src/interface/interface_backend_udev.c |  22 ++---
 src/locking/lock_driver_lockd.c        |  36 +++-----
 src/locking/lock_driver_sanlock.c      |  46 ++++------
 src/remote/remote_driver.c             |  22 ++---
 src/rpc/virnetclientprogram.c          |   6 +-
 src/rpc/virnettlscontext.c             |  16 +---
 src/security/security_apparmor.c       |  94 +++++++------------
 src/util/virconf.c                     |  27 ++----
 src/vmware/vmware_conf.c               |  28 ++----
 src/vz/vz_utils.c                      |  18 ++--
 18 files changed, 239 insertions(+), 462 deletions(-)

-- 
2.33.0
Re: [PATCH V2 0/9] use g_autofree
Posted by Ján Tomko 1 year, 3 months ago
On a Friday in 2023, Jiang Jiacheng wrote:
>Use g_autofree more often and remove unnecessary cleanup/error
>labels.
>
>diff to v1:
>  * use g_autofree for uuid in 'virNWFilterDefParseXML'
>  * remove unused rc in security_apparmor.c
>
>Jiang Jiacheng (9):
>  conf: use g_autofree and remove unnecessary label
>  interface: use g_autofree and remove unnecessary label
>  locking: use g_autofree and remove unnecessary label
>  remote: use g_autofree and remove unnecessary label
>  rpc: use g_autofree and remove unnecessary label
>  security: use g_autofree and remove unnecessary label
>  util: use g_autofree and remove unnecessary label
>  vmware: use g_autofree and remove unnecessary label
>  vz: use g_autofree and remove unnecessary label
>
> src/conf/domain_audit.c                | 121 ++++++++-----------------
> src/conf/node_device_util.c            |  49 ++++------
> src/conf/nwfilter_conf.c               |  55 ++++-------
> src/conf/virnetworkobj.c               |  85 ++++++-----------
> src/conf/virnetworkportdef.c           |  12 +--
> src/conf/virnwfilterbindingobj.c       |  22 ++---
> src/conf/virnwfilterobj.c              |  19 +---
> src/conf/virstorageobj.c               |  23 ++---
> src/interface/interface_backend_udev.c |  22 ++---
> src/locking/lock_driver_lockd.c        |  36 +++-----
> src/locking/lock_driver_sanlock.c      |  46 ++++------
> src/remote/remote_driver.c             |  22 ++---
> src/rpc/virnetclientprogram.c          |   6 +-
> src/rpc/virnettlscontext.c             |  16 +---
> src/security/security_apparmor.c       |  94 +++++++------------
> src/util/virconf.c                     |  27 ++----
> src/vmware/vmware_conf.c               |  28 ++----
> src/vz/vz_utils.c                      |  18 ++--
> 18 files changed, 239 insertions(+), 462 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

and pushed now.

Jano