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
Member

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
Member

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.

3 participants