From da2824b15151f3fcecb43a42db08d1875bd30558 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jos=C3=A9=20Bollo?= Date: Wed, 27 Jan 2016 12:10:51 +0100 Subject: [PATCH] afm-user-daemon: adds verbose info MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Change-Id: I9195b2b10124b573be90ed4b754f7fe0ae5595ac Signed-off-by: José Bollo --- src/CMakeLists.txt | 2 +- src/afm-user-daemon.c | 51 ++++++++++++++++++++++++++++++++++++--------------- src/verbose.c | 2 +- src/verbose.h | 4 ++-- 4 files changed, 40 insertions(+), 19 deletions(-) diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index fb55e60..1c64ad2 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -55,7 +55,7 @@ endif(USE_SIMULATION) ########################################################################### -add_compile_options(-Wall -Wno-pointer-sign) +add_compile_options(-Wall -Wno-pointer-sign -Werror=maybe-uninitialized) add_compile_options(-ffunction-sections -fdata-sections) add_compile_options(-fPIC) add_compile_options(-Wl,--gc-sections) diff --git a/src/afm-user-daemon.c b/src/afm-user-daemon.c index ff58e03..8f8f381 100644 --- a/src/afm-user-daemon.c +++ b/src/afm-user-daemon.c @@ -93,15 +93,20 @@ static void reply_status(struct jreq *jreq, int status) static void on_runnables(struct jreq *jreq, struct json_object *obj) { - struct json_object *resp = afm_db_application_list(afdb); + struct json_object *resp; + INFO("method runnables called"); + resp = afm_db_application_list(afdb); jbus_reply_j(jreq, resp); json_object_put(resp); } static void on_detail(struct jreq *jreq, struct json_object *obj) { - const char *appid = getappid(obj); - struct json_object *resp = afm_db_get_application_public(afdb, appid); + const char *appid; + struct json_object *resp; + appid = getappid(obj); + INFO("method detail called for %s", appid); + resp = afm_db_get_application_public(afdb, appid); reply(jreq, resp, error_not_found); json_object_put(resp); } @@ -114,6 +119,7 @@ static void on_start(struct jreq *jreq, struct json_object *obj) char runidstr[20]; appid = getappid(obj); + INFO("method start called for %s", appid); if (appid == NULL) jbus_reply_error_s(jreq, error_bad_request); else { @@ -135,45 +141,60 @@ static void on_start(struct jreq *jreq, struct json_object *obj) static void on_stop(struct jreq *jreq, struct json_object *obj) { - int runid = getrunid(obj); - int status = afm_run_stop(runid); + int runid, status; + runid = getrunid(obj); + INFO("method stop called for %d", runid); + status = afm_run_stop(runid); reply_status(jreq, status); } static void on_continue(struct jreq *jreq, struct json_object *obj) { - int runid = getrunid(obj); - int status = afm_run_continue(runid); + int runid, status; + runid = getrunid(obj); + INFO("method continue called for %d", runid); + status = afm_run_continue(runid); reply_status(jreq, status); } static void on_terminate(struct jreq *jreq, struct json_object *obj) { - int runid = getrunid(obj); - int status = afm_run_terminate(runid); + int runid, status; + runid = getrunid(obj); + INFO("method terminate called for %d", runid); + status = afm_run_terminate(runid); reply_status(jreq, status); } static void on_runners(struct jreq *jreq, struct json_object *obj) { - struct json_object *resp = afm_run_list(); + struct json_object *resp; + INFO("method runners called"); + resp = afm_run_list(); jbus_reply_j(jreq, resp); json_object_put(resp); } static void on_state(struct jreq *jreq, struct json_object *obj) { - int runid = getrunid(obj); - struct json_object *resp = afm_run_state(runid); + int runid; + struct json_object *resp; + runid = getrunid(obj); + INFO("method state called for %d", runid); + resp = afm_run_state(runid); reply(jreq, resp, error_not_found); json_object_put(resp); } static void propagate(struct jreq *jreq, const char *msg, const char *method) { - char *reply = jbus_call_ss_sync(jbuses[0], method, msg); - if (reply) + char *reply; + INFO("method %s propagated with %s", method, msg); + reply = jbus_call_ss_sync(jbuses[0], method, msg); + if (reply) { jbus_reply_s(jreq, reply); + free(reply); + } else jbus_reply_error_s(jreq, error_system); } @@ -192,7 +213,7 @@ static void on_signal_changed(struct json_object *obj) { /* update the database */ afm_db_update_applications(afdb); - /* propagate now */ + /* re-propagate now */ jbus_send_signal_j(jbuses[1], "changed", obj); } diff --git a/src/verbose.c b/src/verbose.c index 36d2348..7e46a33 100644 --- a/src/verbose.c +++ b/src/verbose.c @@ -23,7 +23,7 @@ int verbosity = 1; #else void verbose_error(const char *file, int line) { - ERROR("error file %s line %d", file, line); + syslog(LOG_ERR, "error file %s line %d", file, line); } #endif diff --git a/src/verbose.h b/src/verbose.h index cf87ba0..7b32a66 100644 --- a/src/verbose.h +++ b/src/verbose.h @@ -24,8 +24,8 @@ extern int verbosity; #define ERROR(...) syslog(LOG_ERR,__VA_ARGS__) #define WARNING(...) do{if(verbosity)syslog(LOG_WARNING,__VA_ARGS__);}while(0) #define NOTICE(...) do{if(verbosity)syslog(LOG_NOTICE,__VA_ARGS__);}while(0) -#define INFO(...) do{if(verbosity)syslog(LOG_INFO,__VA_ARGS__);}while(0) -#define DEBUG(...) do{if(verbosity>1)syslog(LOG_DEBUG,__VA_ARGS__);}while(0) +#define INFO(...) do{if(verbosity>1)syslog(LOG_INFO,__VA_ARGS__);}while(0) +#define DEBUG(...) do{if(verbosity>2)syslog(LOG_DEBUG,__VA_ARGS__);}while(0) #else #include #define LOGUSER(app) openlog(app,LOG_PERROR,LOG_USER) -- 2.16.6