diff options
| author | Simon Glass <sjg@chromium.org> | 2016-06-08 20:55:15 -0600 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2016-06-09 13:52:53 -0400 |
| commit | 3191d8408053674c1b9bb79a82e3973add48830c (patch) | |
| tree | 4ee4b34a53ec766e18389fc6d6439ba2690f156c /lib | |
| parent | 9c2f9b2da650907b928995350cc4e29480fb0f80 (diff) | |
tiny-printf: Correct the snprintf() implementation
This current code passes the variable arguments list to sprintf(). This is
not correct. Fix it by calling _vprintf() directly.
This makes firefly-rk3288 boot again.
Fixes: abeb272 ("tiny-printf: Support snprintf()")
Reviewed-by: Stefan Roese <sr@denx.de>
Acked-by: Marek Vasut <marex@denx.de>
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/tiny-printf.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/tiny-printf.c b/lib/tiny-printf.c index 5ea2555280b..3c65fc90bf2 100644 --- a/lib/tiny-printf.c +++ b/lib/tiny-printf.c @@ -168,8 +168,10 @@ int snprintf(char *buf, size_t size, const char *fmt, ...) int ret; va_start(va, fmt); - ret = sprintf(buf, fmt, va); + outstr = buf; + ret = _vprintf(fmt, va, putc_outstr); va_end(va); + *outstr = '\0'; return ret; } |