diff options
author | John Baldwin <jhb@FreeBSD.org> | 2023-10-16 22:13:31 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2024-01-04 23:43:33 +0000 |
commit | c81bdb4998683aa6b795b6e1cb294be6f493329d (patch) | |
tree | 03d7a52e113f22f062c43c607d68bb4eb337edbd | |
parent | 70f7bf34e3b4d0934c8c38ff49852eb6be343ac5 (diff) | |
download | src-c81bdb4998683aa6b795b6e1cb294be6f493329d.tar.gz src-c81bdb4998683aa6b795b6e1cb294be6f493329d.zip |
bsdinstall partedit: Apply changes from scripted installs
I got a check inverted in a previous cleanup commit and as a result
partedit was only applying GEOM changes (and generating an /etc/fstab)
if it got an error reading the current GEOM mesh. Instead, it needed
to do those actions if it succeeded in reading the mesh.
The lack of /etc/fstab meant that bsdinstall mount didn't mount
anything in a scripted install.
PR: 273723
Reported by: Andrey Fesenko <andrey@bsdnir.info>
Reported by: Michal Nowak <mnowak@startmail.com>
Reviewed by: cognet, brooks
Fixes: 230990991965 bsdinstall: Handle errors from geom_gettree.
MFC after: 3 days
Differential Revision: https://reviews.freebsd.org/D42236
(cherry picked from commit 5307bbcc038f878b4b3714f03a2c824a0caeba4f)
-rw-r--r-- | usr.sbin/bsdinstall/partedit/partedit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/bsdinstall/partedit/partedit.c b/usr.sbin/bsdinstall/partedit/partedit.c index 0f341cbaba12..ffc809507b08 100644 --- a/usr.sbin/bsdinstall/partedit/partedit.c +++ b/usr.sbin/bsdinstall/partedit/partedit.c @@ -210,7 +210,7 @@ main(int argc, const char **argv) if (prompt == NULL) { error = geom_gettree(&mesh); - if (error != 0) { + if (error == 0) { if (validate_setup()) { error = apply_changes(&mesh); } else { |