Hi, We observed NULL pointer during HLS/DASH streaming when we simulated network congestion and disconnects with persistent HTTP connection. That is why added this check to handle segmentation fault.
On 2/7/18, 4:52 PM, "wm4" <nfx...@googlemail.com> wrote: On Wed, 7 Feb 2018 08:57:26 +0530 rpata...@akamai.com wrote: > From: Ravindra <rpata...@akamai.com> > > Signed-off-by: Ravindra <rpata...@akamai.com> > --- > libavformat/avio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/avio.c b/libavformat/avio.c > index 63e8287..18e58ae 100644 > --- a/libavformat/avio.c > +++ b/libavformat/avio.c > @@ -420,7 +420,7 @@ int ffurl_read_complete(URLContext *h, unsigned char *buf, int size) > > int ffurl_write(URLContext *h, const unsigned char *buf, int size) > { > - if (!(h->flags & AVIO_FLAG_WRITE)) > + if (!h || !(h->flags & AVIO_FLAG_WRITE)) > return AVERROR(EIO); > /* avoid sending too big packets */ > if (h->max_packet_size && size > h->max_packet_size) Why would this ever be NULL? It looks like a caller error to me, not something the function should handle. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel