maybe return NULL when find a extent, if we use it later, it will cause oops. Signed-off-by: fanchaoting<fanchaoting@xxxxxxxxxxxxxx> --- fs/nfs/blocklayout/extents.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/fs/nfs/blocklayout/extents.c b/fs/nfs/blocklayout/extents.c index 9c3e117..131ea59 100644 --- a/fs/nfs/blocklayout/extents.c +++ b/fs/nfs/blocklayout/extents.c @@ -762,6 +762,11 @@ set_to_rw(struct pnfs_block_layout *bl, u64 offset, u64 length) spin_lock(&bl->bl_ext_lock); be = bl_find_get_extent_locked(bl, offset); + if (!be) { + spin_unlock(&bl->bl_ext_lock); + goto out_nosplit; + } + rv = be->be_f_offset + be->be_length; if (be->be_state != PNFS_BLOCK_INVALID_DATA) { spin_unlock(&bl->bl_ext_lock); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html