gallant/SDL2
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Fixed bug 2050 - Obvious bugs in SDL_ltoa and SDL_lltoa

Browse source 
pjz

SDL_ltoa(-2147483648,s,10) only returns "-" because there is a bug in the code:

    if ( value < 0 ) {
        *bufp++ = '-';
        value = -value;
    }

but -(-2147483648) is still -2147483648 (0x80000000) as signed int (or long), so the following loop doesn't run at all. Similar bug are also in SDL_lltoa.

BTW, there is no sanity check for radix.
This commit is contained in:
Sam Lantinga 2013-12-11 21:17:24 -08:00
parent 4b72d4b2b8
commit cf053a74d6
1 changed files with 4 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
src/stdlib/SDL_string.c
View file

@ -665,23 +665,9 @@ SDL_ltoa(long value, char *string, int radix)
if (value < 0) {
*bufp++ = '-';
value = -value;
}
if (value) {
while (value > 0) {
*bufp++ = ntoa_table[value % radix];
value /= radix;
}
SDL_ultoa(-value, bufp, radix);
} else {
*bufp++ = '0';
}
*bufp = '\0';
/* The numbers went into the string backwards. :) */
if (*string == '-') {
SDL_strrev(string + 1);
} else {
SDL_strrev(string);
SDL_ultoa(value, bufp, radix);
}
return string;
@ -723,23 +709,9 @@ SDL_lltoa(Sint64 value, char *string, int radix)
if (value < 0) {
*bufp++ = '-';
value = -value;
}
if (value) {
while (value > 0) {
*bufp++ = ntoa_table[value % radix];
value /= radix;
}
SDL_ulltoa(-value, bufp, radix);
} else {
*bufp++ = '0';
}
*bufp = '\0';
/* The numbers went into the string backwards. :) */
if (*string == '-') {
SDL_strrev(string + 1);
} else {
SDL_strrev(string);
SDL_ulltoa(value, bufp, radix);
}
return string;