diff options
author | Cyrille Bagard <nocbos@gmail.com> | 2017-11-26 20:39:28 (GMT) |
---|---|---|
committer | Cyrille Bagard <nocbos@gmail.com> | 2017-11-26 20:39:28 (GMT) |
commit | 1252efcd18a845a7c2641354838c26ece3d6d873 (patch) | |
tree | 13d248454e99b76dd4c61fbd41c8c138ebb61eb7 /src | |
parent | 84efc00d07f9574c3867f96a3a7333b0f9d154e6 (diff) |
Increased the reference counter for provided operands and updated code.
Diffstat (limited to 'src')
-rw-r--r-- | src/analysis/db/items/switcher.c | 20 | ||||
-rw-r--r-- | src/analysis/disass/links.c | 8 | ||||
-rw-r--r-- | src/arch/arm/v7/fetch.c | 14 | ||||
-rw-r--r-- | src/arch/arm/v7/link.c | 4 | ||||
-rw-r--r-- | src/arch/arm/v7/post.c | 2 | ||||
-rw-r--r-- | src/arch/instruction.c | 13 | ||||
-rw-r--r-- | src/arch/link.c | 14 | ||||
-rw-r--r-- | src/arch/post.c | 2 | ||||
-rw-r--r-- | src/arch/raw.c | 6 |
9 files changed, 71 insertions, 12 deletions
diff --git a/src/analysis/db/items/switcher.c b/src/analysis/db/items/switcher.c index 35acf5f..1863098 100644 --- a/src/analysis/db/items/switcher.c +++ b/src/analysis/db/items/switcher.c @@ -269,6 +269,7 @@ GDbSwitcher *g_db_switcher_new(GArchInstruction *instr, const GImmOperand *imm, GDbSwitcher *result; /* Instance à retourner */ size_t count; /* Nombre d'opérandes à visiter*/ size_t i; /* Boucle de parcours */ + GArchOperand *op; /* Opérande manipulé */ const mrange_t *range; /* Localisation de l'instruct° */ /* Recherche de la position de l'opérande */ @@ -278,8 +279,19 @@ GDbSwitcher *g_db_switcher_new(GArchInstruction *instr, const GImmOperand *imm, count = _g_arch_instruction_count_operands(instr); for (i = 0; i < count; i++) - if (G_ARCH_OPERAND(imm) == _g_arch_instruction_get_operand(instr, i)) + { + op = _g_arch_instruction_get_operand(instr, i); + + if (G_ARCH_OPERAND(imm) == op) + { + g_object_unref(G_OBJECT(op)); break; + } + + else + g_object_unref(G_OBJECT(op)); + + } g_arch_instruction_unlock_operands(instr); @@ -523,7 +535,7 @@ static bool g_db_switcher_run(GDbSwitcher *switcher, GLoadedBinary *binary, ImmO if (op == NULL) { result = false; - goto exit_instr; + goto exit_without_operand; } result = G_IS_IMM_OPERAND(op); @@ -561,6 +573,10 @@ static bool g_db_switcher_run(GDbSwitcher *switcher, GLoadedBinary *binary, ImmO exit_operand: + g_object_unref(G_OBJECT(op)); + + exit_without_operand: + g_object_unref(G_OBJECT(instr)); exit_instr: diff --git a/src/analysis/disass/links.c b/src/analysis/disass/links.c index 640d266..09f8d1b 100644 --- a/src/analysis/disass/links.c +++ b/src/analysis/disass/links.c @@ -237,7 +237,7 @@ static void convert_immediate_into_target(GArchInstruction *instr, size_t index, ciit_done: - ; + g_object_unref(G_OBJECT(op)); } @@ -280,7 +280,7 @@ void establish_links_for_instruction(GArchInstruction *instr, GBinFormat *format convert_immediate_into_target(instr, i, format); op = _g_arch_instruction_get_operand(instr, i); - if (!G_IS_TARGET_OPERAND(op)) continue; + if (!G_IS_TARGET_OPERAND(op)) goto next_op; g_target_operand_get_addr(G_TARGET_OPERAND(op), &addr); @@ -295,6 +295,10 @@ void establish_links_for_instruction(GArchInstruction *instr, GBinFormat *format } + next_op: + + g_object_unref(G_OBJECT(op)); + } g_arch_instruction_unlock_operands(instr); diff --git a/src/arch/arm/v7/fetch.c b/src/arch/arm/v7/fetch.c index a1ddd3e..dee9aa2 100644 --- a/src/arch/arm/v7/fetch.c +++ b/src/arch/arm/v7/fetch.c @@ -94,6 +94,8 @@ void help_fetching_with_instruction_b_with_orig(GArchInstruction *instr, GArchPr else assert(0); + g_object_unref(G_OBJECT(op)); + target = pc + offset; //g_armv7_context_define_encoding(context, target, iset); @@ -157,6 +159,8 @@ void help_fetching_with_instruction_bl_with_orig(GArchInstruction *instr, GArchP else assert(0); + g_object_unref(G_OBJECT(op)); + target = pc + offset; //g_armv7_context_define_encoding(context, target, iset); @@ -210,6 +214,8 @@ void help_fetching_with_instruction_blx_with_dest(GArchInstruction *instr, GArch else assert(0); + g_object_unref(G_OBJECT(op)); + target = pc + offset; //g_armv7_context_define_encoding(context, target, iset); @@ -284,7 +290,7 @@ void help_fetching_with_instruction_bx_with_orig(GArchInstruction *instr, GArchP hfwibwo_no_pc: - ; + g_object_unref(G_OBJECT(op)); } @@ -329,6 +335,8 @@ void help_fetching_with_instruction_cb_n_z(GArchInstruction *instr, GArchProcess else assert(0); + g_object_unref(G_OBJECT(op)); + target = pc + offset; //g_armv7_context_define_encoding(context, target, AV7IS_THUMB); @@ -405,6 +413,7 @@ void help_fetching_with_instruction_ldr_literal_with_orig(GArchInstruction *inst if (!ret) { assert(0); + g_object_unref(G_OBJECT(op)); g_arch_instruction_unlock_operands(instr); return; } @@ -416,6 +425,7 @@ void help_fetching_with_instruction_ldr_literal_with_orig(GArchInstruction *inst if (!g_exe_format_translate_offset_into_vmpa(format, val_offset, &loaded_addr)) { assert(0); + g_object_unref(G_OBJECT(op)); g_arch_instruction_unlock_operands(instr); return; } @@ -433,6 +443,7 @@ void help_fetching_with_instruction_ldr_literal_with_orig(GArchInstruction *inst if (!ret) { + g_object_unref(G_OBJECT(op)); g_arch_instruction_unlock_operands(instr); return; } @@ -484,6 +495,7 @@ void help_fetching_with_instruction_ldr_literal_with_orig(GArchInstruction *inst new = g_imm_operand_new_from_value(MDS_32_BITS_UNSIGNED, target); _g_arch_instruction_replace_operand(instr, op, new); + g_object_unref(G_OBJECT(op)); g_arch_instruction_unlock_operands(instr); diff --git a/src/arch/arm/v7/link.c b/src/arch/arm/v7/link.c index 63fb3ae..68cde7b 100644 --- a/src/arch/arm/v7/link.c +++ b/src/arch/arm/v7/link.c @@ -65,6 +65,8 @@ void handle_armv7_conditional_branch_from_register(GArchInstruction *instr, GArc g_object_unref(G_OBJECT(reg)); + g_object_unref(G_OBJECT(op)); + } @@ -107,4 +109,6 @@ void handle_armv7_return_from_pop(GArchInstruction *instr, GArchProcessor *proc, } + g_object_unref(G_OBJECT(op)); + } diff --git a/src/arch/arm/v7/post.c b/src/arch/arm/v7/post.c index 7d8ecb3..46a1792 100644 --- a/src/arch/arm/v7/post.c +++ b/src/arch/arm/v7/post.c @@ -92,6 +92,8 @@ void post_process_ldr_instructions(GArchInstruction *instr, GArchProcessor *proc ppli_release: + g_object_unref(G_OBJECT(op)); + g_arch_instruction_unlock_operands(instr); } diff --git a/src/arch/instruction.c b/src/arch/instruction.c index c11f253..7e1e646 100644 --- a/src/arch/instruction.c +++ b/src/arch/instruction.c @@ -191,6 +191,10 @@ static void g_arch_instruction_dispose(GArchInstruction *instr) rem_item_from_flat_array(&instr->operands, 0, sizeof(GArchOperand *)); + /** + * Une fois pour l'obtention, une autre pour la libération ! + */ + g_object_unref(G_OBJECT(op)); g_object_unref(G_OBJECT(op)); } @@ -564,7 +568,7 @@ GArchOperand *_g_arch_instruction_get_operand(const GArchInstruction *instr, siz result = *ptr; - /* TODO : incrémenter la référence ! */ + g_object_ref(G_OBJECT(result)); return result; @@ -635,6 +639,8 @@ bool _g_arch_instruction_replace_operand(GArchInstruction *instr, GArchOperand * break; } + g_object_unref(G_OBJECT(op)); + } if (result) @@ -678,6 +684,8 @@ void _g_arch_instruction_detach_operand(GArchInstruction *instr, GArchOperand *t if (op == target) break; + g_object_unref(G_OBJECT(op)); + } rem_item_from_flat_array(&instr->operands, i, sizeof(GArchOperand *)); @@ -1192,6 +1200,7 @@ static void _g_arch_instruction_print(GArchInstruction *instr, GBufferLine *line { op = _g_arch_instruction_get_operand(instr, 0); g_arch_operand_print(op, line, 0/*syntax*/); + g_object_unref(G_OBJECT(op)); for (i = 1; i < count; i++) { @@ -1202,6 +1211,8 @@ static void _g_arch_instruction_print(GArchInstruction *instr, GBufferLine *line g_arch_operand_print(op, line, 0/*syntax*/); + g_object_unref(G_OBJECT(op)); + } } diff --git a/src/arch/link.c b/src/arch/link.c index fbcd391..2e8e455 100644 --- a/src/arch/link.c +++ b/src/arch/link.c @@ -61,9 +61,7 @@ void handle_jump_as_link(GArchInstruction *instr, GArchProcessor *proc, GProcCon g_arch_instruction_unlock_operands(instr); - if (!G_IS_IMM_OPERAND(op)) return; - - if (g_imm_operand_to_virt_t(G_IMM_OPERAND(op), &virt)) + if (G_IS_IMM_OPERAND(op) && g_imm_operand_to_virt_t(G_IMM_OPERAND(op), &virt)) { init_vmpa(&addr, VMPA_NO_PHYSICAL, virt); @@ -77,6 +75,8 @@ void handle_jump_as_link(GArchInstruction *instr, GArchProcessor *proc, GProcCon } + g_object_unref(G_OBJECT(op)); + } @@ -131,6 +131,8 @@ void handle_branch_as_link(GArchInstruction *instr, GArchProcessor *proc, GProcC } } + g_object_unref(G_OBJECT(op)); + if (defined) { target = g_arch_processor_find_instr_by_address(proc, &addr); @@ -191,9 +193,7 @@ void handle_call_as_link(GArchInstruction *instr, GArchProcessor *proc, GProcCon g_arch_instruction_unlock_operands(instr); - if (!G_IS_IMM_OPERAND(op)) return; - - if (g_imm_operand_to_virt_t(G_IMM_OPERAND(op), &virt)) + if (G_IS_IMM_OPERAND(op) && g_imm_operand_to_virt_t(G_IMM_OPERAND(op), &virt)) { init_vmpa(&addr, VMPA_NO_PHYSICAL, virt); @@ -207,4 +207,6 @@ void handle_call_as_link(GArchInstruction *instr, GArchProcessor *proc, GProcCon } + g_object_unref(G_OBJECT(op)); + } diff --git a/src/arch/post.c b/src/arch/post.c index 59522bf..f3d1fd6 100644 --- a/src/arch/post.c +++ b/src/arch/post.c @@ -119,6 +119,8 @@ void post_process_target_resolution(GArchInstruction *instr, GArchProcessor *pro } + g_object_unref(G_OBJECT(op)); + g_arch_instruction_unlock_operands(instr); } diff --git a/src/arch/raw.c b/src/arch/raw.c index 8ab788f..e85d865 100644 --- a/src/arch/raw.c +++ b/src/arch/raw.c @@ -468,6 +468,8 @@ static const char *g_raw_instruction_get_keyword(const GRawInstruction *instr, A else size = g_imm_operand_get_size(G_IMM_OPERAND(operand)); + g_object_unref(G_OBJECT(operand)); + return defines[MDS_RANGE(size)]; } @@ -585,6 +587,8 @@ static void g_raw_instruction_print(GRawInstruction *instr, GBufferLine *line, s string[iter++] = byte; + g_object_unref(G_OBJECT(op)); + continue; grip_fallback: @@ -621,6 +625,8 @@ static void g_raw_instruction_print(GRawInstruction *instr, GBufferLine *line, s g_arch_operand_print(op, line, 0/*syntax*/); + g_object_unref(G_OBJECT(op)); + } /* Si au final une chaîne traine encore */ |