https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=15231
Peter Wu <pe...@lekensteyn.nl> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[oss-fuzz] ASAN: 22 byte(s) |[oss-fuzz] ASAN: 22 bytes
|leaked in 1 allocation(s). |leaked in
| |epan/dissectors/packet-rtp-
| |ed137.c:688
CC| |j.no...@netsystem.cz
--- Comment #2 from Peter Wu <pe...@lekensteyn.nl> ---
Hi Jiri, the memleak is rather trivial and could be solved by replacing:
tmp = rel_time_to_secs_str(NULL, &tmp_time);
by
tmp = rel_time_to_secs_str(wmem_packet_scope(), &tmp_time);
However I think that the process_time_value function can be further cleaned up.
Both branches are equivalent (except for the comment). If that is intentional,
then one of them should be removed.
Though if the NTP value is absolute, shouldn't it be using abs_time_to_str
instead? (That will not display a time in seconds, but in a more human-readable
form).
Do you have a packet capture file that contains both forms? Any suggestion on
the correct form?
--
You are receiving this mail because:
You are watching all bug changes.___________________________________________________________________________
Sent via: Wireshark-bugs mailing list <wireshark-bugs@wireshark.org>
Archives: https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe
