Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

issues/3332 use-after-free #3651

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kmokeddem
Copy link

calling m_timer.reset() detroys m_ioService which is later used in doneWorking().

#3332

@chfast
Copy link
Collaborator

chfast commented Mar 17, 2017

Hey @kmokeddem,
thanks for checking this one.

You mentions a test for this. Can it be included?

@chfast
Copy link
Collaborator

chfast commented Mar 20, 2017

It does not fix the issue I think. Also some tests don't want to finish after the change.

I think the issue is more about m_nodeTable.reset() line a bit above your change.

Do you know how to build with ASan? Add option -DSANITIZE=address to cmake config.

@chfast chfast changed the base branch from develop to master August 2, 2018 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants