On 10/15/25 3:40 PM, Roger Pau Monne wrote:
Hello,
The following series contains fixes for Coverity reported issues after
the switch from YAJL to json-c. Those reports are from the internal
XenServer Coverity instance, as the project one hasn't been switched to
use json-c yet, last patch in the series switches thegithub triggered
Coverity build to use json-c.
All should be considered for 4.21.
Agree, we could consider that to have that in 4.21 as it fixes some minor
problems and isn't too intrusive:
Release-Acked-by: Oleksii Kurochko<[email protected]>
Thanks.
~ Oleksii
Thanks, Roger.
Roger Pau Monne (6):
tools/{lib,}xl: fix usage of error return from
json_tokener_parse_verbose()
tools/libxl: avoid freeing stack rubble in
libxl__json_object_to_json()
tools/xl: check return of json_object_object_add()
tools/xl: check return value of printf_info_one_json() in
list_domains_details()
tools/xl: fix possible uninitialized usage in printf_info()
github/coverity: switch to building with json-c instead of yajl
.github/workflows/coverity.yml | 2 +-
tools/libs/light/libxl_json.c | 8 +++++---
tools/xl/xl_info.c | 22 +++++++++++++++-------
3 files changed, 21 insertions(+), 11 deletions(-)
