Changes in / [1db44ea7:f9d8c3a] in mainline

Location:

uspace

Files:

• drv/bus/usb/usbhid/kbd.h (deleted)
• drv/bus/usb/usbhid/kbd/layout.h (deleted)
• drv/bus/usb/usbhid/kbd/main.c (deleted)
• drv/bus/usb/usbhid/layout.h (deleted)
• drv/bus/usb/usbmast/scsi.h (deleted)
• srv/fs/mfs/mfs_ops.c (modified) (6 diffs)
• srv/hid/input/include/sun.h (deleted)
• srv/hid/input/port/dummy.c (deleted)
• srv/hid/input/port/sgcn.c (deleted)
• srv/hid/input/port/sun.c (deleted)
• srv/hid/input/port/z8530.c (deleted)

uspace/srv/fs/mfs/mfs_ops.c

@@ -143 +143 @@
 {
         enum cache_mode cmode;
-        struct mfs_superblock *sb = NULL;
-        struct mfs3_superblock *sb3 = NULL;
-        struct mfs_sb_info *sbi = NULL;
-        struct mfs_instance *instance = NULL;
+        struct mfs_superblock *sb;
+        struct mfs3_superblock *sb3;
+        struct mfs_sb_info *sbi;
+        struct mfs_instance *instance;
         bool native, longnames;
         mfs_version_t version;
@@ -166 +166 @@
         sbi = malloc(sizeof(*sbi));
         if (!sbi) {
-                rc = ENOMEM;
-                goto out_error;
+                block_fini(service_id);
+                return ENOMEM;
         }
 
@@ -173 +173 @@
         instance = malloc(sizeof(*instance));
         if (!instance) {
-                rc = ENOMEM;
-                goto out_error;
+                free(sbi);
+                block_fini(service_id);
+                return ENOMEM;
         }
 
         sb = malloc(MFS_SUPERBLOCK_SIZE);
         if (!sb) {
-                rc = ENOMEM;
-                goto out_error;
+                free(instance);
+                free(sbi);
+                block_fini(service_id);
+                return ENOMEM;
         }
 
         /* Read the superblock */
         rc = block_read_direct(service_id, MFS_SUPERBLOCK << 1, 2, sb);
-        if (rc != EOK)
-                goto out_error;
+        if (rc != EOK) {
+                free(instance);
+                free(sbi);
+                free(sb);
+                block_fini(service_id);
+                return rc;
+        }
 
         sb3 = (struct mfs3_superblock *) sb;
@@ -199 +207 @@
                 /*Not recognized*/
                 mfsdebug("magic number not recognized\n");
-                rc = ENOTSUP;
-                goto out_error;
+                free(instance);
+                free(sbi);
+                free(sb);
+                block_fini(service_id);
+                return ENOTSUP;
         }
 
@@ -245 +256 @@
                                     MFS_MAX_NAME_LEN;
         }
-
-        if (sbi->log2_zone_size != 0) {
-                /* In MFS, file space is allocated per zones.
-                 * Zones are a collection of consecutive blocks on disk.
-                 *
-                 * The current MFS implementation supports only filesystems
-                 * where the size of a zone is equal to the
-                 * size of a block.
-                 */
-                rc = ENOTSUP;
-                goto out_error;
-        }
-
         sbi->itable_off = 2 + sbi->ibmap_blocks + sbi->zbmap_blocks;
+
+        free(sb);
 
         rc = block_cache_init(service_id, sbi->block_size, 0, cmode);
         if (rc != EOK) {
+                free(instance);
+                free(sbi);
+                block_cache_fini(service_id);
+                block_fini(service_id);
                 mfsdebug("block cache initialization failed\n");
-                rc = EINVAL;
-                goto out_error;
+                return EINVAL;
         }
 
@@ -292 +295 @@
         *linkcnt = 1;
 
-        free(sb);
-
         return mfs_node_put(fn);
-
-out_error:
-        block_fini(service_id);
-        if (sb)
-                free(sb);
-        if (sbi)
-                free(sbi);
-        if(instance)
-                free(instance);
-        return rc;
 }