Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a method to write a link name without canonicalization #274

Merged
merged 1 commit into from Dec 14, 2021

Commits on Dec 14, 2021

  1. Add a method to write a link name without canonicalization

    In https://github.com/ostreedev/ostree we generate a cryptographic
    checksum over files and symlinks, and directories.
    
    ostree does not currently perform any canonicalization on symlinks;
    we'll respect and honor whatever bytes we're provided as input,
    and replicate that on the target.
    
    I'm using this crate to do tar serialization, which has so far worked
    fine...except, I hit this corner case:
    
    ```
    [root@cosa-devsh ~]# rpm -qf /usr/lib/systemd/systemd-sysv-install
    chkconfig-1.13-2.el8.x86_64
    [root@cosa-devsh ~]# ll /usr/lib/systemd/systemd-sysv-install
    lrwxrwxrwx. 2 root root 24 Nov 29 18:08 /usr/lib/systemd/systemd-sysv-install -> ../../..//sbin/chkconfig
    [root@cosa-devsh ~]#
    ```
    
    But, using `set_link_name` to write the tarball, we end up with
    the canonicalized path `../../../sbin/chkconfig` - i.e. without the
    double `//`.  This breaks the checksum.
    
    Now, I am a bit tempted to change ostree to do canonicalization.  But
    even if we did, I'd need to *exactly* match what this crate is doing.
    
    In the end, what I think is useful here is a method which skips
    the canonicalization and internal error checking and just writes
    out into the tar stream what's requested.
    
    (I may of course also try to change the rhel8 systemd package, but
     that's going to take a while to propagate and this corner case isn't
     the only one I'm sure)
    cgwalters committed Dec 14, 2021
    Copy the full SHA
    0c15a30 View commit details
    Browse the repository at this point in the history