Am 17/11/2023 um 13:12 schrieb Christoph Heiss:
> Happens due to a force-unwrap() under the false assumption that the
> disk for LVM configurations always exists when switching to a LVM
> filesystem.
> This fails spectacularly with a panic when switching from e.g. Btrfs to
> ext4 in the filesystem chooser.
> 
> Fixes: eda9fa0 ("fix #4856: tui: bootdisk: use correct defaults in advanced 
> dialog")
> Reported-by: Christian Ebner <c.eb...@proxmox.com>
> Signed-off-by: Christoph Heiss <c.he...@proxmox.com>
> ---
>  proxmox-tui-installer/src/views/bootdisk.rs | 29 +++++++++++----------
>  1 file changed, 15 insertions(+), 14 deletions(-)
> 
>

applied, thanks!

Could be nice to get the installer code unwrap free in the future, or
at least add a safety comment for each case (but preferably the former)


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to