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.

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

le: Fix segv in le.c #3927

Merged
merged 1 commit into from
Oct 19, 2023
Merged

le: Fix segv in le.c #3927

merged 1 commit into from
Oct 19, 2023

Conversation

ttxine
Copy link
Contributor

@ttxine ttxine commented Oct 17, 2023 •
edited
Loading

Your checklist for this pull request

  • I've read the guidelines for contributing to this repository
  • I made sure to follow the project's coding style
  • I've documented or updated the documentation of every function and struct this PR changes. If not so I've explained why.
  • I've added tests that prove my fix is effective or that my feature works (if possible)
  • I've updated the rizin book with the relevant information (if needed)

Detailed description

Indexes of some object page table entries may be out of range.

Test plan

CI is green

@github-actions github-actions bot added the RzBin label Oct 17, 2023
@ttxine ttxine force-pushed the fix-le-segfault branch 2 times, most recently from 24f6879 to b4e0fdd Compare October 17, 2023 13:39
@ttxine ttxine closed this Oct 17, 2023
@ttxine ttxine reopened this Oct 18, 2023
@ttxine ttxine changed the title le: Fix segfault in le.c le: Fix segv in le.c Oct 18, 2023
@ttxine ttxine marked this pull request as ready for review October 18, 2023 13:23
wargio
wargio reviewed Oct 18, 2023
View reviewed changes
librz/bin/format/le/le.c Outdated Show resolved Hide resolved
@wargio
Copy link
Member

wargio commented Oct 19, 2023

please rebase

@XVilka XVilka merged commit 24bfd59 into rizinorg:dev Oct 19, 2023
44 checks passed
@ttxine ttxine deleted the fix-le-segfault branch October 19, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wargio wargio wargio left review comments

@XVilka XVilka XVilka approved these changes

@ret2libc ret2libc Awaiting requested review from ret2libc ret2libc is a code owner

Assignees
No one assigned
Labels
RzBin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ttxine @wargio @XVilka