PR/45990 CVS commit: src/distrib/utils/sysinst

["Izumi Tsutsui" <tsutsui%netbsd.org@localhost>]

The following reply was made to PR install/45990; it has been noted by GNATS.

From: "Izumi Tsutsui" <tsutsui%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/45990 CVS commit: src/distrib/utils/sysinst
Date: Sat, 26 May 2012 05:10:00 +0000

 Module Name:   src
 Committed By:  tsutsui
 Date:          Sat May 26 05:09:59 UTC 2012
 
 Modified Files:
        src/distrib/utils/sysinst: mbr.c
 
 Log Message:
 Fix regression by my dumb patch in PR/45990.
 
 sysinst fails with "floating exception" after
 changing MBR partition size in MBR editor menu
 if the target disk doesn't have valid MBR partition
 or has a valid partition 0 whose offset is not 1MB aligned
 (like 63 sectors).
 
 read_mbr() (which calls get_ptn_alignment()) is called
 before set_bios_geom(), so bhead is not initialized there
 and ptn_alignment could be zero.
 
 To workaround, explicitly call get_ptn_alignment() again
 in edit_mbr() to update ptn_alignemnt per BIOS geom values.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.90 -r1.91 src/distrib/utils/sysinst/mbr.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.