diff options
author | Niklas Haas <git@haasn.dev> | 2022-11-21 17:38:25 +0100 |
---|---|---|
committer | Niklas Haas <github-daiK1o@haasn.dev> | 2022-11-21 17:39:37 +0100 |
commit | e1a04cd8ac407cd188a9e6064677d232ae409e9c (patch) | |
tree | 5893268d86573278c10a61337ba0f8fb10f5a2c2 /video/out/gpu/lcms.c | |
parent | b9b3342369c3e58f9fbb4c78dedbf3418a1ebd59 (diff) | |
download | mpv-e1a04cd8ac407cd188a9e6064677d232ae409e9c.tar.bz2 mpv-e1a04cd8ac407cd188a9e6064677d232ae409e9c.tar.xz |
lcms: fix validate_3dlut_size_opt
Not only was this function needlessly convoluted, it was also broken -
since it returned a bool value where an error code was expected.
Diffstat (limited to 'video/out/gpu/lcms.c')
-rw-r--r-- | video/out/gpu/lcms.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/video/out/gpu/lcms.c b/video/out/gpu/lcms.c index 402b612a79..b49b9614a2 100644 --- a/video/out/gpu/lcms.c +++ b/video/out/gpu/lcms.c @@ -486,11 +486,8 @@ bool gl_lcms_get_lut3d(struct gl_lcms *p, struct lut3d **result_lut3d, static int validate_3dlut_size_opt(struct mp_log *log, const m_option_t *opt, struct bstr name, const char **value) { - struct bstr param = bstr0(*value); int p1, p2, p3; - char s[20]; - snprintf(s, sizeof(s), "%.*s", BSTR_P(param)); - return gl_parse_3dlut_size(s, &p1, &p2, &p3); + return gl_parse_3dlut_size(*value, &p1, &p2, &p3) ? 0 : M_OPT_INVALID; } #define OPT_BASE_STRUCT struct mp_icc_opts |