Swiftdiff options
| author | Tobias Markmann <tm@ayena.de> | 2016-01-21 08:43:00 (GMT) |
|---|---|---|
| committer | Tobias Markmann <tm@ayena.de> | 2016-01-21 09:34:29 (GMT) |
| commit | f377207cb896679b4eab9f6773d9d071700852ad (patch) | |
| tree | ec31fa8a6feb3512c4d86c750a1eb5fb45fd75ef /Swiften/StreamStack/WhitespacePingLayer.h | |
| parent | 0ca3ea9a88acb5e2d42baa3a85d61495d72b7352 (diff) | |
| download | swift-f377207cb896679b4eab9f6773d9d071700852ad.zip swift-f377207cb896679b4eab9f6773d9d071700852ad.tar.bz2 | |
Stop timer during clean up of WhitespacePingLayer
AddressSanitizer reported a heap-buffer-overflow in
WhitespacePingLayer::handleTimerTick() which happened during
multiple restarts of a XMPP server. Under the assumption
that it was caused by not correctly stopping the timer, the
timer is now explicitly stopped if still active at destruction.
Test-Information:
Unable to reproduce the ASAN report. Unit tests still pass.
Change-Id: Ia0b7c3b613688750c4ce0ad40d759a0db4a52791
Diffstat (limited to 'Swiften/StreamStack/WhitespacePingLayer.h')
| -rw-r--r-- | Swiften/StreamStack/WhitespacePingLayer.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Swiften/StreamStack/WhitespacePingLayer.h b/Swiften/StreamStack/WhitespacePingLayer.h index 17b0654..7ed56ca 100644 --- a/Swiften/StreamStack/WhitespacePingLayer.h +++ b/Swiften/StreamStack/WhitespacePingLayer.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2010-2015 Isode Limited. + * Copyright (c) 2010-2016 Isode Limited. * All rights reserved. * See the COPYING file for more information. */ @@ -19,6 +19,7 @@ namespace Swift { class SWIFTEN_API WhitespacePingLayer : public StreamLayer, boost::noncopyable { public: WhitespacePingLayer(TimerFactory* timerFactory); + virtual ~WhitespacePingLayer(); void setActive(); void setInactive(); |