gamut_mapping: safety clamp on desat_bounded()

Prevents infinite loop here.
haasn · Aug 2, 2023 · d56e295 · d56e295
1 parent 9affb27
commit d56e295
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/gamut_mapping.c b/src/gamut_mapping.c
@@ -470,6 +470,11 @@ static const float maxDelta = 5e-5f;
 static inline struct ICh
 desat_bounded(float I, float h, float Cmin, float Cmax, struct gamut gamut)
 {
+    if (I <= gamut.min_luma)
+        return (struct ICh) { .I = gamut.min_luma, .C = 0, .h = h };
+    if (I >= gamut.max_luma)
+        return (struct ICh) { .I = gamut.max_luma, .C = 0, .h = h };
+
     const float maxDI = I * maxDelta;
     struct ICh res = { .I = I, .C = (Cmin + Cmax) / 2, .h = h };
     do {