- Differentiate swr::ReplyError from connection errors in xadd/xaddTrim so server-rejected commands (e.g. out-of-order IDs) don't trigger unnecessary reconnect cycles that corrupt heap state - Store reconnect thread as member and join in destructor instead of detaching, preventing use-after-free on object destruction - Add mutex protection for _reader map to prevent data races between reader management, reconnect thread, and destructor - Fix iterator invalidation in reconnect by moving NO_TOKEN entries out of the map before inserting new entries - Make _watchdog_run, _readers_defer, and RedisCache::readIndex atomic to eliminate data races with concurrent thread access - Fix null pointer dereference in RedisCache::copyReadBuffer when pElementsCopied defaults to nullptr on the early-return path - Fix MockRedisAdapter: new/delete[] mismatch in addSingleValue, size_bytes()->size() for correct allocation, wrong counter increment - Return unique_ptr from RedisConnection::subscriber() instead of raw pointer to prevent memory leaks - Add shutdown flag to prevent reconnect during destruction - Fix typos: maxTme -> maxTime, newValueAvaliable -> newValueAvailable - Add unit tests for out-of-order timestamps, destructor cleanup, and concurrent reader modification Fixes #78, #79, #80, #81, #82, #83, #84, #85, #86, #87, #88, #89, #90

Uh oh!

Typo: maxTme variable name in RedisAdapterTempl.hpp #89

Description

Description

Severity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions