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.

1 files changed, 4 insertions, 9 deletions

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) {