From 75c22ca591142d1f9ec8ab030f561dd1b97cdb0d Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Wed, 10 Mar 2021 10:46:10 +0200 Subject: [PATCH] hs-clientmanager: Pass arguments to printf() wrappers Avoid crashing on various platforms, by not setting any argument at all. Bug-AGL: SPEC-3843 Signed-off-by: Marius Vlad Change-Id: Ifb082b95fd6ebc71e863b843c9fd895f5f7efdde --- src/hs-clientmanager.cpp | 8 ++++---- src/hs-proxy.cpp | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/hs-clientmanager.cpp b/src/hs-clientmanager.cpp index 8f02953..66fd699 100644 --- a/src/hs-clientmanager.cpp +++ b/src/hs-clientmanager.cpp @@ -172,8 +172,8 @@ is_application_running(afb_req_t request, std::string id) afm_proxy.ps(request->api, &jobj); if (jobj) { - const int len = json_object_array_length(jobj); - for (int i = 0; i < len; i++) { + size_t len = json_object_array_length(jobj); + for (size_t i = 0; i < len; i++) { struct json_object *aid; struct json_object *item = json_object_array_get_idx(jobj, i); @@ -236,11 +236,11 @@ int HS_ClientManager::handleRequest(afb_req_t request, const char *verb, const c if (strcasecmp(verb, "showWindow") == 0) { ret = is_application_running(request, id); if (ret == AFB_REQ_NOT_STARTED_APPLICATION) { - AFB_INFO("%s is not running. Will attempt to start it"); + AFB_INFO("%s is not running. Will attempt to start it", appid); return ret; } } - AFB_INFO("%s found to be running. Forwarding request to the client"); + AFB_INFO("%s found to be running. Forwarding request to the client", appid); ret = ip->second->handleRequest(request, verb); } else { diff --git a/src/hs-proxy.cpp b/src/hs-proxy.cpp index f8b796c..aaa67c7 100644 --- a/src/hs-proxy.cpp +++ b/src/hs-proxy.cpp @@ -56,7 +56,7 @@ static void api_callback(void *closure, struct json_object *object, const char * /* if we have an error then we couldn't start the application so we remove it */ if (error) { - AFB_INFO("asynchronous call, removing client %s", cdata->appid); + AFB_INFO("asynchronous call, removing client %s", cdata->appid.c_str()); clientManager->removeClient(cdata->appid); } -- 2.16.6