aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Thelen <ffmpeg-dev@c-14.de>2018-04-03 14:41:33 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2018-07-16 19:16:13 +0200
commit28e0e494ee6499e7c37255e54453d791ff79ae54 (patch)
tree415911e6fd8565b0c15b690cb036869d47c2fe99
parentc8ca54961dcc49ac7eb6353b6c007be3a0dfe1cc (diff)
downloadffmpeg-28e0e494ee6499e7c37255e54453d791ff79ae54.tar.gz
avcodec/imgconvert: fix possible null pointer dereference
regression since 354b26a3945eadd4ed8fcd801dfefad2566241de (cherry picked from commit 8c2c97403baf95d0facb53f03e468f023eb943e1) (cherry picked from commit c1e172c2e14ef059dac632f7c67f081dfecd30dc) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/imgconvert.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c
index 7b0005b308..1fd636c83d 100644
--- a/libavcodec/imgconvert.c
+++ b/libavcodec/imgconvert.c
@@ -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;
}