git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit

]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit

author	Kefu Chai <kchai@redhat.com>
	Thu, 13 Jul 2017 06:49:48 +0000 (14:49 +0800)
committer	Kefu Chai <kchai@redhat.com>
	Sun, 20 Aug 2017 06:14:30 +0000 (14:14 +0800)
commit	2449b3a5c365987746ada095fde30e3dc63ee0c7
tree	ebcb6debbe5ec5cb05778e0f52d1d3ae54672b69	tree \| snapshot
parent	e407049a6a9cb588f27ab270948c404159aa2205	commit \| diff

common,mds,mgr,mon,osd: store event only if it's added

otherwise
* we will try to cancel it even it's never been added
* we will keep a dangling pointer around. which is, well,
scaring.
* static analyzer will yell at us:
Memory - illegal accesses (USE_AFTER_FREE)

Signed-off-by: Kefu Chai <kchai@redhat.com>

28 files changed:

src/client/Client.cc		diff \| blob \| history
src/client/Client.h		diff \| blob \| history
src/common/Timer.cc		diff \| blob \| history
src/common/Timer.h		diff \| blob \| history
src/journal/JournalMetadata.cc		diff \| blob \| history
src/journal/ObjectPlayer.cc		diff \| blob \| history
src/journal/ObjectPlayer.h		diff \| blob \| history
src/journal/ObjectRecorder.cc		diff \| blob \| history
src/journal/ObjectRecorder.h		diff \| blob \| history
src/mds/Beacon.cc		diff \| blob \| history
src/mds/Beacon.h		diff \| blob \| history
src/mds/MDSDaemon.cc		diff \| blob \| history
src/mds/MDSDaemon.h		diff \| blob \| history
src/mgr/MgrClient.cc		diff \| blob \| history
src/mon/Elector.cc		diff \| blob \| history
src/mon/MgrMonitor.cc		diff \| blob \| history
src/mon/Monitor.cc		diff \| blob \| history
src/mon/Paxos.cc		diff \| blob \| history
src/mon/PaxosService.cc		diff \| blob \| history
src/osd/PrimaryLogPG.h		diff \| blob \| history
src/osd/Watch.cc		diff \| blob \| history
src/test/perf_local.cc		diff \| blob \| history
src/test/rbd_mirror/mock/MockSafeTimer.h		diff \| blob \| history
src/test/rbd_mirror/test_mock_ImageReplayer.cc		diff \| blob \| history
src/test/rbd_mirror/test_mock_InstanceReplayer.cc		diff \| blob \| history
src/test/rbd_mirror/test_mock_PoolWatcher.cc		diff \| blob \| history
src/tools/rbd_mirror/PoolWatcher.cc		diff \| blob \| history
src/tools/rbd_mirror/image_sync/ImageCopyRequest.cc		diff \| blob \| history

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom