Changes in uspace/lib/c/generic/rtld/module.c [a0e2f9c:1567471] in mainline

File:

• uspace/lib/c/generic/rtld/module.c (modified) (9 diffs)

uspace/lib/c/generic/rtld/module.c

@@ -65 +65 @@
 
         module = calloc(1, sizeof(module_t));
-        if (module == NULL) {
-                DPRINTF("malloc failed\n");
+        if (module == NULL)
                 return ENOMEM;
-        }
 
         module->id = rtld_get_next_id(rtld);
@@ -184 +182 @@
         char name_buf[NAME_BUF_SIZE];
         module_t *m;
-        errno_t rc;
+        int rc;
 
         m = calloc(1, sizeof(module_t));
         if (m == NULL) {
-                DPRINTF("malloc failed\n");
-                goto error;
+                printf("malloc failed\n");
+                exit(1);
         }
 
@@ -199 +197 @@
 
         if (str_size(name) > NAME_BUF_SIZE - 2) {
-                DPRINTF("soname too long. increase NAME_BUF_SIZE\n");
-                goto error;
+                printf("soname too long. increase NAME_BUF_SIZE\n");
+                exit(1);
         }
 
@@ -210 +208 @@
 
         rc = elf_load_file_name(name_buf, RTLD_MODULE_LDF, &info);
-        if (rc != EOK) {
-                DPRINTF("Failed to load '%s'\n", name_buf);
-                goto error;
+        if (rc != EE_OK) {
+                printf("Failed to load '%s'\n", name_buf);
+                exit(1);
         }
 
@@ -220 +218 @@
 
         if (info.dynamic == NULL) {
-                DPRINTF("Error: '%s' is not a dynamically-linked object.\n",
+                printf("Error: '%s' is not a dynamically-linked object.\n",
                     name_buf);
-                goto error;
+                exit(1);
         }
 
@@ -245 +243 @@
 
         return m;
-
-error:
-        if (m)
-                free(m);
-
-        return NULL;
 }
 
 /** Load all modules on which m (transitively) depends.
  */
-errno_t module_load_deps(module_t *m, mlflags_t flags)
+void module_load_deps(module_t *m, mlflags_t flags)
 {
         elf_dyn_t *dp;
@@ -282 +274 @@
                 /* There are no dependencies, so we are done. */
                 m->deps = NULL;
-                return EOK;
+                return;
         }
 
         m->deps = malloc(n * sizeof(module_t *));
         if (!m->deps) {
-                DPRINTF("malloc failed\n");
-                return ENOMEM;
+                printf("malloc failed\n");
+                exit(1);
         }
 
@@ -302 +294 @@
                         if (!dm) {
                                 dm = module_load(m->rtld, dep_name, flags);
-                                if (!dm) {
-                                        return EINVAL;
-                                }
-
-                                errno_t rc = module_load_deps(dm, flags);
-                                if (rc != EOK) {
-                                        return rc;
-                                }
+                                module_load_deps(dm, flags);
                         }
 
@@ -317 +302 @@
                 ++dp;
         }
-
-        return EOK;
 }