]> git.ipfire.org Git - thirdparty/ipxe.git/commit
[romprefix] Do not clobber stack segment when returning to BIOS
authorMichael Brown <mcb30@ipxe.org>
Wed, 5 Mar 2014 12:25:21 +0000 (12:25 +0000)
committerMichael Brown <mcb30@ipxe.org>
Wed, 5 Mar 2014 12:32:00 +0000 (12:32 +0000)
commit1137fa32687cd0e1784683318a5381c41856a531
tree74d0bbb61e61a1a6243118d2d50119a67a8072f9
parentac5c2e851b14b153860c98eb01a5add982c91380
[romprefix] Do not clobber stack segment when returning to BIOS

Commit c429bf0 ("[romprefix] Store boot bus:dev.fn address as autoboot
device location") introduced a regression by using register %cx to
temporarily hold the PCI bus:dev.fn address, despite the fact that %cx
was already being used to hold the stored BIOS stack segment.
Consequently, when returning to the BIOS after a failed or cancelled
boot attempt, iPXE would end up calling INT 18 with the stack segment
set equal to the PCI bus:dev.fn address.  Writing to essentially
random areas of memory tends to upset even the more robust BIOSes.

Fix by using register %ax to temporarily hold the PCI bus:dev.fn
address.

Reported-by: Anton D. Kachalov <mouse@yandex-team.ru>
Tested-by: Anton D. Kachalov <mouse@yandex-team.ru>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/arch/i386/prefix/romprefix.S