Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch/i386/stand/efiboot




To: "Emmanuel Dreyfus" <manu%netbsd.org@localhost>

Subject: Re: CVS commit: src/sys/arch/i386/stand/efiboot

From: Simon Burge <simonb%NetBSD.org@localhost>

Date: Mon, 27 Dec 2021 13:08:15 +1100


Hi Emmanuel,

"Emmanuel Dreyfus" wrote:

> Module Name: src
> Committed By: manu
> Date:  Thu Nov 18 16:18:13 UTC 2021
>
> Modified Files:
>
>  src/sys/arch/i386/stand/efiboot: devopen.c
>
> Log Message:
>
> Fix crash because of NULL pointer reference

What crash did this fix?  All the use of part_name by the
called functions should check if it is NULL before trying
to assign anything to *part_name.

This change has broken loading boot.cfg via the EFI path
"esp:/EFI/NetBSD/boot.cfg" since the call to bios_boot() at
https://nxr.netbsd.org/xref/src/sys/arch/i386/stand/efiboot/devopen.c#292
with a non-NULL last argument means devname gets updated and
now points to the partition with a root filesystem rather
than the EFI system partition.

Cheers,
Simon.


Follow-Ups:

Re: CVS commit: src/sys/arch/i386/stand/efiboot
From: Emmanuel Dreyfus




Prev by Date: Re: CVS commit: src/sys/dev/pci

Next by Date: Re: CVS commit: src/sys/arch/i386/stand/efiboot

Previous by Thread: Re: CVS commit: src/sys/dev/pci

Next by Thread: Re: CVS commit: src/sys/arch/i386/stand/efiboot

Indexes:

reverse Date

reverse Thread

Old Index



Home | Main Index | Thread Index | Old Index