diff options
| author | Niklas Haas <git@haasn.dev> | 2022-11-21 17:38:25 +0100 |
|---|---|---|
| committer | sfan5 <sfan5@live.de> | 2023-01-24 15:56:56 +0100 |
| commit | 954d53612c541d6cc22fedf43fa5c8d119847e78 (patch) | |
| tree | 21b9627432f14eb1ca636e6c08214365313d4490 | |
| parent | 6860343a251c05e9faf20987b1a0cee540c32c5e (diff) | |
| download | mpv-954d53612c541d6cc22fedf43fa5c8d119847e78.tar.bz2 mpv-954d53612c541d6cc22fedf43fa5c8d119847e78.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.
| -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 |