iolog: add va_end on error
authorVincent Fu <vincent.fu@samsung.com>
Thu, 5 Sep 2024 18:35:30 +0000 (14:35 -0400)
committerVincent Fu <vincent.fu@samsung.com>
Thu, 5 Sep 2024 18:35:30 +0000 (14:35 -0400)
commit65098b11f8aa28e328c225dcd39e3338bb005fdc
treea2a0ff9cf136450d36726cd68a32cff016fda1b2
parent20d0ba84d3b7e7abd8a3318175b46f5e5363547d
iolog: add va_end on error

Call va_end when we encounter an error trying to print sample fields.

This was reported by Coverity:

** CID 509197:  API usage errors  (VARARGS)
/iolog.c: 1025 in print_sample_fields()
________________________________________________________________________________________________________
*** CID 509197:  API usage errors  (VARARGS)
/iolog.c: 1025 in print_sample_fields()
1019      int ret;
1020
1021      va_start(ap, fmt);
1022      ret = vsnprintf(*p, *left, fmt, ap);
1023      if (ret < 0 || ret >= *left) {
1024      log_err("sample file write failed: %d\n", ret);
>>>     CID 509197:  API usage errors  (VARARGS)
>>>     "va_end" was not called for "ap".
1025      return -1;
1026      }
1027      va_end(ap);
1028
1029      *p += ret;
1030      *left -= ret;

Fixes: 3ec6b6da ("iolog: refactor flush_samples()")
Signed-off-by: Vincent Fu <vincent.fu@samsung.com>
iolog.c