[vlc-devel] [patch] Adding AMT module as a network access source. AMT allows IP multicast streams to be tunnelled over a non-multicast enabled network
wbrassem at rogers.com
Fri Apr 5 20:39:54 CEST 2019
Yeah, I know but strcpy() doesn’t allocate any memory, merely uses the existing buffer. Anyways in the new patch I standardized on using strdup() which allocates so I know I have to free each time.
> On Apr 5, 2019, at 11:13 AM, Tristan Matthews <tmatth at videolan.org> wrote:
> On Fri, Apr 5, 2019 at 11:03 AM Wayne Brassem <wbrassem at rogers.com> wrote:
>> I’ve checked the code over and can’t see how any of the referenced sys->srcAddr statements below gets leaked. This buffer is freed in Close(), but is in use until then.
>> What am I missing? I don’t want to resubmit if it’s still wrong.
> In the previous patch submission, you were overwriting sys->srcAddr
> with a strcpy without freeing it first.
> The patch you just sent does not have this issue.
> I'll take a closer look at the latest patch soon.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel