Changes in uspace/srv/fs/cdfs/cdfs_ops.c [94e3a03:0cebbac] in mainline

File:

• uspace/srv/fs/cdfs/cdfs_ops.c (modified) (16 diffs)

uspace/srv/fs/cdfs/cdfs_ops.c

@@ -42 +42 @@
 #include <adt/hash_table.h>
 #include <adt/hash.h>
+#include <malloc.h>
 #include <mem.h>
 #include <loc.h>
@@ -48 +49 @@
 #include <block.h>
 #include <scsi/mmc.h>
-#include <stdlib.h>
 #include <str.h>
 #include <byteorder.h>
@@ -566 +566 @@
 }
 
-static int cdfs_readdir(cdfs_t *fs, fs_node_t *fs_node)
+static bool cdfs_readdir(cdfs_t *fs, fs_node_t *fs_node)
 {
         cdfs_node_t *node = CDFS_NODE(fs_node);
@@ -572 +572 @@
         
         if (node->processed)
-                return EOK;
+                return true;
         
         uint32_t blocks = node->size / BLOCK_SIZE;
@@ -582 +582 @@
                 int rc = block_get(&block, fs->service_id, node->lba + i, BLOCK_FLAGS_NONE);
                 if (rc != EOK)
-                        return rc;
+                        return false;
                 
                 cdfs_dir_t *dir;
@@ -616 +616 @@
                         int rc = create_node(&fn, fs, dentry_type,
                             (node->lba + i) * BLOCK_SIZE + offset);
-                        if (rc != EOK)
-                                return rc;
-
-                        assert(fn != NULL);
+                        if ((rc != EOK) || (fn == NULL))
+                                return false;
                         
                         cdfs_node_t *cur = CDFS_NODE(fn);
@@ -628 +626 @@
                             dir->name_length, node->fs->enc, dentry_type);
                         if (name == NULL)
-                                return EIO;
+                                return false;
                         
                         // FIXME: check return value
@@ -643 +641 @@
         
         node->processed = true;
-        return EOK;
+        return true;
 }
 
@@ -963 +961 @@
 
 /** Read the volume descriptors. */
-static int iso_read_vol_desc(service_id_t sid, cdfs_lba_t altroot,
+static bool iso_read_vol_desc(service_id_t sid, cdfs_lba_t altroot,
     uint32_t *rlba, uint32_t *rsize, cdfs_enc_t *enc, char **vol_ident)
 {
@@ -970 +968 @@
         int rc = block_get(&block, sid, altroot + 16, BLOCK_FLAGS_NONE);
         if (rc != EOK)
-                return rc;
+                return false;
         
         cdfs_vol_desc_t *vol_desc = (cdfs_vol_desc_t *) block->data;
@@ -982 +980 @@
             (vol_desc->version != 1)) {
                 block_put(block);
-                return ENOTSUP;
+                return false;
         }
         
@@ -1002 +1000 @@
                  */
                 block_put(block);
-                return ENOTSUP;
+                return false;
         }
         
@@ -1008 +1006 @@
         if (block_size != BLOCK_SIZE) {
                 block_put(block);
-                return ENOTSUP;
+                return false;
         }
         
@@ -1033 +1031 @@
         
         block_put(block);
-        return EOK;
-}
-
-static int iso_readfs(cdfs_t *fs, fs_node_t *rfn,
+        return true;
+}
+
+static bool iso_readfs(cdfs_t *fs, fs_node_t *rfn,
     cdfs_lba_t altroot)
 {
         cdfs_node_t *node = CDFS_NODE(rfn);
         
-        int rc = iso_read_vol_desc(fs->service_id, altroot, &node->lba,
-            &node->size, &fs->enc, &fs->vol_ident);
-        if (rc != EOK)
-                return rc;
+        if (!iso_read_vol_desc(fs->service_id, altroot, &node->lba,
+            &node->size, &fs->enc, &fs->vol_ident))
+                return false;
         
         return cdfs_readdir(fs, rfn);
@@ -1075 +1072 @@
         
         /* Check if there is cdfs in given session */
-        if (iso_readfs(fs, rfn, altroot) != EOK)
+        if (!iso_readfs(fs, rfn, altroot))
                 goto error;
         
@@ -1128 +1125 @@
         uint32_t rsize;
         cdfs_enc_t enc;
-        rc = iso_read_vol_desc(service_id, altroot, &rlba, &rsize, &enc,
-            &vol_ident);
-        if (rc != EOK) {
+        if (!iso_read_vol_desc(service_id, altroot, &rlba, &rsize, &enc,
+            &vol_ident)) {
                 block_cache_fini(service_id);
                 block_fini(service_id);
-                return rc;
+                return EIO;
         }