On Fri, Sep 06, 2019 at 11:28:28PM +0800, [email protected] wrote: > From: Limin Wang <[email protected]> > > Signed-off-by: Limin Wang <[email protected]> > --- > libavcodec/v210dec.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c > index 5a33d8c..6ce18aa 100644 > --- a/libavcodec/v210dec.c > +++ b/libavcodec/v210dec.c > @@ -98,8 +98,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, > int *got_frame, > return AVERROR_INVALIDDATA; > } > } > - if ( avctx->codec_tag == MKTAG('C', '2', '1', '0') > - && avpkt->size > 64 > + if (avctx->codec_tag == MKTAG('C', '2', '1', '0') > && AV_RN32(psrc) == AV_RN32("INFO") > && avpkt->size - 64 >= stride * avctx->height) > psrc += 64;
Iam undecided on this, the change is correct but iam not sure it makes the code easier to understand also it makes the code less robust. For example the check is just unneeded as long as the types of the variables arent changed thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I know you won't believe me, but the highest form of Human Excellence is to question oneself and others. -- Socrates
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
