[pve-devel] [PATCH installer] fix space calculation for small disks for pve product

Stoiko Ivanov s.ivanov at proxmox.com
Thu Jun 22 15:57:13 CEST 2023


The convoluted calculation logic in case the disks is 8GB leads to
datasize becoming 16EiB further down:
* after calculating and removing the rootsize from $rest, $rest becomes
  smaller than $space (which should be the minimal non-used space in the
  volume-group) - this leads to a negative value, which overflows in
  the `& ~0xFFF` opration.

Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
---
tested in a VM with an 8GB disk

 Proxmox/Install.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Proxmox/Install.pm b/Proxmox/Install.pm
index 7970f83..28add10 100644
--- a/Proxmox/Install.pm
+++ b/Proxmox/Install.pm
@@ -425,7 +425,7 @@ sub create_lvm_volumes {
 	my $rootsize_mb;
 	if ($rest_mb < 12 * 1024) {
 	    # no point in wasting space, try to get us actually installed and align down to 4 MB
-	    $rootsize_mb = ($rest_mb - 0.1) & ~3;
+	    $rootsize_mb = ($rest_mb - 4) & ~3;
 	} elsif ($rest_mb < 48 * 1024) {
 	    my $masked = int($rest_mb / 2) & ~3; # align down to 4 MB
 	    $rootsize_mb = $masked;
-- 
2.30.2






More information about the pve-devel mailing list