Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 🔔

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants