On Mon, Dec 26, 2022 at 03:48:41PM +0100, Andreas Rheinhardt wrote: > Michael Niedermayer: > > Fixes: signed integer overflow: -2147483648 - 5 cannot be represented in > > type 'int' > > Fixes: > > 54242/clusterfuzz-testcase-minimized-ffmpeg_BSF_DTS2PTS_fuzzer-472928339243827 > > > > Found-by: continuous fuzzing process > > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > > --- > > libavcodec/dts2pts_bsf.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/dts2pts_bsf.c b/libavcodec/dts2pts_bsf.c > > index 48612e59db..60bec91f99 100644 > > --- a/libavcodec/dts2pts_bsf.c > > +++ b/libavcodec/dts2pts_bsf.c > > @@ -90,9 +90,11 @@ static int cmp_insert(const void *key, const void *node) > > > > static int cmp_find(const void *key, const void *node) > > { > > - int ret = ((const DTS2PTSFrame *)key)->poc - ((const DTS2PTSNode *) > > node)->poc; > > + const DTS2PTSFrame * key1 = key; > > + const DTS2PTSNode *node1 = node; > > + int ret = (key1->poc > node1->poc) - (key1->poc < node1->poc) ; > > Don't we have FFDIFFSIGN (or so) for this?
yes, i must have forgotten will apply with FFDIFFSIGN() thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB When you are offended at any man's fault, turn to yourself and study your own failings. Then you will forget your anger. -- Epictetus
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
