diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2023-06-12 14:53:30 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2023-06-13 09:01:17 +0200 |
commit | bd72a767b1b37e91b1c1cd9a65bae9fdb76d2f70 (patch) | |
tree | 9a3ddf37e44f4c0eee4c82b97af643a021f2a2ee /common/memory_display.c | |
parent | d7fb894dda814fa04bc48c1c4300bc0ef0985acd (diff) | |
download | barebox-bd72a767b1b37e91b1c1cd9a65bae9fdb76d2f70.tar.gz barebox-bd72a767b1b37e91b1c1cd9a65bae9fdb76d2f70.tar.xz |
memory_display: don't skip va_end in error case
There's a va_start at the start of __pr_memory_display, but va_end was
only called in the successful case. Fix this.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20230612125331.1085059-5-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'common/memory_display.c')
-rw-r--r-- | common/memory_display.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/memory_display.c b/common/memory_display.c index 77868be26a..c0ca469703 100644 --- a/common/memory_display.c +++ b/common/memory_display.c @@ -123,9 +123,9 @@ int __pr_memory_display(int level, const void *addr, loff_t offs, unsigned nbyte } while (nbytes > 0); - va_end(args); ret = 0; out: + va_end(args); return ret; } |