| Message ID | 20200227053351.26785-1-andreas.rheinhardt@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [FFmpeg-devel,1/2] avformat/ivfenc: Don't use size_t for size of file | expand |
| Context | Check | Description |
|---|---|---|
| andriy/ffmpeg-patchwork | success | Make fate finished |
lgtm On 2/27/20, Andreas Rheinhardt <andreas.rheinhardt@gmail.com> wrote: > Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> > --- > libavformat/ivfenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c > index eb70421c44..45e5b238dc 100644 > --- a/libavformat/ivfenc.c > +++ b/libavformat/ivfenc.c > @@ -80,7 +80,7 @@ static int ivf_write_trailer(AVFormatContext *s) > IVFEncContext *ctx = s->priv_data; > > if ((pb->seekable & AVIO_SEEKABLE_NORMAL) && ctx->frame_cnt > 1) { > - size_t end = avio_tell(pb); > + int64_t end = avio_tell(pb); > > avio_seek(pb, 24, SEEK_SET); > // overwrite the "length" field (duration) > -- > 2.20.1 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
On Thu, Feb 27, 2020 at 09:44:05AM +0100, Paul B Mahol wrote:
> lgtm
will apply
thx
[...]
diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c index eb70421c44..45e5b238dc 100644 --- a/libavformat/ivfenc.c +++ b/libavformat/ivfenc.c @@ -80,7 +80,7 @@ static int ivf_write_trailer(AVFormatContext *s) IVFEncContext *ctx = s->priv_data; if ((pb->seekable & AVIO_SEEKABLE_NORMAL) && ctx->frame_cnt > 1) { - size_t end = avio_tell(pb); + int64_t end = avio_tell(pb); avio_seek(pb, 24, SEEK_SET); // overwrite the "length" field (duration)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> --- libavformat/ivfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)