-
-
Notifications
You must be signed in to change notification settings - Fork 367
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
Cherry-picks for 0.6.2 release #3796
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Fix memleak in rz_il_handler_empty() * Fix memleak in compare_op() * Fix memleak in assign_reg() * Fix memleak in rz_asm_op_fini() * Fix memleak in rz_core_analysis_bytes() * Fix memleak in add_sub_op() * Fix memleak in bitwise_op() * Fix memleak in shift_and_rotate()
This is a cherry pick of [1]. The removed mappings overwrite previous entries and cannot be found in [2]. [1] radareorg/radare2@c67c47e [2] https://github.com/torvalds/linux/blob/master/include/uapi/asm-generic/unistd.h Signed-off-by: Marcin Sobczyk <[email protected]>
'rizin' is missing a lot of syscalls for i.e. arm. We can check that with: rizin -a x86_64 -b 64 -k linux -q -c asl vs rizin -a arm -b 64 -k linux -q -c asl 'rz_syscall_item_new_from_string' splits the incoming string by commas and expects at least 3 items as a result. This patch fixes syscall preprocessing by appending empty strings to match the expected number of items after a split. This is a clone of a fix posted to radare2 [1]. [1] radareorg/radare2#21508
* Add test for pointer read command `*` Test for `asm.bits=8` uses `asm.arch=6502` because `asm.bits=8` is unsupported for the default arch. * Fix pointer command `*` broken since move from `?` to `%` ##shell * Refactor to use appropriate existing API in `num_callback`
XVilka
requested review from
ret2libc,
wargio,
thestr4ng3r and
kazarmy
as code owners
August 27, 2023 03:35
Looks like only missing things are remaining |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
ret2libc
reviewed
Aug 28, 2023
…shell (#3794) * Update the golang analysis tests to seek to the main function before dumping strings, which was previously done as a side effect of using `@@f` during auto-analysis.
* Test whether there is error on `rizin -d` * Make env more like local env * Add space btw `env` and its arg * rz-run resolve_value: Pass single-quoted strings unaltered
XVilka
force-pushed
the
dist-asan-fuzz-0.6.2-cherrypicks
branch
from
August 29, 2023 04:26
b9fcc68
to
a2f59f0
Compare
* Fix memleak in rz_seek_history_list_handler() * Fix memleaks in rz_eval_getset_handler() * Fix memleaks in visual_comma() * Fix memleak in cmd_pxr() * Fix memleak in arged_stmt
… available in the 13.2 serie (#3820)
wargio
approved these changes
Sep 4, 2023
Something is really wrong with NetBSD (happens also on
The crux of the problem seems to be missing |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Your checklist for this pull request
Detailed description
Bugfixes and documentation updates
Test plan
CI is green