Skip to content

fix: ensure safe extraction of radare2 tarball#479

Open
qwq233 wants to merge 1 commit intokavishdevar:mainfrom
qwq233:main
Open

fix: ensure safe extraction of radare2 tarball#479
qwq233 wants to merge 1 commit intokavishdevar:mainfrom
qwq233:main

Conversation

@qwq233
Copy link

@qwq233 qwq233 commented Mar 13, 2026

On some AOSP builds, the default 'tar' behavior is inconsistent and
may attempt to overwrite the permissions of the destination root
directory. This often leads to permission denied errors or command
failures when extracting as root.

This commit adds '--no-same-owner', '-m', and '-p' flags to mitigate
these issues. Specifically, it prevents tar from attempting to preserve
original ownership and ensures that file modification times and
permissions are handled more predictably during extraction to the
target directory.

Signed-off-by: qwq233 qwq233@qwq2333.top

03-13 13:55:31.609  3206  8609 E RadareOffsetFinder: Extract error: tar: chown 1000:1000 './': Read-only file system
03-13 13:55:31.609  3206  8609 E RadareOffsetFinder: Extract error: tar: settime 1744014133 ./: Read-only file system
03-13 13:55:31.609  3206  8609 E RadareOffsetFinder: Extract error: tar: had errors
03-13 13:55:31.609  3206  8609 E RadareOffsetFinder: Extraction failed with exit code 1

On some AOSP builds, the default 'tar' behavior is inconsistent and
may attempt to overwrite the permissions of the destination root
directory. This often leads to permission denied errors or command
failures when extracting as root.

This commit adds '--no-same-owner', '-m', and '-p' flags to mitigate
these issues. Specifically, it prevents tar from attempting to preserve
original ownership and ensures that file modification times and
permissions are handled more predictably during extraction to the
target directory.

Signed-off-by: qwq233 <qwq233@qwq2333.top>
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.

1 participant