On Sun, Jul 13, 2025 at 08:01:52PM -0300, James Almer wrote:
> Fixes: libavcodec/motion_est.c:94:31: runtime error: applying zero offset to
> null pointer
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> libavcodec/motion_est.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/motion_est.c b/libavcodec/motion_est.c
> index 93be712cc0..bde4591bf7 100644
> --- a/libavcodec/motion_est.c
> +++ b/libavcodec/motion_est.c
> @@ -90,12 +90,14 @@ static inline void init_ref(MotionEstContext *c, uint8_t
> *const src[3],
> ((y*c->uvstride + x)>>1),
> };
> int i;
> - for(i=0; i<3; i++){
> + for (i = 0; i < 3 && src[i]; i++) {
> c->src[0][i]= src [i] + offset[i];
> + }
> + for (i = 0; i < 3 && ref[i]; i++) {
> c->ref[0][i]= ref [i] + offset[i];
> }
> if(ref_index){
> - for(i=0; i<3; i++){
> + for (i = 0; i < 3 && ref2[i]; i++) {
> c->ref[ref_index][i]= ref2[i] + offset[i];
> }
> }I think the destination should be initialized to NULL when source is NULL. But i didnt check if it makes a difference, just from looking at this thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Concerning the gods, I have no means of knowing whether they exist or not or of what sort they may be, because of the obscurity of the subject, and the brevity of human life -- Protagoras
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".
