aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Bollo <jose.bollo@iot.bzh>2018-10-28 18:53:37 +0100
committerJose Bollo <jose.bollo@iot.bzh>2018-10-29 10:15:46 +0100
commit41a1af2431185c4d78a8d4f9d87096f5f3fa9eb9 (patch)
tree35fa77cd8a250750d989bc038d4e42c5ed0aa08d
parent4c384402dc4ab629a1b006cb75db9df2aff71999 (diff)
afm-urun: Force scan of systemd units
It was previously assumed that only launched units had to be scanned. But it was buggy because the uninstall/install of application rebuilds the database without taking care of the running applications. Nevertheless changing the behaviour of the rebuild of application's database is not the best solution for two reasons. The fist is that it will complicates the code. The second, much better, is that it would allow to track also applications launched automaticaly -and/or manually in devel-. Bug-AGL: SPEC-1836 Change-Id: I5fb2bb712ce43be9c851251be6b794a007b1ea6f Signed-off-by: José Bollo <jose.bollo@iot.bzh>
-rw-r--r--src/afm-urun.c20
1 files changed, 7 insertions, 13 deletions
diff --git a/src/afm-urun.c b/src/afm-urun.c
index 0249a60..8c21b2a 100644
--- a/src/afm-urun.c
+++ b/src/afm-urun.c
@@ -40,7 +40,7 @@ static const char key_unit_d_path[] = "-unit-dpath-";
/**************** get appli basis *********************/
-static int get_basis(struct json_object *appli, int *isuser, const char **dpath, int load, int uid)
+static int get_basis(struct json_object *appli, int *isuser, const char **dpath, int uid)
{
char userid[40];
char *dp, *arodot, *nun;
@@ -76,12 +76,6 @@ static int get_basis(struct json_object *appli, int *isuser, const char **dpath,
return 0;
}
- /* not here. load it? */
- if (!load) {
- errno = ENOENT;
- goto error;
- }
-
/* get uname */
if (!j_read_string_at(appli, "unit-name", &uname)) {
ERROR("'unit-name' missing in appli description %s", json_object_get_string(appli));
@@ -246,7 +240,7 @@ int afm_urun_once(struct json_object *appli, int uid)
int rc, isuser;
/* retrieve basis */
- rc = get_basis(appli, &isuser, &udpath, 1, uid);
+ rc = get_basis(appli, &isuser, &udpath, uid);
if (rc < 0)
goto error;
@@ -280,7 +274,7 @@ int afm_urun_once(struct json_object *appli, int uid)
ERROR("can't getpid of %s unit %s for uid %d: %m", uscope, uname, uid);
goto error;
}
-
+
return rc;
error:
@@ -352,7 +346,7 @@ struct json_object *afm_urun_list(struct afm_udb *db, int uid)
n = json_object_array_length(apps);
for (i = 0 ; i < n ; i++) {
appli = json_object_array_get_idx(apps, i);
- if (appli && get_basis(appli, &isuser, &udpath, 0, uid) >= 0) {
+ if (appli && get_basis(appli, &isuser, &udpath, uid) >= 0) {
pid = systemd_unit_pid_of_dpath(isuser, udpath);
if (pid > 0 && j_read_string_at(appli, "id", &id)) {
state = systemd_unit_state_of_dpath(isuser, udpath);
@@ -404,7 +398,7 @@ struct json_object *afm_urun_state(struct afm_udb *db, int runid, int uid)
for (i = 0 ; i < n ; i++) {
appli = json_object_array_get_idx(apps, i);
if (appli
- && get_basis(appli, &isuser, &udpath, 0, uid) >= 0
+ && get_basis(appli, &isuser, &udpath, uid) >= 0
&& !strcmp(dpath, udpath)
&& j_read_string_at(appli, "id", &id)) {
pid = systemd_unit_pid_of_dpath(isuser, udpath);
@@ -418,7 +412,7 @@ struct json_object *afm_urun_state(struct afm_udb *db, int runid, int uid)
WARNING("searched runid %d of dpath %s isn't an applications", runid, dpath);
end:
json_object_put(apps);
- free(dpath);
+ free(dpath);
}
return result;
@@ -439,7 +433,7 @@ int afm_urun_search_runid(struct afm_udb *db, const char *id, int uid)
NOTICE("Unknown appid %s", id);
errno = ENOENT;
pid = -1;
- } else if (get_basis(appli, &isuser, &udpath, 0, uid) < 0) {
+ } else if (get_basis(appli, &isuser, &udpath, uid) < 0) {
pid = -1;
} else {
pid = systemd_unit_pid_of_dpath(isuser, udpath);