dht: fix sendListen loop bug #103

Merged
merged 1 commit into from Aug 4, 2016

Conversation

Projects
None yet
2 participants
@sim590
Contributor

sim590 commented Aug 3, 2016

This is addressing the issue #102.

dht: fix sendListen loop bug
Each time a 'listen' request is sent to the network engine, we track this
request in the upper layer with a map<shared_ptr<Query>, shared_ptr<Request>>.
At some point, this map has to be maintained or totally cleared. However, it was
done wrong, hence the sendListen loop. Clearing a pair <Query, Request> has to
be done when the API user cancels a 'listen' operation.

@sim590 sim590 added the bug label Aug 3, 2016

@kaldoran

This comment has been minimized.

Show comment
Hide comment
@kaldoran

kaldoran Aug 3, 2016

Collaborator

Well cant merge on phone ( so sad ).

Edit : It compile and seems to work lets go for the merge

Collaborator

kaldoran commented Aug 3, 2016

Well cant merge on phone ( so sad ).

Edit : It compile and seems to work lets go for the merge

@kaldoran kaldoran merged commit b366432 into savoirfairelinux:master Aug 4, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment