tools/cgget: fix resource leak in get_cv_value()

Fix a resource leak, reported by the Coverity tool:

CID 258291 (#1 of 1): Resource leak (RESOURCE_LEAK)8. leaked_storage:
Variable handle going out of scope leaks the storage it points to.

In get_cv_value(), currently, we goto end label, on the failure of
strdup() before closing the handle, leaking the resource. Fix it by
removing the goto, that allows the code flow to close the handle and
execute the code under the end label.

Signed-off-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
(cherry picked from commit d8bf0b38a75ed06520d05d130c0c92234df8665f)

diff --git a/src/tools/cgget.c b/src/tools/cgget.c
index f80926ba74328635034b2a5667bfe59a54b8efce..4294980819be00e0db66d9ed6908be4a95d4de01 100644 (file)
--- a/src/tools/cgget.c
+++ b/src/tools/cgget.c
@@ -496,7 +496,7 @@ static int get_cv_value(struct control_value * const cv,
        strncpy(cv->value, tmp_line, CG_CONTROL_VALUE_MAX - 1);
        cv->multiline_value = strdup(cv->value);
        if (cv->multiline_value == NULL)
-               goto end;
+               goto read_end;
 
        while((ret = cgroup_read_value_next(&handle, tmp_line, LL_MAX)) == 0) {
                if (ret == 0) {