[PATCH for-4.21 4/6] tools/xl: check return value of printf_info_one_json() in list_domains_details()

Roger Pau Monne posted 6 patches 2 weeks, 1 day ago
[PATCH for-4.21 4/6] tools/xl: check return value of printf_info_one_json() in list_domains_details()
Posted by Roger Pau Monne 2 weeks, 1 day ago
And return from the function early if printf_info_one_json() returns error.

Reported by internal XenServer Coverity instance.

Fixes: f6c6f2679d49 ("libxl: Convert libxl__object_to_json() to json-c")
Signed-off-by: Roger Pau Monné <roger.pua@citrix.com>
---
 tools/xl/xl_info.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/xl/xl_info.c b/tools/xl/xl_info.c
index 2048669abbd4..696f1f7423cb 100644
--- a/tools/xl/xl_info.c
+++ b/tools/xl/xl_info.c
@@ -585,6 +585,8 @@ static void list_domains_details(const libxl_dominfo *info, int nb_domain)
 #ifdef HAVE_LIBJSONC
             json_object *jso_value;
             rc = printf_info_one_json(&jso_value, info[i].domid, &d_config);
+            if (rc)
+                goto out;
             json_object_array_add(jso, jso_value);
 #elif defined(HAVE_LIBYAJL)
             s = printf_info_one_json(hand, info[i].domid, &d_config);
-- 
2.51.0


Re: [PATCH for-4.21 4/6] tools/xl: check return value of printf_info_one_json() in list_domains_details()
Posted by Jason Andryuk 2 weeks, 1 day ago
On 2025-10-15 09:40, Roger Pau Monne wrote:
> And return from the function early if printf_info_one_json() returns error.
> 
> Reported by internal XenServer Coverity instance.
> 
> Fixes: f6c6f2679d49 ("libxl: Convert libxl__object_to_json() to json-c")
> Signed-off-by: Roger Pau Monné <roger.pua@citrix.com>

Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>