[FFmpeg-devel] [RFC/PATCH] Pass PRIVATE_STREAM_2 MPEG-PS packets to caller

Richard peper03 at yahoo.com
Sat Mar 2 20:46:05 CET 2013


On 02/03/13 13:18, Michael Niedermayer wrote:
> On Tue, Feb 26, 2013 at 05:45:23PM +0100, Richard wrote:
>> On 25/02/13 17:23, Richard wrote:

<snip>

>> +            s->pts = (int64_t)startpts;
>> +            s->duration = endpts - startpts;
>
> missing validity checks

Sorry, missed this in the last reply.  What would be an appropriate 
validity check?  Both fields are unsigned 32-bit values.  As far as I 
can tell, the full range is available to both, so I don't know what 
could be checked.

If you just mean endpts > startpts, then sure, I'll add that.

Richard.



More information about the ffmpeg-devel mailing list