Skip to content
This repository has been archived by the owner on Apr 6, 2019. It is now read-only.

Concurrency issue #5

Closed
tobbe303 opened this issue Nov 30, 2015 · 2 comments
Closed

Concurrency issue #5

tobbe303 opened this issue Nov 30, 2015 · 2 comments
Labels
unix

Comments

@tobbe303
Copy link

Having an application using both a redis_client and a redis_subscriber I receive numerous errors from Valgrind when destructing these objects. Possibly have something to do with destroying the redis_connections, but I'm not sure.

The problem can be reproduced by modifying the redis_subscriber example adding an instance of a redis_client and making a connect() on this object. See attached files.

Modified redis_subscriber
redis_subscriber.cpp.txt

Valgrind output
valgrind_output.txt
@Cylix
Copy link
Owner

Cylix commented Nov 30, 2015

Thanks for the report and the details. I look at this as soon as possible.

@Cylix Cylix mentioned this issue Jul 22, 2016
Merged
@Cylix
Copy link
Owner

Cylix commented Jul 22, 2016

This issue is now patched with #8, mostly by removing the Boost.Asio dependency.

@Cylix Cylix closed this as completed Jul 22, 2016
cmorse pushed a commit to cmorse/cpp_redis that referenced this issue Oct 18, 2017
@Cylix
Download gtest from cmakelist (Cylix#5)
59a50f1 
* download gtest from cmakelist

* remove install-deps from travis

* fix cmakelist for unix
@LazyPlanet LazyPlanet mentioned this issue Nov 19, 2017
Closed
@kiranpradeep kiranpradeep mentioned this issue Dec 23, 2018
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
unix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Cylix @tobbe303