From 50b375730189856bf2efa70dde83d78389d611c9 Mon Sep 17 00:00:00 2001 From: Pascal Terjan Date: Thu, 26 Nov 2009 13:21:41 +0000 Subject: detect better invalid dos partition table (fixes a crash in harddrake) --- perl-install/NEWS | 1 + perl-install/partition_table/dos.pm | 14 ++++++++++++++ 2 files changed, 15 insertions(+) diff --git a/perl-install/NEWS b/perl-install/NEWS index 1adb23681..1127492f2 100644 --- a/perl-install/NEWS +++ b/perl-install/NEWS @@ -11,6 +11,7 @@ o fix detecting some mice - bootloader: o do not add removable drives as "Windows" entry in bootloader +- detect better invalid dos partition table (fixes a crash in harddrake) Version 12.78 - 18 November 2009 diff --git a/perl-install/partition_table/dos.pm b/perl-install/partition_table/dos.pm index 2376fce11..66157c08b 100644 --- a/perl-install/partition_table/dos.pm +++ b/perl-install/partition_table/dos.pm @@ -220,6 +220,20 @@ sub read_one { sysread $F, $tmp, length $magic or die "error reading magic number on disk $hd->{device}"; $tmp eq $magic or die "bad magic number on disk $hd->{device}"; + if(c::get_disk_type($hd->{file}) ne "msdos") { + # libparted may have ignored it because of overlapping partitions or other error + # while it is actually a partition table. + $hd->{fs_type_from_magic} and die "unpartitionned disk"; + my $primary = partition_table::raw::pt_info_to_primary($hd, [ @pt ]); + foreach my $i (@{$primary->{normal}}) { + print Dumper($hd); + if (($i->{active} && $i->{active} != 0x80) || + ($hd->{totalsectors} && $i->{start} > $hd->{totalsectors})) { + die "Invalid DOS partition table"; + } + } + } + [ @pt ]; } -- cgit v1.2.1