mdadm: Rearrange option parsing for KillSubarray and UpdateSubarray

[thirdparty/mdadm.git] / bitmap.c

diff --git a/bitmap.c b/bitmap.c
index 44a86773ddfef02a8530c60b60532a3e87f42f05..d02f16e64604cb74490e17fc6ad0a58fb5db57ef 100644 (file)
--- a/bitmap.c
+++ b/bitmap.c
@@ -130,7 +130,7 @@ bitmap_info_t *bitmap_fd_read(int fd, int brief)
        unsigned long long total_bits = 0, read_bits = 0, dirty_bits = 0;
        bitmap_info_t *info;
        void *buf;
-       int n, skip;
+       unsigned int n, skip;
 
        if (posix_memalign(&buf, 512, 8192) != 0) {
                fprintf(stderr, Name ": failed to allocate 8192 bytes\n");
@@ -147,6 +147,7 @@ bitmap_info_t *bitmap_fd_read(int fd, int brief)
                fprintf(stderr, Name ": failed to allocate %zd bytes\n",
                                sizeof(*info));
 #endif
+               free(buf);
                return NULL;
        }
 
@@ -154,6 +155,7 @@ bitmap_info_t *bitmap_fd_read(int fd, int brief)
                fprintf(stderr, Name ": failed to read superblock of bitmap "
                        "file: %s\n", strerror(errno));
                free(info);
+               free(buf);
                return NULL;
        }
        memcpy(&info->sb, buf, sizeof(info->sb));
@@ -198,6 +200,7 @@ bitmap_info_t *bitmap_fd_read(int fd, int brief)
                total_bits = read_bits;
        }
 out:
+       free(buf);
        info->total_bits = total_bits;
        info->dirty_bits = dirty_bits;
        return info;