CC-tea/coreutils
1
0
Fork 0
mirror of git://git.sv.gnu.org/coreutils.git synced 2026-05-28 13:18:08 +02:00
Code Issues Projects Releases Wiki Activity

From GNU libc.

Browse Source 
(my_strftime): Handle very large width
specifications for numeric values correctly.  Improve checks for
overflow.
This commit is contained in:
Jim Meyering
2003-01-20 15:04:51 +00:00
parent 3fa2b8a8eb
commit 01dfd68984
1 changed files with 42 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/strftime.c
+42 -16
View File
@@ -265,7 +265,7 @@ static const CHAR_T zeroes[16] = /* "0000000000000000" */
int _n = (n); \
int _delta = width - _n; \
int _incr = _n + (_delta > 0 ? _delta : 0); \
if (i + _incr >= maxsize) \
if ((size_t) _incr >= maxsize - i) \
return 0; \
if (p) \
{ \
@@ -748,8 +748,15 @@ my_strftime (s, maxsize, format, tp extra_args LOCALE_PARAM)
width = 0;
do
{
width *= 10;
width += *f - L_('0');
if (width > INT_MAX / 10
|| (width == INT_MAX / 10 && *f - L_('0') > INT_MAX % 10))
/* Avoid overflow. */
width = INT_MAX;
else
{
width *= 10;
width += *f - L_('0');
}
++f;
}
while (ISDIGIT (*f));
@@ -773,10 +780,10 @@ my_strftime (s, maxsize, format, tp extra_args LOCALE_PARAM)
switch (format_char)
{
#define DO_NUMBER(d, v) \
digits = width == -1 ? d : width; \
digits = d > width ? d : width; \
number_value = v; goto do_number
#define DO_NUMBER_SPACEPAD(d, v) \
digits = width == -1 ? d : width; \
digits = d > width ? d : width; \
number_value = v; goto do_number_spacepad
case L_('%'):
@@ -1033,18 +1040,37 @@ my_strftime (s, maxsize, format, tp extra_args LOCALE_PARAM)
int padding = digits - (buf + (sizeof (buf) / sizeof (buf[0]))
- bufp);
if (pad == L_('_'))
if (padding > 0)
{
while (0 < padding--)
*--bufp = L_(' ');
}
else
{
bufp += negative_number;
while (0 < padding--)
*--bufp = L_('0');
if (negative_number)
*--bufp = L_('-');
if (pad == L_('_'))
{
if ((size_t) padding >= maxsize - i)
return 0;
if (p)
memset_space (p, padding);
i += padding;
width = width > padding ? width - padding : 0;
}
else
{
if ((size_t) digits >= maxsize - i)
return 0;
if (negative_number)
{
++bufp;
if (p)
*p++ = L_('-');
++i;
}
if (p)
memset_zero (p, padding);
i += padding;
width = 0;
}
}
}