git: 1323f0aa9cb0 - main - stand/uboot: setup archsw before probing devices

From: Marcin Wojtas <mw_at_FreeBSD.org>
Date: Fri, 01 Jul 2022 11:37:19 UTC
The branch main has been updated by mw:

URL: https://cgit.FreeBSD.org/src/commit/?id=1323f0aa9cb0392d2946226bd76f14463e875a60

commit 1323f0aa9cb0392d2946226bd76f14463e875a60
Author:     Albert Jakiela <aja@semihalf.com>
AuthorDate: 2022-06-28 15:28:35 +0000
Commit:     Marcin Wojtas <mw@FreeBSD.org>
CommitDate: 2022-07-01 11:36:41 +0000

    stand/uboot: setup archsw before probing devices
    
    In some cases ubldr would try to mount a disk device before
    the archsw struct was filled with functions pointers.
    This would result in a NULL pointer derefrence of the arch_getdev field.
    Fix that filling the archsw functions earlier.
    Note that this matches the EFI behavoiur.
    
    Reviewed by: imp, mw
    Sponsored by: Stormshield
    Obtained from: Semihalf
    MFC after: 2 weeks
    Differential Revision: https://reviews.freebsd.org/D35670
---
 stand/uboot/main.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/stand/uboot/main.c b/stand/uboot/main.c
index 85ddf5db1f90..6648dc607c11 100644
--- a/stand/uboot/main.c
+++ b/stand/uboot/main.c
@@ -475,6 +475,13 @@ main(int argc, char **argv)
 
 	meminfo();
 
+	archsw.arch_loadaddr = uboot_loadaddr;
+	archsw.arch_getdev = uboot_getdev;
+	archsw.arch_copyin = uboot_copyin;
+	archsw.arch_copyout = uboot_copyout;
+	archsw.arch_readin = uboot_readin;
+	archsw.arch_autoload = uboot_autoload;
+
 	/* Set up currdev variable to have hooks in place. */
 	env_setenv("currdev", EV_VOLATILE, "", uboot_setcurrdev, env_nounset);
 
@@ -538,13 +545,6 @@ do_interact:
 	setenv("usefdt", "1", 1);
 #endif
 
-	archsw.arch_loadaddr = uboot_loadaddr;
-	archsw.arch_getdev = uboot_getdev;
-	archsw.arch_copyin = uboot_copyin;
-	archsw.arch_copyout = uboot_copyout;
-	archsw.arch_readin = uboot_readin;
-	archsw.arch_autoload = uboot_autoload;
-
 	interact();				/* doesn't return */
 
 	return (0);