chain.c32: Handle nonlinear logical partitions

Some partitioning program, including at least parted, apparently
create the list of logical partition headers nonlinearly with the
partition themselves.  Thus, make it a requirement that then fit
inside the overall extended partition, but not inside the
corresponding sublogical partition.

Reported-by: Gert Huselmans <gerth@zytor.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

diff --git a/com32/modules/chain.c b/com32/modules/chain.c
index 2dd8074..458ea89 100644 (file)
--- a/com32/modules/chain.c
+++ b/com32/modules/chain.c
@@ -1,7 +1,7 @@
 /* ----------------------------------------------------------------------- *
  *
  *   Copyright 2003-2009 H. Peter Anvin - All Rights Reserved
- *   Copyright 2009 Intel Corporation; author: H. Peter Anvin
+ *   Copyright 2009-2010 Intel Corporation; author: H. Peter Anvin
  *
  *   This program is free software; you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -394,10 +394,13 @@ static struct part_entry *find_logical_partition(int whichpart, char *table,
            /* Adjust the offset to account for the extended partition itself */
            ptab[i].start_lba += self->start_lba;
 
-           /* Sanity check entry: must not extend outside the extended partition.
-              This is necessary since some OSes put crap in some entries. */
-           if (ptab[i].start_lba + ptab[i].length <= self->start_lba ||
-               ptab[i].start_lba >= self->start_lba + self->length)
+           /*
+            * Sanity check entry: must not extend outside the
+            * extended partition.  This is necessary since some OSes
+            * put crap in some entries.  Note that root is non-NULL here.
+            */
+           if (ptab[i].start_lba + ptab[i].length <= root->start_lba ||
+               ptab[i].start_lba >= root->start_lba + root->length)
                continue;
 
            /* OK, it's a data partition.  Is it the one we're looking for? */