diff options
author | Sage Weil <sage@inktank.com> | 2013-09-30 16:33:22 -0700 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-09-30 16:33:22 -0700 |
commit | 80cc908bd1cdaf75ac088bfdde1ff61fe32b77e9 (patch) | |
tree | 9f381b9c1d4d58c4da0dc5504bc5e8a15d10d4c1 | |
parent | 56711370c3a7dfce683f73f1f4b7b5ccd66cbeb5 (diff) | |
download | ceph-wip-cancel-copy.tar.gz |
osd/ReplicatedPG: fix iterator corruption in cancel_copy_ops()wip-cancel-copy
The cancel_copy() method removes the entry from copy_ops. Move the
iterator forward before calling.
Fixes segfault when thrashing osds with a copy-from workload.
Signed-off-by: Sage Weil <sage@inktank.com>
-rw-r--r-- | src/osd/ReplicatedPG.cc | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index 5e099c0853d..bc995a6fc86 100644 --- a/src/osd/ReplicatedPG.cc +++ b/src/osd/ReplicatedPG.cc @@ -4583,10 +4583,11 @@ void ReplicatedPG::cancel_copy(CopyOpRef cop) void ReplicatedPG::cancel_copy_ops() { dout(10) << __func__ << dendl; - for (map<hobject_t,CopyOpRef>::iterator p = copy_ops.begin(); - p != copy_ops.end(); - copy_ops.erase(p++)) { - cancel_copy(p->second); + map<hobject_t,CopyOpRef>::iterator p = copy_ops.begin(); + while (p != copy_ops.end()) { + CopyOpRef cop = p->second; + p++; + cancel_copy(cop); } } |