summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorAssaf Gordon <assafgordon@gmail.com>2015-06-29 12:54:41 +0100
committerPádraig Brady <P@draigBrady.com>2015-06-29 12:59:19 +0100
commitbadecb3f6ce21e9ad12402d4dd96d9a532499710 (patch)
treeabb5ae6edadc36497031e788f2167c47b872f641 /src
parenteff511bd3cecaab30df75a854f01566ce45c5876 (diff)
downloadcoreutils-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.c2
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,