diff options
| author | Rich Felker <dalias@aerifal.cx> | 2014-04-07 13:50:05 -0400 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2014-04-07 13:50:05 -0400 |
| commit | e94d0692864ecf9522fd6a97610a47a2f718d3de (patch) | |
| tree | 9270fe106eae71c137ab703d3209ce9c1b466ed8 /src/stdio/vfprintf.c | |
| parent | efe07b0f8914eccf5eabce0dbd6ea5da7d7db38c (diff) | |
| download | musl-e94d0692864ecf9522fd6a97610a47a2f718d3de.tar.gz musl-e94d0692864ecf9522fd6a97610a47a2f718d3de.tar.xz musl-e94d0692864ecf9522fd6a97610a47a2f718d3de.zip | |
fix printf rounding with %g for some corner case midpoints
the subsequent rounding code assumes the end pointer (z) accurately reflects the end of significance in the decimal expansion, but for certain large integers, spurious trailing zero slots were left behind when applying the binary exponent. issue reported by Morten Welinder; the analysis of the cause was performed by nsz, who also proposed this change.
Diffstat (limited to 'src/stdio/vfprintf.c')
| -rw-r--r-- | src/stdio/vfprintf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/stdio/vfprintf.c b/src/stdio/vfprintf.c index 0be75498..f6e7f38d 100644 --- a/src/stdio/vfprintf.c +++ b/src/stdio/vfprintf.c @@ -308,8 +308,8 @@ static int fmt_fp(FILE *f, long double y, int w, int p, int fl, int t) *d = x % 1000000000; carry = x / 1000000000; } - if (!z[-1] && z>a) z--; if (carry) *--a = carry; + while (z>a && !z[-1]) z--; e2-=sh; } while (e2<0) { |