diff options
| author | Assaf Gordon <assafgordon@gmail.com> | 2015-06-29 12:54:41 +0100 |
|---|---|---|
| committer | Pádraig Brady <P@draigBrady.com> | 2015-06-29 12:59:19 +0100 |
| commit | badecb3f6ce21e9ad12402d4dd96d9a532499710 (patch) | |
| tree | abb5ae6edadc36497031e788f2167c47b872f641 /src | |
| parent | eff511bd3cecaab30df75a854f01566ce45c5876 (diff) | |
| download | coreutils-badecb3f6ce21e9ad12402d4dd96d9a532499710.tar.xz | |
numfmt: fix printf argument order
* src/numfmt.c (double_to_human): Fix the argument order
passed to snprintf, which happened to work on amd64 with
its separate va_arg storage area for floats¹,
but would fail tests for example on i686.
¹ https://blog.nelhage.com/2010/10/amd64-and-va_arg/
Diffstat (limited to 'src')
| -rw-r--r-- | src/numfmt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/numfmt.c b/src/numfmt.c index 4af16faea..09cdd422f 100644 --- a/src/numfmt.c +++ b/src/numfmt.c @@ -794,7 +794,7 @@ double_to_human (long double val, int precision, int prec = user_precision == -1 ? show_decimal_point : user_precision; /* buf_size - 1 used here to ensure place for possible scale_IEC_I suffix. */ - num_size = snprintf (buf, buf_size - 1, fmt, val, prec, + num_size = snprintf (buf, buf_size - 1, fmt, prec, val, suffix_power_char (power)); if (num_size < 0 || num_size >= (int) buf_size - 1) error (EXIT_FAILURE, 0, |