]> git.proxmox.com Git - proxmox-backup.git/commitdiff
close #5571: client: fix regression for `map` command
authorChristian Ebner <c.ebner@proxmox.com>
Tue, 2 Jul 2024 07:24:14 +0000 (09:24 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Tue, 2 Jul 2024 08:41:38 +0000 (10:41 +0200)
Commit 08fe5052 introduced functionality to mount split pxar archives
(sharing code with the map command), moving the manifest lookup
exclusive to fixed index archives.

However, the lookup now uses the incorrect archive name, not
containing the `.fidx` extension, which is however required for the
lookup in the manifest.

Fix the issue by calling the method with the correct server archive
name including the required extension.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
Tested-by: Gabriel Goller <g.goller@proxmox.com>
Reviewed-by: Gabriel Goller <g.goller@proxmox.com>
Fixes: 08fe5052 ("client: mount: make split pxar archives mountable")
[FG: reworded, add proper "Fixes:" trailer.]
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
proxmox-backup-client/src/mount.rs

index d96690839f87ce35e9f5d5c997cf88b52fda2386..538a40e68abd5819d67c86c6c4c81b2fc6d14d96 100644 (file)
@@ -302,7 +302,7 @@ async fn mount_do(param: Value, pipe: Option<OwnedFd>) -> Result<Value, Error> {
             }
         }
     } else if server_archive_name.ends_with(".fidx") {
-        let file_info = manifest.lookup_file_info(archive_name)?;
+        let file_info = manifest.lookup_file_info(&server_archive_name)?;
         let index = client
             .download_fixed_index(&manifest, &server_archive_name)
             .await?;