opensource/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-06-24 17:07:51 +08:00
Code Issues Packages Projects Releases Wiki Activity

In the current implementation of rtp_parse_packet(), finalize_packet() is

Browse Source 
called for all packets with an internal handler function but only for
non-first packets from dynamic payload parse_packet() handlers. This patch
fixes that. Bug was noticed by Luca in "[PATCH] rtpdec.c: don't overwrite
pkt->stream_index in finalize_packet()" thread.

Originally committed as revision 17764 to svn://svn.ffmpeg.org/ffmpeg/trunk
This commit is contained in:
Ronald S. Bultje
2009-03-03 13:41:50 +00:00
parent 2db272e93e
commit f3e71942e7
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
libavformat/rtpdec.c
View File

@ -473,6 +473,7 @@ int rtp_parse_packet(RTPDemuxContext *s, AVPacket *pkt,
s->read_buf_index = 0; s->read_buf_index = 0;
return 1; return 1;
} }
return 0;
} else if (s->parse_packet) { } else if (s->parse_packet) {
rv = s->parse_packet(s->ic, s->dynamic_protocol_context, rv = s->parse_packet(s->ic, s->dynamic_protocol_context,
s->st, pkt, &timestamp, buf, len, flags); s->st, pkt, &timestamp, buf, len, flags);
@ -535,10 +536,11 @@ int rtp_parse_packet(RTPDemuxContext *s, AVPacket *pkt,
memcpy(pkt->data, buf, len); memcpy(pkt->data, buf, len);
break; break;
} }
}
// now perform timestamp things.... // now perform timestamp things....
finalize_packet(s, pkt, timestamp); finalize_packet(s, pkt, timestamp);
}
return rv; return rv;
} }