Changes in / [5fe7692:93ebe4e] in mainline

Files:

• defaults/arm32/gta02/Makefile.config (modified) (1 diff)
• uspace/lib/c/include/adt/int_map.h (modified) (1 diff)
• uspace/srv/devman/devman.c (modified) (3 diffs)
• uspace/srv/devman/main.c (modified) (1 diff)
• uspace/srv/devman/util.c (modified) (1 diff)
• uspace/srv/devman/util.h (modified) (1 diff)

defaults/arm32/gta02/Makefile.config

@@ -4 +4 @@
 # RAM disk format
 RDFMT = tmpfs
-
-# Support for userspace debuggers
-CONFIG_UDEBUG = n
-
-# Compile kernel tests
-CONFIG_TEST = n
-
-# Barebone build with essential binaries only
-CONFIG_BAREBONE = y

uspace/lib/c/include/adt/int_map.h

@@ -246 +246 @@
                         if (item->value) { \
                                 if (dtor) \
-                                        free(item->value); \
+                                        dtor(item->value); \
                                 item->value = NULL; \
                         } \

uspace/srv/devman/devman.c

@@ -266 +266 @@
         }
         
-        if (read(fd, buf, len) <= 0) {
+        ssize_t read_bytes = safe_read(fd, buf, len);
+        if (read_bytes <= 0) {
                 printf(NAME ": unable to read file '%s'.\n", conf_path);
                 goto cleanup;
         }
-        buf[len] = 0;
+        buf[read_bytes] = 0;
         
         suc = parse_match_ids(buf, ids);
@@ -1123 +1124 @@
 fun_node_t *find_fun_node_by_path(dev_tree_t *tree, char *path)
 {
+        assert(path != NULL);
+
+        bool is_absolute = path[0] == '/';
+        if (!is_absolute) {
+                return NULL;
+        }
+
         fibril_rwlock_read_lock(&tree->rwlock);
         
@@ -1132 +1140 @@
         char *rel_path = path;
         char *next_path_elem = NULL;
-        bool cont = (rel_path[0] == '/');
+        bool cont = true;
         
         while (cont && fun != NULL) {

uspace/srv/devman/main.c

@@ -477 +477 @@
                 dev = fun->dev;
 
-        if (fun == NULL && dev == NULL) {
+        /*
+         * For a valid function to connect to we need a device. The root
+         * function, for example, has no device and cannot be connected to.
+         * This means @c dev needs to be valid regardless whether we are
+         * connecting to a device or to a function.
+         */
+        if (dev == NULL) {
                 printf(NAME ": devman_forward error - no device or function with "
                     "handle %" PRIun " was found.\n", handle);

uspace/srv/devman/util.c

@@ -111 +111 @@
 }
 
+ssize_t safe_read(int fd, void *buffer, size_t size)
+{
+        if (size == 0) {
+                return 0;
+        }
+
+        uint8_t *buf_ptr = (uint8_t *) buffer;
+
+        size_t total_read = 0;
+        while (total_read < size) {
+                ssize_t bytes_read = read(fd, buf_ptr, size - total_read);
+                if (bytes_read < 0) {
+                        /* Error. */
+                        return bytes_read;
+                } else if (bytes_read == 0) {
+                        /* Possibly end of file. */
+                        break;
+                } else {
+                        /* Read at least something. */
+                        buf_ptr += bytes_read;
+                        total_read += bytes_read;
+                }
+        }
+
+        return (ssize_t) total_read;
+}
+
 /** @}
  */

uspace/srv/devman/util.h

@@ -47 +47 @@
 extern void replace_char(char *, char, char);
 
+extern ssize_t safe_read(int, void *, size_t);
+
 #endif