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

Unable to rename symbol a structure field points to #4240

Closed
joelreymont opened this issue Apr 19, 2023 · 8 comments
Closed

Unable to rename symbol a structure field points to #4240

joelreymont opened this issue Apr 19, 2023 · 8 comments
Labels
Component: UI Issue needs changes to the user interface Impact: Low Issue is a papercut or has a good, supported workaround State: Awaiting Triage Issue is waiting for more in-depth triage from a developer UI: Linear Issues with the Linear view
Milestone

Comments

@joelreymont
Copy link

Version and Platform (required):

  • Binary Ninja Version: 3.4.4242-dev, 2e3a4286
  • OS: macos
  • OS Version: 13.3
  • CPU Architecture: arm64

Bug Description:

I'm looking at an array of structures. One of the fields is a pointer to a func. I select the func name in the field and pick rename symbol. I enter a new name and nothing happens. It throws me back to the beginning of the array and does nothing visible. It sure does something invisible, though, because Undo works. The same thing works if I click through to the func and rename it there.

This is very annoying as I'm going through an array of 200 structures and clicking through to the func to rename and then ESC to go back returns me to the beginning of the array. Then I have to page down to find the next record.

@joelreymont
Copy link
Author

It works fine when renaming function names in code.

@negasora
Copy link
Member

Do you see the updated name in the symbol list and/or does switching ILs show the updated name? It could just be linear view missing an update.

@joelreymont
Copy link
Author

No, that's not it. The symbol list does not update, clicking through to the function does not show updated name. Switching IL does not show updated name either. It's like the operation does nothing without reporting an error.

@joelreymont
Copy link
Author

Here's a screenshot. I'm trying to rename by clicking on the contents of the handler field (the sub... values) and pressing 'N'.

Screenshot 2023-04-19 at 17 15 54

@psifertex
Copy link
Member

Hmm, it's not generally broken as I can reproduce the rename working fine here for a function pointer inside an array of structs. I do see that the focus goes to the top of the array when the rename is done which is annoying, but the rename definitely does take effect. Can you DM me the BNDB via slack?

@joelreymont
Copy link
Author

joelreymont commented Apr 21, 2023

Same issue trying to rename a function pointers.

Apparently, this is a broader issue than what I described above.

000102ed8  void* data_102ed8 = 0x10003cf
000102edc  void* data_102edc = 0xff00ffff
000102ee0  void* data_102ee0 = sub_b441c
000102ee4  void* data_102ee4 = 0x10003c3
000102ee8  void* data_102ee8 = 0xff00ffff
000102eec  void* data_102eec = sub_b4370

@fuzyll fuzyll added Component: UI Issue needs changes to the user interface UI: Linear Issues with the Linear view Impact: Low Issue is a papercut or has a good, supported workaround State: Awaiting Triage Issue is waiting for more in-depth triage from a developer labels May 17, 2023
@fuzyll
Copy link
Contributor

fuzyll commented May 19, 2023

We have the binary above and were able to reproduce the issue. Anyone from V35 should search for "Encourage Salesman Prompt Delay" to find the database.

@plafosse
Copy link
Member

Root cause of this issue is that we weren't properly masking the thumb bit for these symbol tokens.

Should be fixed in 3.5.4305

@plafosse plafosse added this to the Coruscant milestone Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: UI Issue needs changes to the user interface Impact: Low Issue is a papercut or has a good, supported workaround State: Awaiting Triage Issue is waiting for more in-depth triage from a developer UI: Linear Issues with the Linear view
Projects
None yet
Development

No branches or pull requests

5 participants