Hey Zhilizhao, The RTSPState you mentioned is ok, but we need to get the char control_uri buffer from 1024 characters to 2048. Some of the long rtsps links are getting cut, and cannot play. Can you help getting this patch to the next version? The patch does not show up in https://patchwork.ffmpeg.org/ at the moment.
*• **Yiğit (Yeet) Uyan* *•* Smart Home Intelligence | Google Home *•* yigitu...@google.com On Wed, Aug 5, 2020 at 7:58 PM zhilizhao <quinkbl...@foxmail.com> wrote: > > > > On Aug 6, 2020, at 10:46 AM, zhilizhao <quinkbl...@foxmail.com> wrote: > > > > > > > >> On Aug 6, 2020, at 6:38 AM, Yiğit (Yeet) Uyan < > yigituyan-at-google....@ffmpeg.org> wrote: > >> > >> Hello everyone, > >> > >> This is the first time for me to send a patch via email. Is this the > >> correct format? > >> > >> I would appreciate it if someone could confirm, and if there is anything > >> else I need to do to get it into upcoming releases. > >> > >> Best, > >> > >> *• **Yiğit (Yeet) Uyan* > >> *•* Smart Home Intelligence | Google Home > >> *•* yigitu...@google.com > >> > >> > >> > >> On Wed, Aug 5, 2020 at 11:30 AM <device.access.t...@gmail.com> wrote: > >> > >>> From: Yiğit (Yeet) Uyan <yigitu...@google.com> > >>> > >>> Signed-off-by: Yiğit (Yeet) Uyan <yigitu...@google.com> > >>> --- > >>> libavformat/rtsp.h | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/libavformat/rtsp.h b/libavformat/rtsp.h > >>> index 54a9a30c16..2b37f5b49f 100644 > >>> --- a/libavformat/rtsp.h > >>> +++ b/libavformat/rtsp.h > >>> @@ -315,7 +315,7 @@ typedef struct RTSPState { > >>> /** some MS RTSP streams contain a URL in the SDP that we need to > use > >>> * for all subsequent RTSP requests, rather than the input URI; in > >>> * other cases, this is a copy of AVFormatContext->filename. */ > >>> - char control_uri[1024]; > >>> + char control_uri[2048]; > > > > There is a control_url field in RTSPState, and there are string copy > between them. > > My copy-paste error: RTSPState -> RTSPStream. > > > This patch makes them have different length. Please check every usecase > of the > > field. > > > >>> > >>> /** The following are used for parsing raw mpegts in udp */ > >>> //@{ > >>> > >> _______________________________________________ > >> ffmpeg-devel mailing list > >> ffmpeg-devel@ffmpeg.org > >> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > >> > >> To unsubscribe, visit link above, or email > >> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". > > > > _______________________________________________ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > To unsubscribe, visit link above, or email > > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".