On 15/10/2025 2: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.
>
> 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
Other than my query on patch 1, everything else LGTM.
Reviewed-by: Andrew Cooper <[email protected]>
Thankyou for looking at these.
