diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2013-12-12 12:52:00 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2013-12-12 12:57:27 -0500 |
commit | 85b8f7affa5fadb1972e82e04cc3d1828966b238 (patch) | |
tree | fc25f74b1182c6adb3986183b2fd39014157d330 /debugfs | |
parent | ddbe78528f664a43c447e5ffeb0b89fe9fed515d (diff) | |
download | e2fsprogs-85b8f7affa5fadb1972e82e04cc3d1828966b238.tar.gz |
debugfs: don't leak mmp_s memory (on error path)
ext2fs_free_mem() takes a pointer to a pointer, similar to
ext2fs_get_mem(). Improve the documentation, and fix debugfs.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'debugfs')
-rw-r--r-- | debugfs/set_fields.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/debugfs/set_fields.c b/debugfs/set_fields.c index b09e2f89..064de395 100644 --- a/debugfs/set_fields.c +++ b/debugfs/set_fields.c @@ -802,7 +802,7 @@ void do_set_mmp_value(int argc, char *argv[]) if (retval) { com_err(argv[0], retval, "reading MMP block %llu.\n", (long long)current_fs->super->s_mmp_block); - ext2fs_free_mem(mmp_s); + ext2fs_free_mem(&mmp_s); return; } current_fs->mmp_buf = mmp_s; |