diff options
| author | Thomas Jarosch <tomj@simonv.com> | 2014-12-20 13:47:28 +0100 |
|---|---|---|
| committer | Thomas Jarosch <tomj@simonv.com> | 2014-12-20 13:48:43 +0100 |
| commit | ef1497c3dfca2dacdf6f96d97009fe388f726f4b (patch) | |
| tree | e81db53ad173919edde25bbd0896ba153ad314b7 | |
| parent | c54537a6639bc9ea495fac5e9ecb3ee7a699b484 (diff) | |
| download | rockbox-ef1497c3dfca2dacdf6f96d97009fe388f726f4b.tar.gz rockbox-ef1497c3dfca2dacdf6f96d97009fe388f726f4b.zip | |
Add missing va_end() call when the log is full
No need to go out of memory, too ;)
cppcheck reported:
[rockbox/firmware/logf.c:338]: (error) va_list 'ap' was opened but not closed by va_end().
Change-Id: I00e4c04d7e3d5d1415aa5066487ce1d9209e53aa
| -rw-r--r-- | firmware/logf.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/firmware/logf.c b/firmware/logf.c index ade5458ca9..2fdbfa0f0c 100644 --- a/firmware/logf.c +++ b/firmware/logf.c | |||
| @@ -335,6 +335,7 @@ void _logdiskf(const char* file, const char level, const char *fmt, ...) | |||
| 335 | { | 335 | { |
| 336 | strcpy(&logdiskfbuffer[logdiskfindex-8], "LOGFULL"); | 336 | strcpy(&logdiskfbuffer[logdiskfindex-8], "LOGFULL"); |
| 337 | logdiskfindex=MAX_LOGDISKF_SIZE; | 337 | logdiskfindex=MAX_LOGDISKF_SIZE; |
| 338 | va_end(ap); | ||
| 338 | return; | 339 | return; |
| 339 | } | 340 | } |
| 340 | 341 | ||