[FFmpeg-devel] [PATCH 2/6] avformat/rtspdec: use SDP_MAX_SIZE for sdp array

lance.lmwang at gmail.com lance.lmwang at gmail.com
Wed Nov 4 15:52:13 EET 2020


On Wed, Nov 04, 2020 at 03:44:22PM +0200, Martin Storsjö wrote:
> On Wed, 4 Nov 2020, lance.lmwang at gmail.com wrote:
> 
> > From: Limin Wang <lance.lmwang at gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > ---
> > libavformat/rtspdec.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
> > index ef084a8..3abf34b 100644
> > --- a/libavformat/rtspdec.c
> > +++ b/libavformat/rtspdec.c
> > @@ -172,7 +172,7 @@ static int rtsp_read_announce(AVFormatContext *s)
> > {
> >     RTSPState *rt             = s->priv_data;
> >     RTSPMessageHeader request = { 0 };
> > -    char sdp[4096];
> > +    char sdp[SDP_MAX_SIZE];
> >     int  ret;
> 
> How can we use this define in this file, in patch 2/6, when it's only
> available in rtsp.c and rtspenc.c at this stage (and it's moved to the
> header in patch 3/6)?
Sorry, my fault, the patch order isn't right when I apply to the master.
will reformat them.

> 
> // Martin
> 

-- 
Thanks,
Limin Wang


More information about the ffmpeg-devel mailing list