Commit 9b5b1f5b authored by Trond Myklebust's avatar Trond Myklebust

NLM: Fix Oops in nlmclnt_mark_reclaim()

 When mixing -olock and -onolock mounts on the same client, we have to
 check that fl->fl_u.nfs_fl.owner is set before dereferencing it.
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 48e49187
...@@ -157,6 +157,8 @@ void nlmclnt_mark_reclaim(struct nlm_host *host) ...@@ -157,6 +157,8 @@ void nlmclnt_mark_reclaim(struct nlm_host *host)
inode = fl->fl_file->f_dentry->d_inode; inode = fl->fl_file->f_dentry->d_inode;
if (inode->i_sb->s_magic != NFS_SUPER_MAGIC) if (inode->i_sb->s_magic != NFS_SUPER_MAGIC)
continue; continue;
if (fl->fl_u.nfs_fl.owner == NULL)
continue;
if (fl->fl_u.nfs_fl.owner->host != host) if (fl->fl_u.nfs_fl.owner->host != host)
continue; continue;
if (!(fl->fl_u.nfs_fl.flags & NFS_LCK_GRANTED)) if (!(fl->fl_u.nfs_fl.flags & NFS_LCK_GRANTED))
...@@ -226,6 +228,8 @@ reclaimer(void *ptr) ...@@ -226,6 +228,8 @@ reclaimer(void *ptr)
inode = fl->fl_file->f_dentry->d_inode; inode = fl->fl_file->f_dentry->d_inode;
if (inode->i_sb->s_magic != NFS_SUPER_MAGIC) if (inode->i_sb->s_magic != NFS_SUPER_MAGIC)
continue; continue;
if (fl->fl_u.nfs_fl.owner == NULL)
continue;
if (fl->fl_u.nfs_fl.owner->host != host) if (fl->fl_u.nfs_fl.owner->host != host)
continue; continue;
if (!(fl->fl_u.nfs_fl.flags & NFS_LCK_RECLAIM)) if (!(fl->fl_u.nfs_fl.flags & NFS_LCK_RECLAIM))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment