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

Some default panels broken #3647

Closed
imyxh opened this issue Jul 8, 2023 · 5 comments · Fixed by #3690
Closed

Some default panels broken #3647

imyxh opened this issue Jul 8, 2023 · 5 comments · Fixed by #3690
Assignees
Milestone

Comments

@imyxh
Copy link

imyxh commented Jul 8, 2023

Work environment

Questions Answers
OS/arch/bits (mandatory) Linux 6.4.1-arch2-1 x86_64
File format of the file you reverse (mandatory) any
Architecture/bits of the file (mandatory) any
rizin -v full output, not truncated (mandatory) rizin 0.6.0 @ linux-x86-64 (not printed: commit 83e40b0)

Expected behavior

The default panels in visual panels mode should work.

Actual behavior

"Stack" and any other panels using px commands are broken for some reason. The default command for "Stack" is pxq 256@r:SP which runs fine on its own but not in the panel. Changing the command (using the " key) to pxq 256@r:SP again makes the window say pxq 256@r:SP (xc 256@r:SP) which seems to work fine but also prints the results of xc, not pxq. Changing the command to xc 256@r:SP works as intended.

The "Registers" command is out of date and should be %== true `e cfg.debug`; %! dr; %% ar. Changing it to this works fine.

Steps to reproduce the behavior

Open anything with -d and check V!

Additional Logs, screenshots, source code, configuration dump, ...

image

imyxh added a commit to imyxh/rizin that referenced this issue Jul 8, 2023
`?==` is now deprecated, and `px` doesn't work in panels (see rizinorg#3647)
@imyxh
Copy link
Author

imyxh commented Jul 8, 2023

I can submit a PR for the quick fix at imyxh@755535f if you wish but it makes more sense to figure out what's going on with px first

wargio added a commit that referenced this issue Jul 26, 2023
@wargio wargio mentioned this issue Jul 26, 2023
5 tasks
@wargio
Copy link
Member

wargio commented Jul 26, 2023

could you test my fix?

@imyxh
Copy link
Author

imyxh commented Jul 27, 2023

could you test my fix?

Works fine for me! It's blank when not in debug mode whereas the old code did ar but I don't think it matters much. It's probably nicer to have the command shorter like this.

That being said, although changing the default panels fixes the issue at a glance (already did this in imyxh@755535f) I'm still worried about why px doesn't work and if that's pointing to some deeper problem.

@wargio
Copy link
Member

wargio commented Jul 27, 2023

the reason why it doesn't work using px is because the default output is xc not px.
All the panels should be ported to use the C code and many things should be removed from there.

@XVilka
Copy link
Member

XVilka commented Jul 27, 2023

the reason why it doesn't work using px is because the default output is xc not px. All the panels should be ported to use the C code and many things should be removed from there.

Right, a bunch of code can be shared between "classic" visual mode and panels: #384

wargio added a commit that referenced this issue Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants