Changes in uspace/lib/gpt/libgpt.c [8559fa0:9bdfde73] in mainline

File:

• uspace/lib/gpt/libgpt.c (modified) (7 diffs)

uspace/lib/gpt/libgpt.c

@@ -127 +127 @@
         rc = block_init(EXCHANGE_ATOMIC, dev_handle, 512);
         if (rc != EOK)
-                goto fail;
+                return rc;
         
         rc = block_get_bsize(dev_handle, &b_size);
         if (rc != EOK)
-                goto fini_fail;
+                return rc;
         
         if (label->gpt == NULL) {
                 label->gpt = gpt_alloc_header(b_size);
-                if (label->gpt == NULL) {
-                        rc = ENOMEM;
-                        goto fini_fail;
-                }
+                if (label->gpt == NULL)
+                        return ENOMEM;
         }
         
@@ -146 +144 @@
                 rc = block_get_nblocks(dev_handle, &n_blocks);
                 if (rc != EOK)
-                        goto free_fail;
+                        goto fail;
 
                 rc = load_and_check_header(dev_handle, n_blocks - 1, b_size, label->gpt->header);
                 if (rc == EBADCHECKSUM || rc == EINVAL)
-                        goto free_fail;
+                        goto fail;
         }
         
@@ -157 +155 @@
         return EOK;
         
-free_fail:
+fail:
+        block_fini(dev_handle);
         gpt_free_gpt(label->gpt);
         label->gpt = NULL;
-fini_fail:
-        block_fini(dev_handle);
-fail:
         return rc;
 }
@@ -244 +240 @@
          *  - we don't need more bytes
          *  - the size of GPT partition entry can be different to 128 bytes */
-        /* comm_size is ignored */
         rc = block_init(EXCHANGE_SERIALIZE, label->device, sizeof(gpt_entry_t));
         if (rc != EOK)
@@ -252 +247 @@
         rc = block_get_bsize(label->device, &block_size);
         if (rc != EOK)
-                goto fini_fail;
+                goto fail;
 
         //size_t bufpos = 0;
@@ -271 +266 @@
 
                 if (rc != EOK)
-                        goto fini_fail;
+                        goto fail;
         }
 
@@ -279 +274 @@
          * on all of the partition entry array.
          */
-        uint32_t crc = compute_crc32((uint8_t *) label->parts->part_array, 
-                           label->parts->fill * sizeof(gpt_entry_t));
+        uint32_t crc = compute_crc32((uint8_t *) label->parts->part_array, label->parts->fill * sizeof(gpt_entry_t));
 
         if(uint32_t_le2host(label->gpt->header->pe_array_crc32) != crc)
         {
                 rc = EBADCHECKSUM;
-                goto fini_fail;
-        }
-        
-        block_fini(label->device);
+                goto fail;
+        }
+
         return EOK;
-        
-fini_fail:
-        block_fini(label->device);
         
 fail: