Changes in uspace/lib/block/libblock.c [96b02eb9:956d4df8] in mainline

File:

• uspace/lib/block/libblock.c (modified) (9 diffs)

uspace/lib/block/libblock.c

@@ -44 +44 @@
 #include <sys/mman.h>
 #include <async.h>
-#include <ipc/ipc.h>
 #include <as.h>
 #include <assert.h>
@@ -177 +176 @@
         if (rc != EOK) {
                 munmap(comm_area, comm_size);
-                ipc_hangup(dev_phone);
+                async_hangup(dev_phone);
                 return rc;
         }
@@ -183 +182 @@
         if (get_block_size(dev_phone, &bsize) != EOK) {
                 munmap(comm_area, comm_size);
-                ipc_hangup(dev_phone);
+                async_hangup(dev_phone);
                 return rc;
         }
@@ -190 +189 @@
         if (rc != EOK) {
                 munmap(comm_area, comm_size);
-                ipc_hangup(dev_phone);
+                async_hangup(dev_phone);
                 return rc;
         }
@@ -211 +210 @@
 
         munmap(devcon->comm_area, devcon->comm_size);
-        ipc_hangup(devcon->dev_phone);
+        async_hangup(devcon->dev_phone);
 
         free(devcon);   
@@ -295 +294 @@
 
         /* Allow 1:1 or small-to-large block size translation */
-        if (cache->lblock_size % devcon->pblock_size != 0)
+        if (cache->lblock_size % devcon->pblock_size != 0) {
+                free(cache);
                 return ENOTSUP;
+        }
 
         cache->blocks_cluster = cache->lblock_size / devcon->pblock_size;
@@ -437 +438 @@
                         if (!b->data) {
                                 free(b);
+                                b = NULL;
                                 goto recycle;
                         }
@@ -564 +566 @@
         assert(devcon);
         assert(devcon->cache);
+        assert(block->refcnt >= 1);
 
         cache = devcon->cache;
@@ -623 +626 @@
                         unsigned long key = block->lba;
                         hash_table_remove(&cache->block_hash, &key, 1);
+                        fibril_mutex_unlock(&block->lock);
+                        free(block->data);
                         free(block);
-                        free(block->data);
                         cache->blocks_cached--;
                         fibril_mutex_unlock(&cache->lock);