aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLeo Izen <leo.izen@gmail.com>2024-07-02 09:24:18 -0400
committerLeo Izen <leo.izen@gmail.com>2024-07-03 10:21:06 -0400
commitc1af34c25bd48a547930094fd60e5458dd1c6d9b (patch)
tree4cb903c9fc32d289d392d2442f7abe6ce352576d
parentd241edc2b45f6865b1acea1ab54216359d68a64f (diff)
downloadffmpeg-c1af34c25bd48a547930094fd60e5458dd1c6d9b.tar.gz
avcodec/pngdec: fix mDCv typo
When mDCv support was added, there was a typo in both variable names and also the MKTAG itself, incorrectly listing it as mDVc. The tag name stands for Mastering Display Color Volume so mDCv is correct. See other files such as av1dec.c which uses mdcv. Typo originally introduced in c7a57b0f70f8d1574aa0f0dbe98db85d8ac91c76. Signed-off-by: Leo Izen <leo.izen@gmail.com> Reported-by: Ramiro Polla <ramiro.polla@gmail.com>
-rw-r--r--libavcodec/pngdec.c53
1 files changed, 27 insertions, 26 deletions
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index 1c910e6a5b..180806e5e1 100644
--- a/libavcodec/pngdec.c
+++ b/libavcodec/pngdec.c
@@ -86,11 +86,12 @@ typedef struct PNGDecContext {
int have_clli;
uint32_t clli_max;
uint32_t clli_avg;
- int have_mdvc;
- uint16_t mdvc_primaries[3][2];
- uint16_t mdvc_white_point[2];
- uint32_t mdvc_max_lum;
- uint32_t mdvc_min_lum;
+ /* Mastering Display Color Volume */
+ int have_mdcv;
+ uint16_t mdcv_primaries[3][2];
+ uint16_t mdcv_white_point[2];
+ uint32_t mdcv_max_lum;
+ uint32_t mdcv_min_lum;
enum PNGHeaderState hdr_state;
enum PNGImageState pic_state;
@@ -763,24 +764,24 @@ static int populate_avctx_color_fields(AVCodecContext *avctx, AVFrame *frame)
}
}
- if (s->have_mdvc) {
- AVMasteringDisplayMetadata *mdvc;
+ if (s->have_mdcv) {
+ AVMasteringDisplayMetadata *mdcv;
- ret = ff_decode_mastering_display_new(avctx, frame, &mdvc);
+ ret = ff_decode_mastering_display_new(avctx, frame, &mdcv);
if (ret < 0)
return ret;
- if (mdvc) {
- mdvc->has_primaries = 1;
+ if (mdcv) {
+ mdcv->has_primaries = 1;
for (int i = 0; i < 3; i++) {
- mdvc->display_primaries[i][0] = av_make_q(s->mdvc_primaries[i][0], 50000);
- mdvc->display_primaries[i][1] = av_make_q(s->mdvc_primaries[i][1], 50000);
+ mdcv->display_primaries[i][0] = av_make_q(s->mdcv_primaries[i][0], 50000);
+ mdcv->display_primaries[i][1] = av_make_q(s->mdcv_primaries[i][1], 50000);
}
- mdvc->white_point[0] = av_make_q(s->mdvc_white_point[0], 50000);
- mdvc->white_point[1] = av_make_q(s->mdvc_white_point[1], 50000);
- mdvc->has_luminance = 1;
- mdvc->max_luminance = av_make_q(s->mdvc_max_lum, 10000);
- mdvc->min_luminance = av_make_q(s->mdvc_min_lum, 10000);
+ mdcv->white_point[0] = av_make_q(s->mdcv_white_point[0], 50000);
+ mdcv->white_point[1] = av_make_q(s->mdcv_white_point[1], 50000);
+ mdcv->has_luminance = 1;
+ mdcv->max_luminance = av_make_q(s->mdcv_max_lum, 10000);
+ mdcv->min_luminance = av_make_q(s->mdcv_min_lum, 10000);
}
}
@@ -1571,20 +1572,20 @@ static int decode_frame_common(AVCodecContext *avctx, PNGDecContext *s,
s->clli_max = bytestream2_get_be32u(&gb_chunk);
s->clli_avg = bytestream2_get_be32u(&gb_chunk);
break;
- case MKTAG('m', 'D', 'V', 'c'):
+ case MKTAG('m', 'D', 'C', 'v'):
if (bytestream2_get_bytes_left(&gb_chunk) != 24) {
- av_log(avctx, AV_LOG_WARNING, "Invalid mDVc chunk size: %d\n", bytestream2_get_bytes_left(&gb_chunk));
+ av_log(avctx, AV_LOG_WARNING, "Invalid mDCv chunk size: %d\n", bytestream2_get_bytes_left(&gb_chunk));
break;
}
- s->have_mdvc = 1;
+ s->have_mdcv = 1;
for (int i = 0; i < 3; i++) {
- s->mdvc_primaries[i][0] = bytestream2_get_be16u(&gb_chunk);
- s->mdvc_primaries[i][1] = bytestream2_get_be16u(&gb_chunk);
+ s->mdcv_primaries[i][0] = bytestream2_get_be16u(&gb_chunk);
+ s->mdcv_primaries[i][1] = bytestream2_get_be16u(&gb_chunk);
}
- s->mdvc_white_point[0] = bytestream2_get_be16u(&gb_chunk);
- s->mdvc_white_point[1] = bytestream2_get_be16u(&gb_chunk);
- s->mdvc_max_lum = bytestream2_get_be32u(&gb_chunk);
- s->mdvc_min_lum = bytestream2_get_be32u(&gb_chunk);
+ s->mdcv_white_point[0] = bytestream2_get_be16u(&gb_chunk);
+ s->mdcv_white_point[1] = bytestream2_get_be16u(&gb_chunk);
+ s->mdcv_max_lum = bytestream2_get_be32u(&gb_chunk);
+ s->mdcv_min_lum = bytestream2_get_be32u(&gb_chunk);
break;
case MKTAG('I', 'E', 'N', 'D'):
if (!(s->pic_state & PNG_ALLIMAGE))