[FFmpeg-devel] [PATCH] RTP reordering, again

Martin Storsjö martin
Fri Oct 1 14:47:47 CEST 2010


On Fri, 1 Oct 2010, Luca Barbato wrote:

> On 09/30/2010 02:11 PM, Martin Storsj? wrote:
> > +++ b/libavformat/rtsp.c
> > @@ -1845,7 +1845,7 @@ static int rtsp_fetch_packet(AVFormatContext *s,
> AVPacket *pkt)
> >       if (rt->transport == RTSP_TRANSPORT_RDT) {
> >           ret = ff_rdt_parse_packet(rtsp_st->transport_priv, pkt,
> rt->recvbuf, len);
> >       } else {
> > -        ret = rtp_parse_packet(rtsp_st->transport_priv, pkt, rt->recvbuf,
> len);
> > +        ret = rtp_parse_packet(rtsp_st->transport_priv, pkt,&rt->recvbuf,
> len);
> 
> what about ff_rdt_parse_packet ?

If there's a need for the RDT code to take ownership of the buffer, we can 
change ff_rdt_parse_packet to take an uint8_t** argument, too, as a 
completely separate change.

> The rest seems ok

Ok, good, good. :-)

// Martin



More information about the ffmpeg-devel mailing list