linux-kernel - Re: Extended partition mapping wrong size

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20100406114706.GA30340@nb.net.home>
Date:	Tue, 6 Apr 2010 13:47:07 +0200
From:	Karel Zak <kzak@...hat.com>
To:	Phillip Susi <psusi@....rr.com>
Cc:	Linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: Extended partition mapping wrong size

On Mon, Mar 29, 2010 at 12:11:23PM -0400, Phillip Susi wrote:
> I've been investigating a problem I ran into trying to create partitions
> in sector mode and found that the first logical partition can not begin
> on the very first sector of the extended partition, immediately
> following the EBR.  This apparently is because the kernel creates a dev
> node to represent the extended partition and sizes it to two sectors.

This is probably kernel bug. It's really insane that the extended
pseudo partition overflows to the next logical partition.

> could have sworn that the kernel did not used to create a device for the
> extended partition itself, but I wondered why it was 2 sectors long.  I
> found this:
> 
> from fs/partitions/msdos.c:
> 
>             /* prevent someone doing mkfs or mkswap on an
>                extended partition, but leave room for LILO */
>             put_partition(state, slot, start, size == 1 ? 1 : 2);
>
> This appears to set the size of the device to 2 sectors, unless the
> extended partition is only 1 sector long.  Shouldn't the size be
> whatever length there is between the start of the extended partition,
> and the first logical partition it contains?  So if there are 63 sectors
> there, as is the usual case when using cylinder alignment, then the

 Please no. I think the size should not be more than 2 sectors (1024
 bytes). The current concept works for years and we have in userspace 
 /etc/partitions parsers that use "if (blocks <= 1)" to detect
 extended partitions.

 The other problem are mkfs programs, the space used for alignment
 could be 1MiB (or more) -- it's enough many mkfs programs.


 BTW, Linux does not use this policy for the others nested partition
 tables (e.g. Solaris, BSD, Minix, ...). The extended dos partition
 table is exception. The primary partitions for the others nested PT
 are exported to the system with its real size :-)

    Karel

-- 
 Karel Zak  <kzak@...hat.com>
 http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/