On Thu, Jul 06, 2017 at 06:34:21PM +0100, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis <derek.buitenh...@gmail.com> > --- > libavformat/mpegtsenc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c > index acea2e93a6..f62ed55dea 100644 > --- a/libavformat/mpegtsenc.c > +++ b/libavformat/mpegtsenc.c > @@ -277,6 +277,8 @@ static void putstr8(uint8_t **q_ptr, const char *str, int > write_len) > len = strlen(str); > if (write_len) > *q++ = len; > + if (!str) > + return; > memcpy(q, str, len); > q += len;
> *q_ptr = q; this should not be skiped if str==NULL otherwise LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who would give up essential Liberty, to purchase a little temporary Safety, deserve neither Liberty nor Safety -- Benjamin Franklin
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel