aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Bollo <jose.bollo@iot.bzh>2016-01-27 12:10:51 +0100
committerJosé Bollo <jose.bollo@iot.bzh>2016-02-04 15:47:01 +0100
commitda2824b15151f3fcecb43a42db08d1875bd30558 (patch)
tree29c764d360260ce02239d58f9933c804f60d1468
parente4809eb5f021bfe2e52da7b62e66b2fb349ff91d (diff)
afm-user-daemon: adds verbose info
Change-Id: I9195b2b10124b573be90ed4b754f7fe0ae5595ac Signed-off-by: José Bollo <jose.bollo@iot.bzh>
-rw-r--r--src/CMakeLists.txt2
-rw-r--r--src/afm-user-daemon.c51
-rw-r--r--src/verbose.c2
-rw-r--r--src/verbose.h4
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 <syslog.h>
#define LOGUSER(app) openlog(app,LOG_PERROR,LOG_USER)