[PATCH 11/30] util: convert pointers to use g_autofree
Ján Tomko
jtomko at redhat.com
Tue Nov 24 12:02:07 UTC 2020
util: resctl: convert pointers to use g_autofree
On a Monday in 2020, Ryan Gahagan wrote:
>From: Barrett Schonefeld <bschoney at utexas.edu>
>
>- src/util/virresctrl.c
>
>Signed-off-by: Barrett Schonefeld <bschoney at utexas.edu>
>---
> src/util/virresctrl.c | 25 ++++++++-----------------
> 1 file changed, 8 insertions(+), 17 deletions(-)
>
>diff --git a/src/util/virresctrl.c b/src/util/virresctrl.c
>index d3087b98c1..1c2d175295 100644
>--- a/src/util/virresctrl.c
>+++ b/src/util/virresctrl.c
>@@ -709,7 +709,7 @@ virResctrlGetMonitorInfo(virResctrlInfoPtr resctrl)
> {
> int ret = -1;
> int rv = -1;
>- char *featurestr = NULL;
>+ g_autofree char *featurestr = NULL;
> char **features = NULL;
> size_t nfeatures = 0;
> virResctrlInfoMongrpPtr info_monitor = NULL;
>@@ -771,7 +771,6 @@ virResctrlGetMonitorInfo(virResctrlInfoPtr resctrl)
>
> ret = 0;
> cleanup:
>- VIR_FREE(featurestr);
> g_strfreev(features);
> VIR_FREE(info_monitor);
> return ret;
>@@ -1736,7 +1735,7 @@ virResctrlAllocGetGroup(virResctrlInfoPtr resctrl,
> const char *groupname,
> virResctrlAllocPtr *alloc)
> {
>- char *schemata = NULL;
>+ g_autofree char *schemata = NULL;
> int rv = virFileReadValueString(&schemata,
> SYSFS_RESCTRL_PATH "/%s/schemata",
> groupname);
>@@ -1753,11 +1752,9 @@ virResctrlAllocGetGroup(virResctrlInfoPtr resctrl,
> if (virResctrlAllocParse(resctrl, *alloc, schemata) < 0)
> goto error;
>
>- VIR_FREE(schemata);
> return 0;
>
> error:
>- VIR_FREE(schemata);
> virObjectUnref(*alloc);
> *alloc = NULL;
> return -1;
>@@ -2354,8 +2351,8 @@ virResctrlAllocCreate(virResctrlInfoPtr resctrl,
> virResctrlAllocPtr alloc,
> const char *machinename)
> {
>- char *schemata_path = NULL;
>- char *alloc_str = NULL;
>+ g_autofree char *schemata_path = NULL;
>+ g_autofree char *alloc_str = NULL;
> int ret = -1;
> int lockfd = -1;
>
>@@ -2403,8 +2400,6 @@ virResctrlAllocCreate(virResctrlInfoPtr resctrl,
> ret = 0;
> cleanup:
> virResctrlUnlock(lockfd);
>- VIR_FREE(alloc_str);
>- VIR_FREE(schemata_path);
> return ret;
> }
>
>@@ -2413,8 +2408,8 @@ static int
> virResctrlAddPID(const char *path,
> pid_t pid)
> {
>- char *tasks = NULL;
>- char *pidstr = NULL;
>+ g_autofree char *tasks = NULL;
>+ g_autofree char *pidstr = NULL;
> int ret = 0;
>
> if (!path) {
>@@ -2436,8 +2431,6 @@ virResctrlAddPID(const char *path,
>
> ret = 0;
> cleanup:
>- VIR_FREE(tasks);
>- VIR_FREE(pidstr);
> return ret;
> }
>
>@@ -2657,8 +2650,8 @@ virResctrlMonitorGetStats(virResctrlMonitorPtr monitor,
> size_t i = 0;
> unsigned long long val = 0;
> g_autoptr(DIR) dirp = NULL;
>- char *datapath = NULL;
>- char *filepath = NULL;
>+ g_autofree char *datapath = NULL;
>+ g_autofree char *filepath = NULL;
filepath should be moved into the loop.
Jano
> struct dirent *ent = NULL;
> virResctrlMonitorStatsPtr stat = NULL;
>
>@@ -2737,8 +2730,6 @@ virResctrlMonitorGetStats(virResctrlMonitorPtr monitor,
>
> ret = 0;
> cleanup:
>- VIR_FREE(datapath);
>- VIR_FREE(filepath);
> virResctrlMonitorStatsFree(stat);
> return ret;
> }
>--
>2.29.0
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20201124/958e4f66/attachment-0001.sig>
More information about the libvir-list
mailing list