On Fri, Aug 8, 2014 at 6:35 PM, Michael Niedermayer <michae...@gmx.at> wrote:
> On Fri, Aug 08, 2014 at 10:37:06AM -0700, Mark Reid wrote: > > --- > > libavformat/movenc.c | 25 +++++++++++++++++++++++-- > > 1 file changed, 23 insertions(+), 2 deletions(-) > > > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > > index 86dbe7f..fe4413c 100644 > > --- a/libavformat/movenc.c > > +++ b/libavformat/movenc.c > > @@ -1340,6 +1340,16 @@ static int mov_write_rtp_tag(AVIOContext *pb, > MOVTrack *track) > > return update_size(pb, pos); > > } > > > > +static int mov_write_source_reference_tag(AVIOContext *pb, MOVTrack > *track, const char *reel_name){ > > + int64_t pos = avio_tell(pb); > > + avio_wb32(pb, 0); /* size */ > > + ffio_wfourcc(pb, "name"); /* Data format */ > > > + avio_wb16(pb, strlen(reel_name)); /* string size */ > > forgot but strictly speaking this requires a check on the strlen > fitting in 16bit > > can you send a patch added a check with appropriate action ? > okay, I'll send a patch that adds a check and fixes this. > > thanks > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Many that live deserve death. And some that die deserve life. Can you give > it to them? Then do not be too eager to deal out death in judgement. For > even the very wise cannot see all ends. -- Gandalf > > _______________________________________________ > 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