]> git.ipfire.org Git - thirdparty/asterisk.git/commit
rtp_engine: Skip useless self-assignment in ast_rtp_engine_unload_format. 65/765/1
authorWalter Doekes <walter+asterisk@wjd.nu>
Thu, 2 Jul 2015 11:10:59 +0000 (13:10 +0200)
committerWalter Doekes <walter+asterisk@wjd.nu>
Thu, 2 Jul 2015 11:10:59 +0000 (13:10 +0200)
commitc139956e9570bf1e86af61199612600e0ad06bbf
tree7e300958f2992dfdf94fda8cd437dd1142c4102e
parent9b74dcb687f4e05da012d80ee54d04df82210520
rtp_engine: Skip useless self-assignment in ast_rtp_engine_unload_format.

When running valgrind on Asterisk, it complained about:

    ==32423== Source and destination overlap in memcpy(0x85a920, 0x85a920, 304)
    ==32423==    at 0x4C2F71C: memcpy@@GLIBC_2.14 (in /usr/lib/valgrind/...)
    ==32423==    by 0x55BA91: ast_rtp_engine_unload_format (rtp_engine.c:2292)
    ==32423==    by 0x4EEFB7: ast_format_attr_unreg_interface (format.c:1437)

The code in question is a struct assignment, which may be performed by
memcpy as a compiler optimization. It is changed to only copy the struct
contents if source and destination are different.

ASTERISK-25219 #close

Change-Id: I6d3546c326b03378ca8e9b8cefd41c16e0088b9a
main/rtp_engine.c