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

Darwin strip is evil and can modify files even if they aren't valid Mach-O #22520

Closed
copumpkin opened this issue Feb 7, 2017 · 3 comments
Closed
Labels
0.kind: bug Something is broken

Comments

@copumpkin
Copy link
Member

I just spent some time tracking down a super weird bug because the standard fixupPhase was corrupting a .class file. Turns out that if you run strip -S in cctools against a .class file, it'll complain that it's not a valid Mach-O file but flip the endianness of the magic number at the start of the file anyway, thus making it an invalid Java class file.

This will hurt us if we ever package things that contain raw class files not in jars.

Mostly leaving this as a reminder for myself to fix, but if someone else wants to look into it, please do!

@copumpkin copumpkin added the 0.kind: bug Something is broken label Feb 7, 2017
@vcunat
Copy link
Member

vcunat commented Feb 7, 2017

Maybe #15339.

@mmahut
Copy link
Member

mmahut commented Aug 11, 2019

Are there any updates to this issue, please?

@prusnak
Copy link
Member

prusnak commented May 8, 2020

No reaction. Closing. Please reopen if needed.

@prusnak prusnak closed this as completed May 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken
Projects
None yet
Development

No branches or pull requests

4 participants