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

AssumptionFailed { reason: "outside long mode, block group desc size must be zero, not 32" } #4

Open
surban opened this issue Sep 27, 2020 · 0 comments · May be fixed by #9
Open

AssumptionFailed { reason: "outside long mode, block group desc size must be zero, not 32" } #4

surban opened this issue Sep 27, 2020 · 0 comments · May be fixed by #9

Comments

@surban
Copy link

surban commented Sep 27, 2020

I am getting this error trying to read an ext4 filesystem extracted from an Android device. The filesystem mounts fine on a standard Linux kernel and fsck.ext4 reports no errors.

Commenting out the corresponding check in parse.rs allows ext4-rs to parse the filesystem and correctly extract files from it.

However I am afraid that we might introduce data corruption issues by simply skipping this check.

surban added a commit to ENQT-GmbH/ext4-rs that referenced this issue Jan 27, 2022
If set to true, the check that the block group descriptor size is zero
outside long mode is skipped. This is necessary to read some ext4
filesystems created by possibly buggy tools or kernels.

Fixes FauxFaux#4.
@surban surban linked a pull request Jan 27, 2022 that will close this issue
pirroman pushed a commit to passware/ext4-rs that referenced this issue Nov 22, 2022
…ых нод

Merge in CRATES/ext4-rs from cases/PKM-1243-generic-crypto to without_state_verification

* commit '0ac09052ce27fce6771836e454af1234d0771b4c':
  [PKM-1243] change panic behavior to throw errors
  [PKM-1243] use matches macro
  [PKM-1243] use trim
  [PKM-1243] use slice
  [PKM-1243] fixed errors
  [PKM-1243] clean code
  [PKM-1243] ref inner
  [PKM-1243] fixed decrypt page
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 a pull request may close this issue.

1 participant