Merge pull request #7221 from gebart/pr/fmt-float-negative-bugfix

sys/fmt: Fix unintentional pointer advancement for negative numbers in fmt_float
RIOT-OS · Jun 22, 2017 · 02a3c9f · 02a3c9f
2 parents 1014663 + 084db19
commit 02a3c9f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/sys/fmt/fmt.c b/sys/fmt/fmt.c
@@ -274,12 +274,11 @@ size_t fmt_float(char *out, float f, unsigned precision)
 
     uint32_t fraction = f * _tenmap[precision];
 
-    size_t res = negative;
     if (negative && out) {
         *out++ = '-';
     }
 
-    res += fmt_u32_dec(out, integer);
+    size_t res = fmt_u32_dec(out, integer);
     if (precision && fraction) {
         if (out) {
             out += res;
@@ -289,6 +288,7 @@ size_t fmt_float(char *out, float f, unsigned precision)
         }
         res += (1 + precision);
     }
+    res += negative;
 
     return res;
 }