On 3/27/19 8:56 PM, Roman Gushchin wrote:
Dan reported, that cleanup path in test_memcg_subtree_control() triggers a static checker warning: ./tools/testing/selftests/cgroup/test_memcontrol.c:76 \ test_memcg_subtree_control() error: uninitialized symbol 'child2'.
Fix this by initializing child2 and parent2 variables and split the cleanup path into few stages.
Signed-off-by: Roman Gushchin guro@fb.com Fixes: 84092dbcf901 ("selftests: cgroup: add memory controller self-tests") Reported-by: Dan Carpenter dan.carpenter@oracle.com Cc: Dan Carpenter dan.carpenter@oracle.com Cc: Shuah Khan (Samsung OSG) shuah@kernel.org Cc: Mike Rapoport rppt@linux.vnet.ibm.com
From and Signed-off don't match. Please fix it and resend the patch.
thanks, -- Shuah