diff options
| author | Dave Jones <davej@redhat.com> | 2013-10-11 00:05:35 +0000 |
|---|---|---|
| committer | Osvaldo Banuelos <osvaldob@codeaurora.org> | 2014-01-06 14:28:28 -0800 |
| commit | 052c8ea195a4fd456909397c74fb0bb867ef5109 (patch) | |
| tree | 112dc55f7fd683eb1696e6175a26369ac1aab987 /scripts/build-all.py | |
| parent | 08a9c39ed854446569f8d17376c82d28f44bd268 (diff) | |
ext4: fix memory leak in xattr
If we take the 2nd retry path in ext4_expand_extra_isize_ea, we
potentionally return from the function without having freed these
allocations. If we don't do the return, we over-write the previous
allocation pointers, so we leak either way.
Spotted with Coverity.
[ Fixed by tytso to set is and bs to NULL after freeing these
pointers, in case in the retry loop we later end up triggering an
error causing a jump to cleanup, at which point we could have a double
free bug. -- Ted ]
Change-Id: I49b8ca41a6c6d44b563eb23306870258a3affd3b
Signed-off-by: Dave Jones <davej@fedoraproject.org>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Cc: stable@vger.kernel.org
Git-commit: 6e4ea8e33b2057b85d75175dd89b93f5e26de3bc
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
Signed-off-by: Osvaldo Banuelos <osvaldob@codeaurora.org>
Diffstat (limited to 'scripts/build-all.py')
0 files changed, 0 insertions, 0 deletions