[MPlayer-dev-eng] [PATCH] boundary check for demux_real.c

陆然 hephooey at gmail.com
Thu Jul 12 16:13:59 CEST 2007


Hi,

On Wednesday 11 July 2007, Roberto Togni wrote:
> Hmm, if the fragment belongs to the next packet (the previous packet is
> incomplete because of data loss) it may be wrongly discarded.
> Can you test the attached patch?

Correct me if I'm wrong, it looks to me like the patch only checks the 
appending data.  If I get some garbage looks like a start of a packet, Is 
this patch going to stop the crash. How about we combine the two checks. 
Check if len is larger than the packet length first, then see if it will fit 
in the remaining space of the packet.

-- 
Best Regards,
LR



More information about the MPlayer-dev-eng mailing list