Rewrite the getn function (again) to fix any remaining issues.
This commit is contained in:
parent
9d2fefeac4
commit
68c4dae6ef
41
misc.c
41
misc.c
@ -30,7 +30,7 @@
|
||||
#include <grp.h>
|
||||
#endif
|
||||
|
||||
__RCSID("$MirOS: src/bin/mksh/misc.c,v 1.201 2012/11/30 17:34:46 tg Exp $");
|
||||
__RCSID("$MirOS: src/bin/mksh/misc.c,v 1.202 2012/12/04 01:12:11 tg Exp $");
|
||||
|
||||
#define KSH_CHVT_FLAG
|
||||
#ifdef MKSH_SMALL
|
||||
@ -469,33 +469,40 @@ parse_args(const char **argv,
|
||||
int
|
||||
getn(const char *s, int *ai)
|
||||
{
|
||||
int c;
|
||||
unsigned int i, j, k;
|
||||
char c;
|
||||
unsigned int i = 0;
|
||||
bool neg = false;
|
||||
int rv = 0;
|
||||
|
||||
do {
|
||||
c = *s++;
|
||||
} while (ksh_isspace(c));
|
||||
if (c == '-') {
|
||||
|
||||
switch (c) {
|
||||
case '-':
|
||||
neg = true;
|
||||
/* FALLTHROUGH */
|
||||
case '+':
|
||||
c = *s++;
|
||||
} else if (c == '+')
|
||||
c = *s++;
|
||||
k = neg ? 2147483648U : 2147483647U;
|
||||
j = i = 0;
|
||||
break;
|
||||
}
|
||||
|
||||
do {
|
||||
if (!ksh_isdigit(c))
|
||||
goto getn_out;
|
||||
if ((j = i * 10 + c - '0') > k)
|
||||
goto getn_out;
|
||||
i = j;
|
||||
/* not numeric */
|
||||
return (0);
|
||||
if (i > 214748364U)
|
||||
/* overflow on multiplication */
|
||||
return (0);
|
||||
i = i * 10U + (unsigned int)(c - '0');
|
||||
/* now: i <= 2147483649U */
|
||||
} while ((c = *s++));
|
||||
rv = 1;
|
||||
|
||||
getn_out:
|
||||
*ai = i == 2147483648U ? (int)i : neg ? -(int)i : (int)i;
|
||||
return (rv);
|
||||
if (i > (neg ? 2147483648U : 2147483647U))
|
||||
/* overflow for signed 32-bit int */
|
||||
return (0);
|
||||
|
||||
*ai = neg ? -(int)i : (int)i;
|
||||
return (1);
|
||||
}
|
||||
|
||||
/**
|
||||
|
Loading…
x
Reference in New Issue
Block a user