diff options
author | José Bollo <jose.bollo@iot.bzh> | 2018-06-22 14:05:28 +0200 |
---|---|---|
committer | José Bollo <jose.bollo@iot.bzh> | 2018-06-22 14:05:28 +0200 |
commit | cd6d4492a271951c856ce611b88df0b4cc0ae7f0 (patch) | |
tree | 97b716bc99143f4582bb7b59ff44738ea20d9f2a /src/afb-api-v3.c | |
parent | 211343c77944333d550301da4d757e343e5f5390 (diff) |
api-v3: Add monitoring of faults in preinit functions
Change-Id: I0abea1aa62d86372f07f61b7c5dc2e854137f99a
Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Diffstat (limited to 'src/afb-api-v3.c')
-rw-r--r-- | src/afb-api-v3.c | 31 |
1 files changed, 30 insertions, 1 deletions
diff --git a/src/afb-api-v3.c b/src/afb-api-v3.c index 8c755f94..39f627a9 100644 --- a/src/afb-api-v3.c +++ b/src/afb-api-v3.c @@ -35,6 +35,7 @@ #include "afb-export.h" #include "afb-xreq.h" #include "verbose.h" +#include "sig-monitor.h" /* * Description of a binding @@ -342,12 +343,40 @@ int afb_api_v3_set_binding_fields(const struct afb_binding_v3 *desc, struct afb_ return rc; } +struct safe_preinit_data +{ + int (*preinit)(struct afb_api_x3 *); + struct afb_api_x3 *api; + int result; +}; + +static void safe_preinit(int sig, void *closure) +{ + struct safe_preinit_data *spd = closure; + if (!sig) + spd->result = spd->preinit(spd->api); + else { + spd->result = -1; + errno = EFAULT; + } +} + +int afb_api_v3_safe_preinit(struct afb_api_x3 *api, int (*preinit)(struct afb_api_x3 *)) +{ + struct safe_preinit_data spd; + + spd.preinit = preinit; + spd.api = api; + sig_monitor(60, safe_preinit, &spd); + return spd.result; +} + static int init_binding(void *closure, struct afb_api_x3 *api) { const struct afb_binding_v3 *desc = closure; int rc = afb_api_v3_set_binding_fields(desc, api); if (!rc && desc->preinit) - rc = desc->preinit(api); + rc = afb_api_v3_safe_preinit(api, desc->preinit); return rc; } |