Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread Andreas Rheinhardt
James Almer: > On 5/31/2020 4:34 PM, Andreas Rheinhardt wrote: >> James Almer: >>> On 5/31/2020 4:11 PM, Andreas Rheinhardt wrote: James Almer: > On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote:>> (Anyway, when this > function is switched to size_t, >> the correct error would be

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread James Almer
On 5/31/2020 4:34 PM, Andreas Rheinhardt wrote: > James Almer: >> On 5/31/2020 4:11 PM, Andreas Rheinhardt wrote: >>> James Almer: On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote:>> (Anyway, when this function is switched to size_t, > the correct error would be AVERROR(ERANGE). It is

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread Andreas Rheinhardt
James Almer: > On 5/31/2020 4:11 PM, Andreas Rheinhardt wrote: >> James Almer: >>> On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote:>> (Anyway, when this >>> function is switched to size_t, the correct error would be AVERROR(ERANGE). It is actually already the correct error in case size >

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread James Almer
On 5/31/2020 4:11 PM, Andreas Rheinhardt wrote: > James Almer: >> On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote:>> (Anyway, when this >> function is switched to size_t, >>> the correct error would be AVERROR(ERANGE). It is actually already the >>> correct error in case size > INT_MAX -

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread Andreas Rheinhardt
James Almer: > On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote:>> (Anyway, when this function > is switched to size_t, >> the correct error would be AVERROR(ERANGE). It is actually already the >> correct error in case size > INT_MAX - AV_INPUT_BUFFER_PADDING_SIZE.) >> 3. That's unfortunately only

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread James Almer
On 5/31/2020 3:35 PM, James Almer wrote: > On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote: >> James Almer: >>> Signed-off-by: James Almer >>> --- >>> doc/APIchanges| 4 ++-- >>> libavcodec/avpacket.c | 49 +++ >>> libavcodec/packet.h | 45

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread James Almer
On 5/31/2020 2:58 PM, Andreas Rheinhardt wrote: > James Almer: >> Signed-off-by: James Almer >> --- >> doc/APIchanges| 4 ++-- >> libavcodec/avpacket.c | 49 +++ >> libavcodec/packet.h | 45 +++ >>

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread Andreas Rheinhardt
James Almer: > Signed-off-by: James Almer > --- > doc/APIchanges| 4 ++-- > libavcodec/avpacket.c | 49 +++ > libavcodec/packet.h | 45 +++ > libavutil/frame.h | 4 > 4 files changed, 100

[FFmpeg-devel] [PATCH 3/3] avcodec/packet: change public function and struct size parameter types to size_t

2020-05-31 Thread James Almer
Signed-off-by: James Almer --- doc/APIchanges| 4 ++-- libavcodec/avpacket.c | 49 +++ libavcodec/packet.h | 45 +++ libavutil/frame.h | 4 4 files changed, 100 insertions(+), 2 deletions(-) diff