bsd disk labels can either be at the start of the second sector, or 64
bytes into the first sector, but kpartx only handled the first case.
However the second case is what parted creates, and what the linux
kernel partition code expects.  kpartx should handle both cases.

Reviewed-by: Martin Wilck <mwi...@suse.com>
Signed-off-by: Benjamin Marzinski <bmarz...@redhat.com>
---
 kpartx/bsd.c | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/kpartx/bsd.c b/kpartx/bsd.c
index 0e661fbc..950b0f92 100644
--- a/kpartx/bsd.c
+++ b/kpartx/bsd.c
@@ -1,6 +1,7 @@
 #include "kpartx.h"
 #include <stdio.h>
 
+#define BSD_LABEL_OFFSET       64
 #define BSD_DISKMAGIC  (0x82564557UL)  /* The disk magic number */
 #define XBSD_MAXPARTITIONS     16
 #define BSD_FS_UNUSED          0
@@ -60,8 +61,19 @@ read_bsd_pt(int fd, struct slice all, struct slice *sp, 
unsigned int ns) {
                return -1;
 
        l = (struct bsd_disklabel *) bp;
-       if (l->d_magic != BSD_DISKMAGIC)
-               return -1;
+       if (l->d_magic != BSD_DISKMAGIC) {
+               /*
+                * BSD disklabels can also start 64 bytes offset from the
+                * start of the first sector
+                */
+               bp = getblock(fd, offset);
+               if (bp == NULL)
+                       return -1;
+
+               l = (struct bsd_disklabel *)(bp + 64);
+               if (l->d_magic != BSD_DISKMAGIC)
+                       return -1;
+       }
 
        max_partitions = 16;
        if (l->d_npartitions < max_partitions)
-- 
2.17.2

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to