On Mon, Sep 25, 2023 at 12:52:21PM +0800, Su Hui wrote: > diff --git a/fs/smb/client/file.c b/fs/smb/client/file.c > index 2108b3b40ce9..37eed057ded0 100644 > --- a/fs/smb/client/file.c > +++ b/fs/smb/client/file.c > @@ -4878,6 +4878,12 @@ void cifs_oplock_break(struct work_struct *work) > struct cifsFileInfo *cfile = container_of(work, struct cifsFileInfo, > oplock_break); > struct inode *inode = d_inode(cfile->dentry); > + > + if (!inode) { > + cifs_dbg(FYI, "%s : failed to find inode\n", __func__); > + return; > + } > + Are we allowing this in the middle of the declaration block these days? > struct cifs_sb_info *cifs_sb = CIFS_SB(inode->i_sb); > struct cifsInodeInfo *cinode = CIFS_I(inode); > struct cifs_tcon *tcon; regards, dan carpenter