diff options
author | Yan, Zheng <zyan@redhat.com> | 2019-02-11 02:18:52 -0500 |
---|---|---|
committer | Ilya Dryomov <idryomov@gmail.com> | 2019-02-18 12:08:29 -0500 |
commit | 04242ff3ac0abbaa4362f97781dac268e6c3541a (patch) | |
tree | 26b1402043c10e71033660d05689bf6315be7e8c | |
parent | 0fd3fd0a9bb0b02b6435bb7070e9f7b82a23f068 (diff) |
ceph: avoid repeatedly adding inode to mdsc->snap_flush_list
Otherwise, mdsc->snap_flush_list may get corrupted.
Cc: stable@vger.kernel.org
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
-rw-r--r-- | fs/ceph/snap.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ceph/snap.c b/fs/ceph/snap.c index 041c27ea8de1..f74193da0e09 100644 --- a/fs/ceph/snap.c +++ b/fs/ceph/snap.c | |||
@@ -616,7 +616,8 @@ int __ceph_finish_cap_snap(struct ceph_inode_info *ci, | |||
616 | capsnap->size); | 616 | capsnap->size); |
617 | 617 | ||
618 | spin_lock(&mdsc->snap_flush_lock); | 618 | spin_lock(&mdsc->snap_flush_lock); |
619 | list_add_tail(&ci->i_snap_flush_item, &mdsc->snap_flush_list); | 619 | if (list_empty(&ci->i_snap_flush_item)) |
620 | list_add_tail(&ci->i_snap_flush_item, &mdsc->snap_flush_list); | ||
620 | spin_unlock(&mdsc->snap_flush_lock); | 621 | spin_unlock(&mdsc->snap_flush_lock); |
621 | return 1; /* caller may want to ceph_flush_snaps */ | 622 | return 1; /* caller may want to ceph_flush_snaps */ |
622 | } | 623 | } |