avcodec/imgconvert: fix possible null pointer dereference
authorSimon Thelen <ffmpeg-dev@c-14.de>
Tue, 3 Apr 2018 12:41:33 +0000 (14:41 +0200)
committerPaul B Mahol <onemda@gmail.com>
Tue, 3 Apr 2018 14:20:31 +0000 (16:20 +0200)
regression since 354b26a3945eadd4ed8fcd801dfefad2566241de

libavcodec/imgconvert.c

index 7b0005b..1fd636c 100644 (file)
@@ -72,11 +72,12 @@ enum AVPixelFormat avcodec_find_best_pix_fmt_of_list(const enum AVPixelFormat *p
     int loss;
 
     for (i=0; pix_fmt_list[i] != AV_PIX_FMT_NONE; i++) {
-        loss = *loss_ptr;
+        loss = loss_ptr ? *loss_ptr : 0;
         best = avcodec_find_best_pix_fmt_of_2(best, pix_fmt_list[i], src_pix_fmt, has_alpha, &loss);
     }
 
-    *loss_ptr = loss;
+    if (loss_ptr)
+        *loss_ptr = loss;
     return best;
 }