summaryrefslogtreecommitdiff
path: root/fs/ext4/move_extent.c
diff options
context:
space:
mode:
authorCarlos Maiolino <cmaiolino@redhat.com>2012-09-27 17:31:33 +0400
committerTheodore Ts'o <tytso@mit.edu>2012-09-27 17:31:33 +0400
commit6d1ab10e69ff5f3cb63920ba965ec0f1f0bdaf8d (patch)
tree7a6efd62087edb8103c6b409aa81e0229bdd4ef4 /fs/ext4/move_extent.c
parentbbdd68086ca4a8976226e23efd08e2058d34dd81 (diff)
downloadlinux-6d1ab10e69ff5f3cb63920ba965ec0f1f0bdaf8d.tar.xz
ext4: ext4_bread usage audit
When ext4_bread() returns NULL and err is set to zero, this means there is no phyical block mapped to the specified logical block number. (Previous to commit 90b0a97323, err was uninitialized in this case, which caused other problems.) The directory handling routines use ext4_bread() in many places, the fact that ext4_bread() now returns NULL with err set to zero could cause problems since a number of these functions will simply return the value of err if the result of ext4_bread() was the NULL pointer, causing the caller of the function to think that the function was successful. Since directories should never contain holes, this case can only happen if the file system is corrupted. This commit audits all of the callers of ext4_bread(), and makes sure they do the right thing if a hole in a directory is found by ext4_bread(). Some ext4_bread() callers did not need any changes either because they already had its own hole detector paths. Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/move_extent.c')
0 files changed, 0 insertions, 0 deletions