Replace Ram statements #10
Comments
|
Maybe better to label the address. If this is an hardcoded address which is
used, vetter to call it a label
…On Sun, Aug 18, 2019, 18:59 radare ***@***.***> wrote:
uRam0000000100005658 = 1;
into
[0x100005658] = 1;
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#10>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEZ7M4TKFV3EXWAO4WQCQL3QFFWXBANCNFSM4IMTMKLQ>
.
|
|
For me a label is a pointer not a dereferenced memory address. It reads wrong to me. But still if we want to use them as variables we should soecify its type as global and use a shorter name. Despite this be coming from ghidra
… On 18 Aug 2019, at 18:02, Itay Cohen ***@***.***> wrote:
Maybe better to label the address. If this is an hardcoded address which is
used, vetter to call it a label
On Sun, Aug 18, 2019, 18:59 radare ***@***.***> wrote:
> uRam0000000100005658 = 1;
>
> into
>
> [0x100005658] = 1;
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#10>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEZ7M4TKFV3EXWAO4WQCQL3QFFWXBANCNFSM4IMTMKLQ>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
|
Ideally it should be something like And if there is a flag, it will be displayed |
|
yeah that lgtm too
… On 18 Aug 2019, at 18:57, Florian Märkl ***@***.***> wrote:
Ideally it should be something like
*((uint64_t *)0x100005658) = 1;
And if there is a flag, it will be displayed
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#10>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAG75FSIFQ4Q3CWVPLYSZKLQFF5QNANCNFSM4IMTMKLQ>.
|
|
Fixed by 3862c8b |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
into
The text was updated successfully, but these errors were encountered: