diff options
| author | Josh Allmann <joshua.allmann@gmail.com> | 2010-06-14 08:23:59 +0000 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2010-06-14 08:23:59 +0000 |
| commit | afcea58c53a54b0238601106bf668b990251e94e (patch) | |
| tree | 67768115fad1d29d42a4cd1b9dc6e7110a96ecea /libavformat/rtsp.c | |
| parent | 41874d0a5df35732367f0c675eac914c23d65aee (diff) | |
| download | ffmpeg-afcea58c53a54b0238601106bf668b990251e94e.tar.gz | |
RTSP: Shrink SDP fmtp parsing buffer size
Since the parsing of Vorbis/Theora fmtp headers is handled by the
parse_sdp_a_line function pointer now, the buffer in sdp_parse_fmtp
doesn't need to be this large any longer.
Patch by Josh Allmann, joshua dot allmann at gmail
Originally committed as revision 23599 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavformat/rtsp.c')
| -rw-r--r-- | libavformat/rtsp.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index fc422aa488..f1bd870b50 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -273,9 +273,7 @@ int ff_rtsp_next_attr_and_value(const char **p, char *attr, int attr_size, static void sdp_parse_fmtp(AVStream *st, const char *p) { char attr[256]; - /* Vorbis setup headers can be up to 12KB and are sent base64 - * encoded, giving a 12KB * (4/3) = 16KB FMTP line. */ - char value[16384]; + char value[4096]; int i; RTSPStream *rtsp_st = st->priv_data; AVCodecContext *codec = st->codec; @@ -541,7 +539,8 @@ static int sdp_parse(AVFormatContext *s, const char *content) * "rulebooks" describing their properties. Therefore, the SDP line * buffer is large. * - * The Vorbis FMTP line can be up to 16KB - see sdp_parse_fmtp. */ + * The Vorbis FMTP line can be up to 16KB - see xiph_parse_sdp_line + * in rtpdec_xiph.c. */ char buf[16384], *q; SDPParseState sdp_parse_state, *s1 = &sdp_parse_state; |