Re: [PATCH for-4.21 0/6] tools/{lib,}xl: Coverity falllout from the json-c switch

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

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 <oleksii.kurochko@xxxxxxxxx>

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(-)
>
>