diff options
author | Ronnie Sahlberg <lsahlber@redhat.com> | 2018-08-30 10:12:59 +1000 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2018-10-02 18:12:31 -0500 |
commit | ddf83afb9f60ee58cdec30bb615eff65d00ba05e (patch) | |
tree | 41a64ddbced37ea36da1defdcd8f2eb6c0a270fa /fs/cifs/transport.c | |
parent | 0595751f267994c3c7027377058e4185b3a28e75 (diff) | |
download | linux-stable-ddf83afb9f60ee58cdec30bb615eff65d00ba05e.tar.gz linux-stable-ddf83afb9f60ee58cdec30bb615eff65d00ba05e.tar.bz2 linux-stable-ddf83afb9f60ee58cdec30bb615eff65d00ba05e.zip |
cifs: add a warning if we try to to dequeue a deleted mid
cifs_delete_mid() is called once we are finished handling a mid and we
expect no more work done on this mid.
Needed to fix recent commit:
commit 730928c8f4be88e9d6a027a16b1e8fa9c59fc077
("cifs: update smb2_queryfs() to use compounding")
Add a warning if someone tries to dequeue a mid that has already been
flagged to be deleted.
Also change list_del() to list_del_init() so that if we have similar bugs
resurface in the future we will not oops.
Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Diffstat (limited to 'fs/cifs/transport.c')
-rw-r--r-- | fs/cifs/transport.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/cifs/transport.c b/fs/cifs/transport.c index 78f96fa3d7d9..9cc9a127749e 100644 --- a/fs/cifs/transport.c +++ b/fs/cifs/transport.c @@ -142,7 +142,8 @@ void cifs_delete_mid(struct mid_q_entry *mid) { spin_lock(&GlobalMid_Lock); - list_del(&mid->qhead); + list_del_init(&mid->qhead); + mid->mid_flags |= MID_DELETED; spin_unlock(&GlobalMid_Lock); DeleteMidQEntry(mid); |