Changes in / [e950803:bd81386] in mainline

Location:

uspace

Files:

• lib/block/libblock.c (modified) (4 diffs)
• lib/c/generic/async.c (modified) (2 diffs)
• srv/devman/devman.c (modified) (1 diff)
• srv/devmap/devmap.c (modified) (3 diffs)
• srv/fs/devfs/devfs_ops.c (modified) (4 diffs)

uspace/lib/block/libblock.c

@@ -294 +294 @@
 
         /* Allow 1:1 or small-to-large block size translation */
-        if (cache->lblock_size % devcon->pblock_size != 0) {
-                free(cache);
+        if (cache->lblock_size % devcon->pblock_size != 0)
                 return ENOTSUP;
-        }
 
         cache->blocks_cluster = cache->lblock_size / devcon->pblock_size;
@@ -438 +436 @@
                         if (!b->data) {
                                 free(b);
-                                b = NULL;
                                 goto recycle;
                         }
@@ -566 +563 @@
         assert(devcon);
         assert(devcon->cache);
-        assert(block->refcnt >= 1);
 
         cache = devcon->cache;
@@ -626 +622 @@
                         unsigned long key = block->lba;
                         hash_table_remove(&cache->block_hash, &key, 1);
+                        free(block);
                         free(block->data);
-                        free(block);
                         cache->blocks_cached--;
                         fibril_mutex_unlock(&cache->lock);

uspace/lib/c/generic/async.c

@@ -294 +294 @@
 }
 
+/** Connection hash table removal callback function.
+ *
+ * This function is called whenever a connection is removed from the connection
+ * hash table.
+ *
+ * @param item Connection hash table item being removed.
+ *
+ */
 static void conn_remove(link_t *item)
 {
+        free(hash_table_get_instance(item, connection_t, link));
 }
 
@@ -638 +647 @@
                 ipc_answer_0(FIBRIL_connection->close_callid, EOK);
         
-        free(FIBRIL_connection);
         return 0;
 }

uspace/srv/devman/devman.c

@@ -1063 +1063 @@
         if (info != NULL) {
                 memset(info, 0, sizeof(dev_class_info_t));
-                link_initialize(&info->dev_classes);
-                link_initialize(&info->devmap_link);
-                link_initialize(&info->link);
+                list_initialize(&info->dev_classes);
+                list_initialize(&info->devmap_link);
+                list_initialize(&info->link);
         }
         

uspace/srv/devmap/devmap.c

@@ -423 +423 @@
          */
         list_initialize(&driver->devices);
-
-        link_initialize(&driver->drivers);
+        list_initialize(&(driver->drivers));
         
         fibril_mutex_lock(&drivers_list_mutex);
@@ -539 +538 @@
         }
         
-        link_initialize(&device->devices);
-        link_initialize(&device->driver_devices);
+        list_initialize(&(device->devices));
+        list_initialize(&(device->driver_devices));
         
         /* Check that device is not already registered */
@@ -943 +942 @@
         }
         
-        link_initialize(&device->devices);
-        link_initialize(&device->driver_devices);
+        list_initialize(&(device->devices));
+        list_initialize(&(device->driver_devices));
         
         /* Get unique device handle */

uspace/srv/fs/devfs/devfs_ops.c

@@ -130 +130 @@
 {
         devfs_node_t *node = (devfs_node_t *) pfn->data;
-        int ret;
         
         if (node->handle == 0) {
@@ -146 +145 @@
                                 
                                 if (str_cmp(devs[pos].name, component) == 0) {
-                                        ret = devfs_node_get_internal(rfn, DEV_HANDLE_NAMESPACE, devs[pos].handle);
                                         free(devs);
-                                        return ret;
+                                        return devfs_node_get_internal(rfn, DEV_HANDLE_NAMESPACE, devs[pos].handle);
                                 }
                         }
@@ -164 +162 @@
                                 for (pos = 0; pos < count; pos++) {
                                         if (str_cmp(devs[pos].name, component) == 0) {
-                                                ret = devfs_node_get_internal(rfn, DEV_HANDLE_DEVICE, devs[pos].handle);
                                                 free(devs);
-                                                return ret;
+                                                return devfs_node_get_internal(rfn, DEV_HANDLE_DEVICE, devs[pos].handle);
                                         }
                                 }
@@ -187 +184 @@
                         for (pos = 0; pos < count; pos++) {
                                 if (str_cmp(devs[pos].name, component) == 0) {
-                                        ret = devfs_node_get_internal(rfn, DEV_HANDLE_DEVICE, devs[pos].handle);
                                         free(devs);
-                                        return ret;
+                                        return devfs_node_get_internal(rfn, DEV_HANDLE_DEVICE, devs[pos].handle);
                                 }
                         }