return MP_VAL if n < 0. remove check for max int value

rlm2002 · rlm2002 · commit 682901e32e7d · 2026-03-03T09:50:16.000-07:00
remove comment
diff --git a/wolfcrypt/src/hpke.c b/wolfcrypt/src/hpke.c
@@ -101,13 +101,12 @@ static int I2OSP(int n, int w, byte* out)
 {
     int i;
 
-    if (w <= 0 || w > 32) {
+    if (w <= 0 || w > 32 || n < 0) {
         return MP_VAL;
     }
 
     /* if width is less than int max check that n is less than w bytes max */
-    /* if width is greater than int max check that n is less than int max */
-    if ((w < 4 && n > ((1 << (w * 8)) - 1)) || (w >= 4 && n > 0x7fffffff)) {
+    if (w < 4 && n > ((1 << (w * 8)) - 1)) {
         return MP_VAL;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -101,13 +101,12 @@ static int I2OSP(int n, int w, byte* out)`
`101`	`101`	`{`
`102`	`102`	`int i;`
`103`	`103`
`104`		`- if (w <= 0 \|\| w > 32) {`
	`104`	`+ if (w <= 0 \|\| w > 32 \|\| n < 0) {`
`105`	`105`	`return MP_VAL;`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`/* if width is less than int max check that n is less than w bytes max */`
`109`		`- /* if width is greater than int max check that n is less than int max */`
`110`		`- if ((w < 4 && n > ((1 << (w * 8)) - 1)) \|\| (w >= 4 && n > 0x7fffffff)) {`
	`109`	`+ if (w < 4 && n > ((1 << (w * 8)) - 1)) {`
`111`	`110`	`return MP_VAL;`
`112`	`111`	`}`
`113`	`112`