Ok to commit msdos-disk-label-detection ?

[Sven Luther]

Hello, 

Since i got no reply to my list of debian patches i sent indirectly earlier, i
will send mail here for each of them, asking if it is ok to commit or not.

The first of those is the msdos-disk-label-detection :

#! /bin/sh -e
## msdos-disk-label-detection.dpatch by Matt Kraai <address@hidden>
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: If the label contains the Smart Boot Manager loader, do not
## DP: treat it as a single FAT partition.
## DP: Closes: #259248

. $(dirname $0)/DPATCH

@DPATCH@
diff -urNad /tmp/parted-1.6.11/libparted/disk_dos.c 
parted-1.6.11/libparted/disk_dos.c
--- /tmp/parted-1.6.11/libparted/disk_dos.c     2004-03-09 02:17:08.000000000 
-0800
+++ parted-1.6.11/libparted/disk_dos.c  2004-09-13 07:23:53.000000000 -0700
@@ -145,7 +145,8 @@
                return 0;
 
        /* if this is a FAT fs, fail here */
-       if (!strncmp (part_table.boot_code + 0x36, "FAT", 3)
+       if ((!strncmp (part_table.boot_code + 0x36, "FAT", 3)
+            && strncmp (part_table.boot_code + 0x40, "SBML", 4))
            || !strncmp (part_table.boot_code + 0x52, "FAT", 3))
                return 0;
 

It is straight forward, but i did no author it, nor am i familiar with the MBR
partition table, so comments are welcome ...

Friendly,

Sven Luther