[MPlayer-dev-eng] [PATCH] fix FreeSDP memleak of media level connection address

Stefan Pöschel stefan.poeschel at gmx.de
Thu Mar 13 22:30:22 CET 2014


Hi,

the FreeSDP lib snapshot in mplayer contains a memleak, where a possible
connection address on media level is not freed. This connection address
is allocated via strdup in line 941/951 of /stream/freesdp/parser.c.
In the process of freeing a fsdp_description_t object
(fsdp_description_recycle in /stream/freesdp/commmon.c), a possible
connection address on session level already gets freed, but not such one
at media level.

The free this patch introduces is already contained in the latest CVS
version of FreeSDP (though commented, for no reason I can follow).
I tested the patch before/after with connection address both on session
and on media level with Valgrind successfully.

Regards,
	Stefan


-------------- next part --------------
A non-text attachment was scrubbed...
Name: freesdp_media_level_connection_address_memleak.diff
Type: text/x-patch
Size: 532 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20140313/e9631e2c/attachment.bin>


More information about the MPlayer-dev-eng mailing list