From bc9c60382b226e5a75a11a99364b9b799dc2b0c2 Mon Sep 17 00:00:00 2001 From: dimitri staessens Date: Tue, 5 Jul 2016 18:52:12 +0200 Subject: lib: Change invalid pid to -1 The stack used pid 0 (the scheduler) to indicate an invalid process instance, probably as a leftover from the deprecated application process instance id. Using -1 is a better solution. Fixes #16. --- src/irmd/registry.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'src/irmd/registry.c') diff --git a/src/irmd/registry.c b/src/irmd/registry.c index 063309a4..a4e2fcfa 100644 --- a/src/irmd/registry.c +++ b/src/irmd/registry.c @@ -290,7 +290,7 @@ pid_t reg_entry_resolve_api(struct reg_entry * e) return r->api; } - return 0; + return -1; } char ** reg_entry_resolve_auto(struct reg_entry * e) @@ -498,7 +498,7 @@ struct reg_instance * registry_add_ap_instance(struct list_head * registry, struct reg_entry * e = NULL; struct reg_instance * i = NULL; - if (name == NULL || api == 0) + if (name == NULL || api == -1) return NULL; e = registry_get_entry_by_name(registry, name); @@ -507,11 +507,6 @@ struct reg_instance * registry_add_ap_instance(struct list_head * registry, return NULL; } - if (api == 0) { - LOG_DBG("Invalid api."); - return NULL; - } - if (reg_entry_has_api(e, api)) { LOG_DBG("Instance already registered with this name."); return NULL; @@ -546,8 +541,8 @@ int registry_remove_ap_instance(struct list_head * registry, struct reg_entry * e = NULL; struct reg_instance * i = NULL; - if (name == NULL || api == 0) - return -1; + if (name == NULL || api == -1) + return -EINVAL; e = registry_get_entry_by_name(registry, name); if (e == NULL) { -- cgit v1.2.3