[FFmpeg-devel] [PATCH] sdp: Add a framesize attribute to H263 SDP descriptions

Martin Storsjo martin
Mon Dec 20 13:42:24 CET 2010

While not mentioned in RFC 4629, this is required for H263 in
3GPP TS 26.234. It is also required in practice for playback with
Android stagefright and on Samsung bada phones.

I'll apply this soon if there are no objections.

 libavformat/sdp.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/libavformat/sdp.c b/libavformat/sdp.c
index 35cbabc..86e67c2 100644
--- a/libavformat/sdp.c
+++ b/libavformat/sdp.c
@@ -314,7 +314,10 @@ static char *sdp_write_media_attributes(char *buff, int size, AVCodecContext *c,
         case CODEC_ID_H263:
         case CODEC_ID_H263P:
-            av_strlcatf(buff, size, "a=rtpmap:%d H263-2000/90000\r\n", payload_type);
+            av_strlcatf(buff, size, "a=rtpmap:%d H263-2000/90000\r\n"
+                                    "a=framesize:%d %d-%d\r\n",
+                                    payload_type,
+                                    payload_type, c->width, c->height);
         case CODEC_ID_MPEG4:
             if (c->extradata_size) {

More information about the ffmpeg-devel mailing list