cea-sec/miasm

View on GitHub

Showing 1,647 of 3,020 total issues

Similar blocks of code found in 6 locations. Consider refactoring.
Open

def kernel32_GetVersion(jitter):
    ret_ad, _ = jitter.func_args_stdcall(0)
    jitter.func_ret_stdcall(ret_ad, winobjs.getversion)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 5 other locations - About 30 mins to fix
miasm/os_dep/win_api_x86_32.py on lines 321..323
miasm/os_dep/win_api_x86_32.py on lines 331..333
miasm/os_dep/win_api_x86_32.py on lines 336..338
miasm/os_dep/win_api_x86_32.py on lines 457..459
miasm/os_dep/win_api_x86_32.py on lines 1002..1004

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 6 locations. Consider refactoring.
Open

def kernel32_GetCurrentProcess(jitter):
    ret_ad, _ = jitter.func_args_stdcall(0)
    jitter.func_ret_stdcall(ret_ad, winobjs.handle_curprocess)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 5 other locations - About 30 mins to fix
miasm/os_dep/win_api_x86_32.py on lines 321..323
miasm/os_dep/win_api_x86_32.py on lines 336..338
miasm/os_dep/win_api_x86_32.py on lines 374..376
miasm/os_dep/win_api_x86_32.py on lines 457..459
miasm/os_dep/win_api_x86_32.py on lines 1002..1004

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Identical blocks of code found in 7 locations. Consider refactoring.
Open

                    addr, size = (int(x, 16) for x in buf.read().split(b",", 1))
Severity: Major
Found in miasm/analysis/gdbserver.py and 6 other locations - About 30 mins to fix
miasm/analysis/gdbserver.py on lines 129..129
miasm/analysis/gdbserver.py on lines 168..168
miasm/analysis/gdbserver.py on lines 183..183
miasm/analysis/gdbserver.py on lines 202..202
miasm/analysis/gdbserver.py on lines 214..214
miasm/analysis/gdbserver.py on lines 227..227

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def kernel32_CloseHandle(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["hwnd"])
    jitter.func_ret_stdcall(ret_ad, 1)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def user32_GetTopWindow(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["hwnd"])
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def msvcrt_free(jitter):
    ret_ad, _ = jitter.func_args_cdecl(["ptr"])
    jitter.func_ret_cdecl(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def kernel32_TlsFree(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["tlsindex"])
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def advapi32_CryptDestroyHash(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["hhash"])
    jitter.func_ret_stdcall(ret_ad, 1)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def kernel32_GetPriorityClass(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["hwnd"])
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def ntdll_RtlFreeUnicodeString(jitter):
    ret_ad, args = jitter.func_args_stdcall(['src'])
    # l1, l2, p_src = struct.unpack('HHI', jitter.vm.get_mem(args.src, 0x8))
    # s = get_win_str_w(jitter, p_src)
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def msvcrt_atexit(jitter):
    ret_ad, _ = jitter.func_args_cdecl(["func"])
    jitter.func_ret_cdecl(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def kernel32_LocalFree(jitter):
    ret_ad, _ = jitter.func_args_stdcall(["lpvoid"])
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1348..1350
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 16 locations. Consider refactoring.
Open

def kernel32_Sleep(jitter):
    ret_ad, _ = jitter.func_args_stdcall(['t'])
    jitter.func_ret_stdcall(ret_ad, 0)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 15 other locations - About 30 mins to fix
miasm/os_dep/linux_stdlib.py on lines 197..199
miasm/os_dep/win_api_x86_32.py on lines 293..295
miasm/os_dep/win_api_x86_32.py on lines 310..312
miasm/os_dep/win_api_x86_32.py on lines 316..318
miasm/os_dep/win_api_x86_32.py on lines 412..414
miasm/os_dep/win_api_x86_32.py on lines 422..424
miasm/os_dep/win_api_x86_32.py on lines 473..475
miasm/os_dep/win_api_x86_32.py on lines 478..480
miasm/os_dep/win_api_x86_32.py on lines 590..592
miasm/os_dep/win_api_x86_32.py on lines 1994..1998
miasm/os_dep/win_api_x86_32.py on lines 2429..2431
miasm/os_dep/win_api_x86_32.py on lines 2556..2558
miasm/os_dep/win_api_x86_32.py on lines 2806..2808
miasm/os_dep/win_api_x86_32.py on lines 2936..2938
miasm/os_dep/win_api_x86_32.py on lines 3006..3008

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 6 locations. Consider refactoring.
Open

def kernel32_IsDebuggerPresent(jitter):
    ret_ad, _ = jitter.func_args_stdcall(0)
    jitter.func_ret_stdcall(ret_ad, winobjs.dbg_present)
Severity: Major
Found in miasm/os_dep/win_api_x86_32.py and 5 other locations - About 30 mins to fix
miasm/os_dep/win_api_x86_32.py on lines 331..333
miasm/os_dep/win_api_x86_32.py on lines 336..338
miasm/os_dep/win_api_x86_32.py on lines 374..376
miasm/os_dep/win_api_x86_32.py on lines 457..459
miasm/os_dep/win_api_x86_32.py on lines 1002..1004

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

armtop("udiv", [bs('111110111011'), rn, bs('1111'), rd, bs('1111'), rm], [rd, rn, rm] )
Severity: Major
Found in miasm/arch/arm/arch.py and 3 other locations - About 30 mins to fix
miasm/arch/arm/arch.py on lines 3369..3369
miasm/arch/arm/arch.py on lines 3408..3408
miasm/arch/arm/arch.py on lines 3412..3412

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

armtop("mul",  [bs('111110110000'), rn, bs('1111'), rd, bs('0000'), rm], [rd, rn, rm] )
Severity: Major
Found in miasm/arch/arm/arch.py and 3 other locations - About 30 mins to fix
miasm/arch/arm/arch.py on lines 3369..3369
miasm/arch/arm/arch.py on lines 3408..3408
miasm/arch/arm/arch.py on lines 3409..3409

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

mips32op("ctc1",    [cpu.bs('010001'), cpu.bs('00110'), rt, fs,
                     cpu.bs('00000000000')])
Severity: Major
Found in miasm/arch/mips32/arch.py and 3 other locations - About 30 mins to fix
miasm/arch/mips32/arch.py on lines 690..691
miasm/arch/mips32/arch.py on lines 748..749
miasm/arch/mips32/arch.py on lines 751..752

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

def update_flag_addwc_cf(op1, op2, op3):
    "Compute cf in @res = @op1 + @op2 + @op3"
    return [m2_expr.ExprAssign(cf, m2_expr.ExprOp("FLAG_ADDWC_CF", op1, op2, op3))]
Severity: Major
Found in miasm/arch/x86/sem.py and 3 other locations - About 30 mins to fix
miasm/arch/x86/sem.py on lines 225..227
miasm/arch/x86/sem.py on lines 231..233
miasm/arch/x86/sem.py on lines 236..238

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

def update_flag_addwc_of(op1, op2, op3):
    "Compute of in @res = @op1 + @op2 + @op3"
    return [m2_expr.ExprAssign(of, m2_expr.ExprOp("FLAG_ADDWC_OF", op1, op2, op3))]
Severity: Major
Found in miasm/arch/x86/sem.py and 3 other locations - About 30 mins to fix
miasm/arch/x86/sem.py on lines 220..222
miasm/arch/x86/sem.py on lines 231..233
miasm/arch/x86/sem.py on lines 236..238

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 4 locations. Consider refactoring.
Open

mips32op("divu",    [cpu.bs('000000'), rs, rt, cpu.bs('0000000000'),
                     cpu.bs('011011')])
Severity: Major
Found in miasm/arch/mips32/arch.py and 3 other locations - About 30 mins to fix
miasm/arch/mips32/arch.py on lines 682..683
miasm/arch/mips32/arch.py on lines 703..704
miasm/arch/mips32/arch.py on lines 705..706

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 32.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Severity
Category
Status
Source
Language