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

update to build on Yosemite and run on iOS 8 #16

Closed
wants to merge 10 commits into from

Conversation

100apps
Copy link
Contributor

@100apps 100apps commented Jul 2, 2015

Keychain-Dumper is an awesome tool for jailbreaked iOS devices. I made some changes to build on Yosemite and run on iOS 8.

  1. changed Makefile to build more easily
  2. fixed #7
  3. add a build and run gif demo in Readme, for new users.

Thank you @ptoomey3


main.o: main.m
$(GCC_ARM) -c main.m

clean:
rm -f keychain_dumper *.o

list:
security find-identity -pcodesigning -v
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not super familiar with find-identity, but it looks like -v will only return "valid" code signing certificates. That seems to exclude self-signed certificates (as are used in the rest of the Readme). I'm assuming that self-signed still works fine (give this is being run on a jail broken device), right? If so, can you update this to remove the-v?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested on my devices, self-signed certificates do work.

@ptoomey3
Copy link
Owner

Thanks so much for the build update. Once this is ready to merge, I'd like to build it locally, sign it with a self signed cert, and update the binary that is included in the repo. But, I currently have no jailbroken device to test it with. Would you be willing to test it? If not, I totally understand 😄.

@ptoomey3
Copy link
Owner

Also, I can update the Readme to use the new make build targets after we merge this.

@100apps
Copy link
Contributor Author

100apps commented Jul 14, 2015

@ptoomey3, I have more than 5 jailbroken devices, ready for your tests. 😄

@nderkach
Copy link

Works for me with iOS 8.4 built on 10.10.4.

@ptoomey3
Copy link
Owner

Cool. I'm going to clean up a few things and get this merged in the next few days.

@ptoomey3 ptoomey3 mentioned this pull request Jul 20, 2015
@ptoomey3
Copy link
Owner

Closed in preference to #17

@ptoomey3 ptoomey3 closed this Jul 20, 2015
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.

Error with Compiling the Keychain Dumper
3 participants