DynamoRIO / DynamoRIO/dynamorio

Update and extend existing INSTR_CREATE_nop macros

Open
#3,877 9 comments 0 reactions 0 assignees View on GitHub
Component-IR Component-Tests OpSys-x64 OpSys-x86
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 18h
Merged PRs (30d)
30

Description

xref #3828

This issue covers and requests updating the multi-byte nops INSTR_CREATE macros with the byte sequences as recently added by #3868

e.g. update instr_create_nbyte_nop(), INSTR_CREATE_RAW_nop2byte() etc, and add versions up to INSTR_CREATE_RAW_nop9byte()

This issue also covers adding a test to api.ir for the new macros.

Contributor guide

Open the contributing guide

Research direction

Start by reading the existing INSTR_CREATE_nop macros, then compare their byte sequences with the versions added by #3868. Inspect api.ir for the current API tests. Done means updating the existing multi-byte nop macros, adding INSTR_CREATE_RAW_nop9byte() and the intervening versions, and adding coverage for the new macros.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
devtools
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.