> On Sep 27, 2020, at 6:26 AM, Andriy Gelman <[email protected]> wrote: > > From: Andriy Gelman <[email protected]> > > There is one general rtsp connection plus two connections per stream > (rtp/rtcp). > > Signed-off-by: Andriy Gelman <[email protected]> > --- > libavformat/rtsp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c > index 5d8491b74b..90f912feb9 100644 > --- a/libavformat/rtsp.c > +++ b/libavformat/rtsp.c > @@ -1994,7 +1994,7 @@ static int udp_read_packet(AVFormatContext *s, > RTSPStream **prtsp_st, > int *fds = NULL, fdsnum, fdsidx; > > if (!p) { > - p = rt->p = av_malloc_array(2 * (rt->nb_rtsp_streams + 1), > sizeof(struct pollfd)); > + p = rt->p = av_malloc_array(2 * rt->nb_rtsp_streams + 1, > sizeof(struct pollfd));
LGTM. I’m not sure is it appropriate to modify sizeof(struct pollfd) to sizeof(*p) in this patch. > if (!p) > return AVERROR(ENOMEM); > > -- > 2.27.0 > > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
