diff options
author | Alessio Balsini <balsini@android.com> | 2019-10-21 11:02:32 +0100 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2019-10-26 17:36:46 -0500 |
commit | 55d937611597a4e54a17b344c81810ad1fe9f75d (patch) | |
tree | 53934600cf92e250de21996c4dfe8e1e5342d380 /tests/setfattr.test | |
parent | 49feb50f22e2db90556f79dbc0de36f8ecb8b521 (diff) | |
download | toybox-55d937611597a4e54a17b344c81810ad1fe9f75d.tar.gz |
losetup: Fix memory leaks in loopback_setup()
The function loopback_setup() uses xabspath() to get the loopback path.
This function allocates dynamic memory which should be freed by the
function caller.
But there are early return cases where the dynamic memory is not freed.
Besides the special cases of perror_exit(), for which the "early" free
operation is simply used to silence memory analysis tools, the
if (racy && errno == EBUSY) return 1;
branch may be a real cause of memory leak.
Fix by adding a new free() in the racy+EBUSY branch and anticipating the
existing free().
Signed-off-by: Alessio Balsini <balsini@android.com>
Diffstat (limited to 'tests/setfattr.test')
0 files changed, 0 insertions, 0 deletions