diff options
author | Jinjie Ruan <ruanjinjie@huawei.com> | 2023-09-03 09:10:26 +0200 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2023-09-05 20:29:55 +0200 |
commit | 4b00920da1dd2bbb33baeb2e7b9808af4c68de97 (patch) | |
tree | 1613042f4302bb1d7617c0766592997cfae2278c /MAINTAINERS | |
parent | kunit: Fix wild-memory-access bug in kunit_free_suite_set() (diff) | |
download | linux-4b00920da1dd2bbb33baeb2e7b9808af4c68de97.tar.xz linux-4b00920da1dd2bbb33baeb2e7b9808af4c68de97.zip |
kunit: Fix the wrong err path and add goto labels in kunit_filter_suites()
Take the last kfree(parsed_filters) and add it to be the first. Take
the first kfree(copy) and add it to be the last. The Best practice is to
return these errors reversely.
And as David suggested, add several labels which target only the things
which actually have been allocated so far.
Fixes: 529534e8cba3 ("kunit: Add ability to filter attributes")
Fixes: abbf73816b6f ("kunit: fix possible memory leak in kunit_filter_suites()")
Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com>
Reviewed-by: Rae Moar <rmoar@google.com>
Suggested-by: David Gow <davidgow@google.com>
Reviewed-by: David Gow <davidgow@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'MAINTAINERS')
0 files changed, 0 insertions, 0 deletions