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

FileVault, recovery keys and more #21

Merged
merged 14 commits into from
Mar 28, 2018
Merged

FileVault, recovery keys and more #21

merged 14 commits into from
Mar 28, 2018

Conversation

puccia
Copy link

@puccia puccia commented Mar 19, 2018

Hi Simon!

Thank you very much for your apfs-fuse work!

I wanted to be able to use it to mount encrypted APFS volumes that have a recovery key, so I started implementing that functionality; then I also fixed some bits that I encountered as I went on.

The main changes in these commits are:

  1. Allow mounting of FileVault-encrypted APFS volumes, as opposed to regular encrypted volumes;
  2. Allow decryption through a (personal, not institutional) recovery key, as opposed to a regular password;
  3. Fix decryption when an extent's crypto_id does not match the block number;
  4. Fix reading fragmented files;
  5. Fix decompression for several algos, especially the handling of uncompressed blocks;
  6. Provide a "strict" mode that returns errors when decompression fails;
  7. Minor enhancements (allow spaces in passwords; allow specifying a container offset, which is handy when working with raw disk images).

I'd very much like to have this functionality integrated upstream. Would you consider that? Is there something else you'd like me to do? I'll do my best to meet your requirements for that (e.g. style changes, split PRs, different code, you name it.)

Thanks again!

@sgan81 sgan81 merged commit b69c458 into sgan81:master Mar 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants