Fix a signed integer overflow which could trigger a FPE_INTOVF

mm2 · mm2 · commit 20142e6a806c · 2025-12-09T16:53:58.000+01:00
diff --git a/src/cmsintrp.c b/src/cmsintrp.c
@@ -957,7 +957,7 @@ void Eval4Inputs(CMSREGISTER const cmsUInt16Number Input[],
 
         Rest = c1 * rx + c2 * ry + c3 * rz;
 
-        Tmp1[OutChan] = (cmsUInt16Number)(c0 + ROUND_FIXED_TO_INT(_cmsToFixedDomain(Rest)));
+        Tmp1[OutChan] = (cmsUInt16Number)c0 + ((Rest + (Rest >> 16)) >> 16);
     }
 
 
@@ -1021,7 +1021,7 @@ void Eval4Inputs(CMSREGISTER const cmsUInt16Number Input[],
 
         Rest = c1 * rx + c2 * ry + c3 * rz;
 
-        Tmp2[OutChan] = (cmsUInt16Number) (c0 + ROUND_FIXED_TO_INT(_cmsToFixedDomain(Rest)));
+        Tmp2[OutChan] = (cmsUInt16Number) c0 + ((Rest + (Rest >> 16)) >> 16);
     }
 
 

Original file line number	Diff line number	Diff line change
`@@ -957,7 +957,7 @@ void Eval4Inputs(CMSREGISTER const cmsUInt16Number Input[],`
`957`	`957`
`958`	`958`	`Rest = c1 * rx + c2 * ry + c3 * rz;`
`959`	`959`
`960`		`- Tmp1[OutChan] = (cmsUInt16Number)(c0 + ROUND_FIXED_TO_INT(_cmsToFixedDomain(Rest)));`
	`960`	`+ Tmp1[OutChan] = (cmsUInt16Number)c0 + ((Rest + (Rest >> 16)) >> 16);`
`961`	`961`	`}`
`962`	`962`
`963`	`963`
`@@ -1021,7 +1021,7 @@ void Eval4Inputs(CMSREGISTER const cmsUInt16Number Input[],`
`1021`	`1021`
`1022`	`1022`	`Rest = c1 * rx + c2 * ry + c3 * rz;`
`1023`	`1023`
`1024`		`- Tmp2[OutChan] = (cmsUInt16Number) (c0 + ROUND_FIXED_TO_INT(_cmsToFixedDomain(Rest)));`
	`1024`	`+ Tmp2[OutChan] = (cmsUInt16Number) c0 + ((Rest + (Rest >> 16)) >> 16);`
`1025`	`1025`	`}`
`1026`	`1026`
`1027`	`1027`