mirror of
https://github.com/meshtastic/firmware.git
synced 2025-02-02 02:39:57 +00:00
Merge pull request #2295 from meshtastic/2228-bug-heap-leak-in-rangetestmodule
tryfix #2228 as suggested by @mverch67
This commit is contained in:
commit
9fc18c2a19
@ -148,7 +148,11 @@ bool ReliableRouter::stopRetransmission(GlobalPacketId key)
|
|||||||
if (old) {
|
if (old) {
|
||||||
auto numErased = pending.erase(key);
|
auto numErased = pending.erase(key);
|
||||||
assert(numErased == 1);
|
assert(numErased == 1);
|
||||||
|
// remove the 'original' (identified by originator and packet->id) from the txqueue and free it
|
||||||
cancelSending(getFrom(old->packet), old->packet->id);
|
cancelSending(getFrom(old->packet), old->packet->id);
|
||||||
|
// now free the pooled copy for retransmission too. tryfix for #2228
|
||||||
|
if (old->packet)
|
||||||
|
packetPool.release(old->packet);
|
||||||
return true;
|
return true;
|
||||||
} else
|
} else
|
||||||
return false;
|
return false;
|
||||||
|
Loading…
Reference in New Issue
Block a user