luo980/hiredis
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove dangling pointer dereference

Browse Source 
The destructor of RedisQtAdapter will be called
_AFTER_ the context has been freed. At this point,
m_ctx is a dangling pointer and should not be dereferenced.
This commit is contained in:
David Ochsner 2023-09-23 10:59:10 +02:00
parent 869f3d0ef1
commit 956f69ffdb
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
adapters/qt.h
View File

@ -72,12 +72,6 @@ class RedisQtAdapter : public QObject {
RedisQtAdapter(QObject * parent = 0)
: QObject(parent), m_ctx(0), m_read(0), m_write(0) { }
~RedisQtAdapter() {
if (m_ctx != 0) {
m_ctx->ev.data = NULL;
}
}
int setContext(redisAsyncContext * ac) {
if (ac->ev.data != NULL) {
return REDIS_ERR;