]> git.ipfire.org Git - thirdparty/libcgroup.git/commit
wrapper.c: return on failure in cgroup_add_all_controllers()
authorKamalesh Babulal <kamalesh.babulal@oracle.com>
Fri, 8 Jul 2022 16:42:29 +0000 (10:42 -0600)
committerTom Hromatka <tom.hromatka@oracle.com>
Fri, 8 Jul 2022 16:42:37 +0000 (10:42 -0600)
commit71eb00ef127d4e626699ee4a27a0a14c7d5145ff
treeadd6e093281923b00a1ef6ecb4db06091e58721c
parent7cd6e36a7c3f89364888804e79c31c1ff0c98c19
wrapper.c: return on failure in cgroup_add_all_controllers()

Add missing goto statement, to return on failure, reported by Coverity
tool:

CID 258281 (#1 of 1): Unused value (UNUSED_VALUE)assigned_value:
Assigning value ECGINVAL to ret here, but that stored value is
overwritten before it can be used.

the tool reported about unused ret value, but it turned out that the
NULL ret value is for failed attempt to add a controller to the desired
cgroup and needs bailing out after losing the handle. Fix it by
introducing the goto statement in cgroup_add_all_controllers()

Signed-off-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
TJH: Also fix unrelated grammar error near this change.
(cherry picked from commit 83d64a4474366938e1459403eb0c98c3d3b7ee00)
src/wrapper.c