diff options
| author | Pascal Quantin <pascal@wireshark.org> | 2021-06-26 10:31:59 +0200 |
|---|---|---|
| committer | Jaap Keuter <jaap.keuter@xs4all.nl> | 2021-07-10 10:51:40 +0000 |
| commit | 490fe2684bf63cf4fe03e6495c9ef74c8bc86a2b (patch) | |
| tree | dec32b1324ea80e988ac019dc68647e71caab396 | |
| parent | be7bb332fe6666ce0879233869eb252a07619680 (diff) | |
DNP: use the proper free function
g618661b22e introduced a free for a so called memory leak (which wasn't
a real leak due to the pinfo->pool garbage collector) but used the wrong
free function. Let's keep the explicit free but use the right function.
Closes #17462
(cherry picked from commit 029a7fcec5c6fc9407bd7f50ad6c3ce79f4d393b)
| -rw-r--r-- | epan/dissectors/packet-dnp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-dnp.c b/epan/dissectors/packet-dnp.c index c07ae24180..91a501b075 100644 --- a/epan/dissectors/packet-dnp.c +++ b/epan/dissectors/packet-dnp.c @@ -3455,7 +3455,7 @@ dissect_dnp3_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* else { /* CRC error - throw away the data. */ - g_free(al_buffer); + wmem_free(pinfo->pool, al_buffer); next_tvb = NULL; } } |