Skip to content

"ERROR: Found overlapping memory" on encrypted binary #210

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.

Sign up for GitHub

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

Jump to bottom
Open
richard9999999999 opened this issue Feb 27, 2025 · 8 comments · May be fixed by #207
Open

"ERROR: Found overlapping memory" on encrypted binary #210

richard9999999999 opened this issue Feb 27, 2025 · 8 comments · May be fixed by #207

Comments

@richard9999999999
Copy link

If I run:
picotool info -all ../build/hello_serial_enc.uf2

I am getting following error:

File ../build/hello_serial_enc.uf2 family ID 'rp2350-arm-s':

ERROR: Found overlapping memory ranges 0x20000000->0x20003d5c and 0x20000000->20000014

I tried that with hello_serial_enc from pico examples and also with my own encrypted binary
@kilograham
Copy link
Contributor

What compiler are you using

@richard9999999999
Copy link
Author

richard9999999999 commented Feb 27, 2025 •
edited
Loading

GCC 10.3.1 on pico site, some MinGW on picotool. I can see the problem also with Windows X64 binaries from "pico-sdk-tools" project.

@richard9999999999
Copy link
Author

Plus I am completely on Windows (if that matters)..

@will-v-pi
Copy link
Contributor

This is fixed in #207

@will-v-pi will-v-pi linked a pull request Feb 28, 2025 that will close this issue
Add support for creating self-decrypting binaries, and use 4-way AES key shares instead of just the AES key #207
Open
@richard9999999999
Copy link
Author

I confirm that I don't see the problem anymore using picotool compiled from branch "picotool-encrypted-shares". Thank you.

will-v-pi added a commit that referenced this issue Mar 4, 2025
@will-v-pi
Better fix for #210
6dbc22d 
Instead of emptying the range_map, just overwrite overlapping entries, like the bootrom would do when loading the load_map
will-v-pi added a commit that referenced this issue Apr 1, 2025
@will-v-pi
Better fix for #210
ab1a9ed 
Instead of emptying the range_map, just overwrite overlapping entries, like the bootrom would do when loading the load_map
will-v-pi added a commit that referenced this issue Apr 15, 2025
@will-v-pi
Better fix for #210
6502813 
Instead of emptying the range_map, just overwrite overlapping entries, like the bootrom would do when loading the load_map
will-v-pi added a commit that referenced this issue Apr 22, 2025
@will-v-pi
Better fix for #210
8e9eaba 
Instead of emptying the range_map, just overwrite overlapping entries, like the bootrom would do when loading the load_map
@richard9999999999
Copy link
Author

I've reported few bugs related to encrypted binaries recently. Now most of them are solved in DEVELOP with exception of this one. Could this fix be also merged to DEVELOP so we will have a picotool version which works with enctypted binaries?

@will-v-pi
Copy link
Contributor

This fix is part of a larger PR which still needs more work, so you will need to wait for that PR to be merged, or cherry-pick the relevant commits from that PR. This only causes issues when reading info from encrypted binaries, so you can still compile and use encrypted binaries without this fix.

@richard9999999999
Copy link
Author

OK I see.. I guessed the problem is trivial to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@kilograham @richard9999999999 @will-v-pi