diff options
| author | van Hauser <vh@thc.org> | 2021-04-16 11:32:37 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-16 11:32:37 +0200 |
| commit | 72a4bc703996481d6f471b1729ccf04b0dd7840f (patch) | |
| tree | d0ec8d2b2bc972092de0595b415d02a6720e4b2b /frida_mode/src | |
| parent | e41d1183cca02fb4d6398df4fc3e028dfd9c5f72 (diff) | |
| parent | 950648c513d031b5c829565255a7c157a33ba7c3 (diff) | |
| download | afl++-72a4bc703996481d6f471b1729ccf04b0dd7840f.tar.gz | |
Merge pull request #876 from WorksButNotTested/dev-frida-fixes
Changes following code review
Diffstat (limited to 'frida_mode/src')
| -rw-r--r-- | frida_mode/src/instrument.c | 8 | ||||
| -rw-r--r-- | frida_mode/src/ranges.c | 3 |
2 files changed, 8 insertions, 3 deletions
diff --git a/frida_mode/src/instrument.c b/frida_mode/src/instrument.c index 042fdab8..22910062 100644 --- a/frida_mode/src/instrument.c +++ b/frida_mode/src/instrument.c @@ -174,7 +174,13 @@ void instrument_coverage_optimize(const cs_insn * instr, static void on_basic_block(GumCpuContext *context, gpointer user_data) { - /* Avoid stack operations in potentially performance critical code */ + /* + * This function is performance critical as it is called to instrument every + * basic block. By moving our print buffer to a global, we avoid it affecting + * the critical path with additional stack adjustments if tracing is not + * enabled. If tracing is enabled, then we're printing a load of diagnostic + * information so this overhead is unlikely to be noticeable. + */ static char buffer[200]; int len; guint64 current_pc = (guint64)user_data; diff --git a/frida_mode/src/ranges.c b/frida_mode/src/ranges.c index fc14710f..49ef5a62 100644 --- a/frida_mode/src/ranges.c +++ b/frida_mode/src/ranges.c @@ -29,8 +29,7 @@ static void convert_address_token(gchar *token, GumMemoryRange *range) { gchar **tokens; int token_count; tokens = g_strsplit(token, "-", 2); - for (token_count = 0; tokens[token_count] != NULL; token_count++) - ; + for (token_count = 0; tokens[token_count] != NULL; token_count++) {} if (token_count != 2) { |