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

Fix issue where ToUintE() fails to parse uint bigger than math.MaxInt64 #155

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

blacktop
Copy link

This prevented me from using 0x9b199b1c002b4b00 as a cmd.Flags().Uint64 (which would return 0 as the value)

ref spf13/cobra#1791

@CLAassistant
Copy link

CLAassistant commented Aug 31, 2022

CLA assistant check
All committers have signed the CLA.

@blacktop
Copy link
Author

Ping

@blacktop
Copy link
Author

blacktop commented Dec 5, 2022

ping ping 🔔 this prevents people from using kernel addresses as arguments to disassemblers 😩

❯ ipsw macho disass test-caches/IPSWs/20C5058d__iPhone15,2/kernelcache.release.iPhone15,2 -a 0xFFFFFFF008493FE0 -V --fileset-entry com.apple.AGXFirmwareKextG15P_A0RTBuddy

doesn't work because it thinks I gave it 0 as the virtual address

@blacktop
Copy link
Author

adding another ping ping 🔔

@blacktop
Copy link
Author

blacktop commented Jun 6, 2024

adding another ping ping ping 🔔

@blacktop
Copy link
Author

blacktop commented Jun 6, 2024

another dup is #213 #175 #178

@ionrover2
Copy link

BUMP! NEED!

@blacktop
Copy link
Author

blacktop commented Aug 7, 2024

adding a ping-o-ling-o-ding-dong 🔔

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.

3 participants