git: 3ced4a2b482f - stable/14 - bsdinstall: Fix newfs bsddialog menu

From: Ed Maste <emaste_at_FreeBSD.org>
Date: Wed, 12 Nov 2025 15:48:14 UTC
The branch stable/14 has been updated by emaste:

URL: https://cgit.FreeBSD.org/src/commit/?id=3ced4a2b482f50672bba26de3f26273a73f75b9c

commit 3ced4a2b482f50672bba26de3f26273a73f75b9c
Author:     Ed Maste <emaste@FreeBSD.org>
AuthorDate: 2025-11-07 22:53:01 +0000
Commit:     Ed Maste <emaste@FreeBSD.org>
CommitDate: 2025-11-12 15:47:42 +0000

    bsdinstall: Fix newfs bsddialog menu
    
    The second and third members of struct bsddialog_menuitem are `bool on`
    and `unsigned int depth`. The newfs dialog options in bsdinstall's
    partition tool had these two swapped, so the default selection did not
    work.
    
    PR: 290857
    Reviewed by: asiciliano
    Fixes: 50e244964e9b ("bsdinstall/partedit: Replace libdialog with libbsddialog")
    Sponsored by: The FreeBSD Foundation
    Differential Revision: https://reviews.freebsd.org/D53639
    
    (cherry picked from commit 4e36942420712c2ab6ebc2c646e61d47b2b68e7b)
    (cherry picked from commit 980aa8d4cfdf57a1f99401fa4160c0d82c927d7c)
---
 usr.sbin/bsdinstall/partedit/gpart_ops.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/usr.sbin/bsdinstall/partedit/gpart_ops.c b/usr.sbin/bsdinstall/partedit/gpart_ops.c
index 5c4ec5706c1c..b598d4cb8bcb 100644
--- a/usr.sbin/bsdinstall/partedit/gpart_ops.c
+++ b/usr.sbin/bsdinstall/partedit/gpart_ops.c
@@ -139,16 +139,16 @@ newfs_command(const char *fstype, int use_default)
 	} else if (strcmp(fstype, "freebsd-zfs") == 0) {
 		int i;
 		struct bsddialog_menuitem items[] = {
-			{"", 0, true, "fletcher4", "checksum algorithm: fletcher4",
+			{"", true, 0, "fletcher4", "checksum algorithm: fletcher4",
 			    "Use fletcher4 for data integrity checking. "
 			    "(default)"},
-			{"", 0, false, "fletcher2", "checksum algorithm: fletcher2",
+			{"", false, 0, "fletcher2", "checksum algorithm: fletcher2",
 			    "Use fletcher2 for data integrity checking. "
 			    "(not recommended)"},
-			{"", 0, false, "sha256", "checksum algorithm: sha256",
+			{"", false, 0, "sha256", "checksum algorithm: sha256",
 			    "Use sha256 for data integrity checking. "
 			    "(not recommended)"},
-			{"", 0, false, "atime", "Update atimes for files",
+			{"", false, 0, "atime", "Update atimes for files",
 			    "Disable atime update"},
 		};
 
@@ -188,11 +188,11 @@ newfs_command(const char *fstype, int use_default)
 	     strcmp(fstype, "ms-basic-data") == 0) {
 		int i;
 		struct bsddialog_menuitem items[] = {
-			{"", 0, true, "FAT32", "FAT Type 32",
+			{"", true, 0, "FAT32", "FAT Type 32",
 			    "Create a FAT32 filesystem (default)"},
-			{"", 0, false, "FAT16", "FAT Type 16",
+			{"", false, 0, "FAT16", "FAT Type 16",
 			    "Create a FAT16 filesystem"},
-			{"", 0, false, "FAT12", "FAT Type 12",
+			{"", false, 0, "FAT12", "FAT Type 12",
 			    "Create a FAT12 filesystem"},
 		};