]> git.proxmox.com Git - proxmox-backup.git/blobdiff - src/pxar/decoder.rs
switch from failure to anyhow
[proxmox-backup.git] / src / pxar / decoder.rs
index 5cec5095eca7b6c6b0504c54001be970c5deb9f6..2512adaf14e6c963b091b2b35f3207bac17689e7 100644 (file)
@@ -3,28 +3,47 @@
 //! This module contain the code to decode *pxar* archive files.
 
 use std::convert::TryFrom;
-use std::ffi::OsString;
+use std::ffi::{OsString, OsStr};
 use std::io::{Read, Seek, SeekFrom};
 use std::path::{Path, PathBuf};
+use std::os::unix::ffi::OsStrExt;
 
-use failure::*;
+use anyhow::{bail, format_err, Error};
 use libc;
 
+use super::binary_search_tree::search_binary_tree_by;
 use super::format_definition::*;
-use super::sequential_decoder::*;
+use super::sequential_decoder::SequentialDecoder;
+use super::match_pattern::MatchPattern;
 
 use proxmox::tools::io::ReadExt;
 
 pub struct DirectoryEntry {
+    /// Points to the `PxarEntry` of the directory
     start: u64,
+    /// Points past the goodbye table tail
     end: u64,
+    /// Filename of entry
     pub filename: OsString,
+    /// Entry (mode, permissions)
     pub entry: PxarEntry,
+    /// Extended attributes
+    pub xattr: PxarAttributes,
+    /// Payload size
+    pub size: u64,
+    /// Target path for symbolic links
+    pub target: Option<PathBuf>,
+    /// Start offset of the payload if present.
+    pub payload_offset: Option<u64>,
 }
 
+/// Trait to create ReadSeek Decoder trait objects.
+trait ReadSeek: Read + Seek {}
+impl <R: Read + Seek> ReadSeek for R {}
+
 // This one needs Read+Seek
-pub struct Decoder<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> {
-    inner: SequentialDecoder<R, F>,
+pub struct Decoder {
+    inner: SequentialDecoder<Box<dyn ReadSeek + Send>>,
     root_start: u64,
     root_end: u64,
 }
@@ -32,15 +51,17 @@ pub struct Decoder<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> {
 const HEADER_SIZE: u64 = std::mem::size_of::<PxarHeader>() as u64;
 const GOODBYE_ITEM_SIZE: u64 = std::mem::size_of::<PxarGoodbyeItem>() as u64;
 
-impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
-    pub fn new(mut reader: R, callback: F) -> Result<Self, Error> {
+impl Decoder {
+    pub fn new<R: Read + Seek + Send + 'static>(mut reader: R) -> Result<Self, Error> {
         let root_end = reader.seek(SeekFrom::End(0))?;
+        let boxed_reader: Box<dyn ReadSeek + 'static + Send> = Box::new(reader);
+        let inner = SequentialDecoder::new(boxed_reader, super::flags::DEFAULT);
+  
+        Ok(Self { inner, root_start: 0, root_end })
+    }
 
-        Ok(Self {
-            inner: SequentialDecoder::new(reader, super::flags::DEFAULT, callback),
-            root_start: 0,
-            root_end: root_end,
-        })
+    pub fn set_callback<F: Fn(&Path) -> Result<(), Error> + Send + 'static>(&mut self, callback: F ) {
+        self.inner.set_callback(callback);
     }
 
     pub fn root(&mut self) -> Result<DirectoryEntry, Error> {
@@ -48,11 +69,21 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
         let header: PxarHeader = self.inner.read_item()?;
         check_ca_header::<PxarEntry>(&header, PXAR_ENTRY)?;
         let entry: PxarEntry = self.inner.read_item()?;
+        let (header, xattr) = self.inner.read_attributes()?;
+        let (size, payload_offset) = match header.htype {
+            PXAR_PAYLOAD => (header.size - HEADER_SIZE, Some(self.seek(SeekFrom::Current(0))?)),
+            _ => (0, None),
+        };
+
         Ok(DirectoryEntry {
             start: self.root_start,
             end: self.root_end,
             filename: OsString::new(), // Empty
-            entry: entry,
+            entry,
+            xattr,
+            size,
+            target: None,
+            payload_offset,
         })
     }
 
@@ -65,17 +96,23 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
         self.root_end
     }
 
-    pub fn restore(&mut self, dir: &DirectoryEntry, path: &Path) -> Result<(), Error> {
+    /// Restore the subarchive starting at `dir` to the provided target `path`.
+    ///
+    /// Only restore the content matched by the MatchPattern `pattern`.
+    /// An empty Vec `pattern` means restore all.
+    pub fn restore(&mut self, dir: &DirectoryEntry, path: &Path, pattern: &Vec<MatchPattern>) -> Result<(), Error> {
         let start = dir.start;
-
         self.seek(SeekFrom::Start(start))?;
-
-        self.inner.restore(path, &Vec::new())?;
+        self.inner.restore(path, pattern)?;
 
         Ok(())
     }
 
-    fn read_directory_entry(&mut self, start: u64, end: u64) -> Result<DirectoryEntry, Error> {
+    pub(crate) fn read_directory_entry(
+        &mut self,
+        start: u64,
+        end: u64,
+    ) -> Result<DirectoryEntry, Error> {
         self.seek(SeekFrom::Start(start))?;
 
         let head: PxarHeader = self.inner.read_item()?;
@@ -99,41 +136,54 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
         }
         check_ca_header::<PxarEntry>(&head, PXAR_ENTRY)?;
         let entry: PxarEntry = self.inner.read_item()?;
+        let (header, xattr) = self.inner.read_attributes()?;
+        let (size, payload_offset, target) = match header.htype {
+            PXAR_PAYLOAD =>
+                (header.size - HEADER_SIZE, Some(self.seek(SeekFrom::Current(0))?), None),
+            PXAR_SYMLINK =>
+                (header.size - HEADER_SIZE, None, Some(self.inner.read_link(header.size)?)),
+            _ => (0, None, None),
+        };
 
         Ok(DirectoryEntry {
             start: entry_start,
-            end: end,
-            filename: filename,
+            end,
+            filename,
             entry,
+            xattr,
+            size,
+            target,
+            payload_offset,
         })
     }
 
-    pub fn list_dir(&mut self, dir: &DirectoryEntry) -> Result<Vec<DirectoryEntry>, Error> {
-        let start = dir.start;
-        let end = dir.end;
-
-        //println!("list_dir1: {} {}", start, end);
-
-        if (end - start) < (HEADER_SIZE + GOODBYE_ITEM_SIZE) {
-            bail!("detected short object [{}..{}]", start, end);
-        }
-
+    /// Return the goodbye table based on the provided end offset.
+    ///
+    /// Get the goodbye table entries and the start and end offsets of the
+    /// items they reference.
+    /// If the start offset is provided, we use that to check the consistency of
+    /// the data, else the start offset calculated based on the goodbye tail is
+    /// used.
+    pub(crate) fn goodbye_table(
+        &mut self,
+        start: Option<u64>,
+        end: u64,
+    ) -> Result<Vec<(PxarGoodbyeItem, u64, u64)>, Error> {
         self.seek(SeekFrom::Start(end - GOODBYE_ITEM_SIZE))?;
 
-        let item: PxarGoodbyeItem = self.inner.read_item()?;
-
-        if item.hash != PXAR_GOODBYE_TAIL_MARKER {
-            bail!(
-                "missing goodbye tail marker for object [{}..{}]",
-                start,
-                end
-            );
+        let tail: PxarGoodbyeItem = self.inner.read_item()?;
+        if tail.hash != PXAR_GOODBYE_TAIL_MARKER {
+            bail!("missing goodbye tail marker for object at offset {}", end);
         }
 
-        let goodbye_table_size = item.size;
+        // If the start offset was provided, we use and check based on that.
+        // If not, we rely on the offset calculated from the goodbye table entry.
+        let start = start.unwrap_or(end - tail.offset - tail.size);
+        let goodbye_table_size = tail.size;
         if goodbye_table_size < (HEADER_SIZE + GOODBYE_ITEM_SIZE) {
             bail!("short goodbye table size for object [{}..{}]", start, end);
         }
+
         let goodbye_inner_size = goodbye_table_size - HEADER_SIZE - GOODBYE_ITEM_SIZE;
         if (goodbye_inner_size % GOODBYE_ITEM_SIZE) != 0 {
             bail!(
@@ -144,13 +194,7 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
         }
 
         let goodbye_start = end - goodbye_table_size;
-
-        if item.offset != (goodbye_start - start) {
-            println!(
-                "DEBUG: {} {}",
-                u64::from_le(item.offset),
-                goodbye_start - start
-            );
+        if tail.offset != (goodbye_start - start) {
             bail!(
                 "wrong offset in goodbye tail marker for entry [{}..{}]",
                 start,
@@ -160,7 +204,6 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
 
         self.seek(SeekFrom::Start(goodbye_start))?;
         let head: PxarHeader = self.inner.read_item()?;
-
         if head.htype != PXAR_GOODBYE {
             bail!(
                 "wrong goodbye table header type for entry [{}..{}]",
@@ -173,11 +216,9 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
             bail!("wrong goodbye table size for entry [{}..{}]", start, end);
         }
 
-        let mut range_list = Vec::new();
-
+        let mut gb_entries = Vec::new();
         for i in 0..goodbye_inner_size / GOODBYE_ITEM_SIZE {
             let item: PxarGoodbyeItem = self.inner.read_item()?;
-
             if item.offset > (goodbye_start - start) {
                 bail!(
                     "goodbye entry {} offset out of range [{}..{}] {} {} {}",
@@ -194,13 +235,25 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
             if item_end > goodbye_start {
                 bail!("goodbye entry {} end out of range [{}..{}]", i, start, end);
             }
+            gb_entries.push((item, item_start, item_end));
+        }
+
+        Ok(gb_entries)
+    }
+
+    pub fn list_dir(&mut self, dir: &DirectoryEntry) -> Result<Vec<DirectoryEntry>, Error> {
+        let start = dir.start;
+        let end = dir.end;
+
+        //println!("list_dir1: {} {}", start, end);
 
-            range_list.push((item_start, item_end));
+        if (end - start) < (HEADER_SIZE + GOODBYE_ITEM_SIZE) {
+            bail!("detected short object [{}..{}]", start, end);
         }
 
         let mut result = vec![];
-
-        for (item_start, item_end) in range_list {
+        let goodbye_table = self.goodbye_table(Some(start), end)?;
+        for (_, item_start, item_end) in goodbye_table {
             let entry = self.read_directory_entry(item_start, item_end)?;
             //println!("ENTRY: {} {} {:?}", item_start, item_end, entry.filename);
             result.push(entry);
@@ -240,133 +293,73 @@ impl<R: Read + Seek, F: Fn(&Path) -> Result<(), Error>> Decoder<R, F> {
         Ok(())
     }
 
-    /// Get the `DirectoryEntry` located at `offset`.
+    /// Lookup the item identified by `filename` in the provided `DirectoryEntry`.
     ///
-    /// `offset` is expected to point to the directories `PXAR_GOODBYE_TAIL_MARKER`.
-    pub fn get_dir(&mut self, offset: u64) -> Result<DirectoryEntry, Error> {
-        self.seek(SeekFrom::Start(offset))?;
-
-        let gb: PxarGoodbyeItem = self.inner.read_item()?;
-        if gb.hash != PXAR_GOODBYE_TAIL_MARKER {
-            bail!("Expected goodbye tail marker, encountered 0x{:x?}", gb.hash);
-        }
-
-        let distance = i64::try_from(gb.offset + gb.size)?;
-        let start = self.seek(SeekFrom::Current(0 - distance))?;
-        let mut header: PxarHeader = self.inner.read_item()?;
-        let filename = if header.htype == PXAR_FILENAME {
-            let name = self.inner.read_filename(header.size)?;
-            header = self.inner.read_item()?;
-            name
-        } else {
-            OsString::new()
-        };
-        check_ca_header::<PxarEntry>(&header, PXAR_ENTRY)?;
-        let entry: PxarEntry = self.inner.read_item()?;
-
-        Ok(DirectoryEntry {
-            start,
-            end: offset + GOODBYE_ITEM_SIZE,
-            filename,
-            entry,
-        })
-    }
+    /// Calculates the hash of the filename and searches for matching entries in
+    /// the goodbye table of the provided `DirectoryEntry`.
+    /// If found, also the filename is compared to avoid hash collision.
+    /// If the filename does not match, the search resumes with the next entry in
+    /// the goodbye table.
+    /// If there is no entry with matching `filename`, `Ok(None)` is returned.
+    pub fn lookup(
+        &mut self,
+        dir: &DirectoryEntry,
+        filename: &OsStr,
+    ) -> Result<Option<DirectoryEntry>, Error> {
+        let gbt = self.goodbye_table(Some(dir.start), dir.end)?;
+        let hash = compute_goodbye_hash(filename.as_bytes());
+
+        let mut start_idx = 0;
+        let mut skip_multiple = 0;
+        loop {
+            // Search for the next goodbye entry with matching hash.
+            let idx = search_binary_tree_by(
+                start_idx,
+                gbt.len(),
+                skip_multiple,
+                |idx| hash.cmp(&gbt[idx].0.hash),
+            );
+            let (_item, start, end) = match idx {
+                Some(idx) => &gbt[idx],
+                None => return Ok(None),
+            };
 
-    /// Get attributes for the archive item located at `offset`.
-    ///
-    /// Returns the entry, attributes and the payload size for the item.
-    /// For regular archive itmes a `PXAR_FILENAME` or a `PXAR_ENTRY` header is
-    /// expected at `offset`.
-    /// For directories, `offset` might also (but not necessarily) point at the
-    /// directories `PXAR_GOODBYE_TAIL_MARKER`. This is not mandatory and it can
-    /// also directly point to its `PXAR_FILENAME` or `PXAR_ENTRY`, thereby
-    /// avoiding an additional seek.
-    pub fn attributes(&mut self, offset: u64) -> Result<(PxarEntry, PxarAttributes, u64), Error> {
-        self.seek(SeekFrom::Start(offset))?;
-
-        let mut marker: u64 = self.inner.read_item()?;
-        if marker == PXAR_GOODBYE_TAIL_MARKER {
-            let dir_offset: u64 = self.inner.read_item()?;
-            let gb_size: u64 = self.inner.read_item()?;
-            let distance = i64::try_from(dir_offset + gb_size)?;
-            self.seek(SeekFrom::Current(0 - distance))?;
-            marker = self.inner.read_item()?;
-        }
+            let entry = self.read_directory_entry(*start, *end)?;
 
-        if marker == PXAR_FILENAME {
-            let size: u64 = self.inner.read_item()?;
-            let _filename = self.inner.read_filename(size)?;
-            marker = self.inner.read_item()?;
-        }
-        if marker != PXAR_ENTRY {
-            bail!("Expected PXAR_ENTRY, found 0x{:x?}", marker);
+            // Possible hash collision, need to check if the found entry is indeed
+            // the filename to lookup.
+            if entry.filename == filename {
+                return Ok(Some(entry));
+            }
+            // Hash collision, check the next entry in the goodbye table by starting
+            // from given index but skipping one more match (so hash at index itself).
+            start_idx = idx.unwrap();
+            skip_multiple = 1;
         }
-        let _size: u64 = self.inner.read_item()?;
-        let entry: PxarEntry = self.inner.read_item()?;
-        let (header, xattr) = self.inner.read_attributes()?;
-        let file_size = match header.htype {
-            PXAR_PAYLOAD => header.size - HEADER_SIZE,
-            _ => 0,
-        };
-
-        Ok((entry, xattr, file_size))
     }
 
-    /// Opens the file by validating the given `offset` and returning its attrs,
-    /// xattrs and size.
-    pub fn open(&mut self, offset: u64) -> Result<(PxarEntry, PxarAttributes, u64), Error> {
-        self.attributes(offset)
-    }
-
-    /// Read the payload of the file given by `offset`.
+    /// Read the payload of the file given by `entry`.
     ///
-    /// This will read the file by first seeking to `offset` within the archive,
-    /// check if there is indeed a valid item with payload and then read `size`
-    /// bytes of content starting from `data_offset`.
-    /// If EOF is reached before reading `size` bytes, the reduced buffer is
-    /// returned.
-    pub fn read(&mut self, offset: u64, size: usize, data_offset: u64) -> Result<Vec<u8>, Error> {
-        self.seek(SeekFrom::Start(offset))?;
-
-        let head: PxarHeader = self.inner.read_item()?;
-        check_ca_header::<PxarEntry>(&head, PXAR_ENTRY)?;
-        let _: PxarEntry = self.inner.read_item()?;
-
-        let (header, _) = self.inner.read_attributes()?;
-        if header.htype != PXAR_PAYLOAD {
-            bail!("Expected PXAR_PAYLOAD, found 0x{:x?}", header.htype);
-        }
-
-        let payload_size = header.size - HEADER_SIZE;
-        if data_offset >= payload_size {
+    /// This will read a files payload as raw bytes starting from `offset` after
+    /// the payload marker, reading `size` bytes.
+    /// If the payload from `offset` to EOF is smaller than `size` bytes, the
+    /// buffer with reduced size is returned.
+    /// If `offset` is larger than the payload size of the `DirectoryEntry`, an
+    /// empty buffer is returned.
+    pub fn read(&mut self, entry: &DirectoryEntry, size: usize, offset: u64) -> Result<Vec<u8>, Error> {
+        let start_offset = entry.payload_offset
+            .ok_or_else(|| format_err!("entry has no payload offset"))?;
+        if offset >= entry.size {
             return Ok(Vec::new());
         }
-
-        let len = if data_offset + u64::try_from(size)? > payload_size {
-            usize::try_from(payload_size - data_offset)?
+        let len = if u64::try_from(size)? > entry.size {
+            usize::try_from(entry.size)?
         } else {
             size
         };
-        self.inner.skip_bytes(usize::try_from(data_offset)?)?;
+        self.seek(SeekFrom::Start(start_offset + offset))?;
         let data = self.inner.get_reader_mut().read_exact_allocated(len)?;
 
         Ok(data)
     }
-
-    /// Read the target of a hardlink in the archive.
-    pub fn read_link(&mut self, offset: u64) -> Result<(PathBuf, PxarEntry), Error> {
-        self.seek(SeekFrom::Start(offset))?;
-
-        let mut header: PxarHeader = self.inner.read_item()?;
-        check_ca_header::<PxarEntry>(&header, PXAR_ENTRY)?;
-        let entry: PxarEntry = self.inner.read_item()?;
-
-        header = self.inner.read_item()?;
-        if header.htype != PXAR_SYMLINK {
-            bail!("Expected PXAR_SYMLINK, encountered 0x{:x?}", header.htype);
-        }
-        let target = self.inner.read_link(header.size)?;
-
-        Ok((target, entry))
-    }
 }