Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Memory leak in ettercap, ec_globals_alloc, src/ec_globals.c #1201

Open
5hadowblad3 opened this issue Oct 28, 2022 · 1 comment · May be fixed by #1205
Open

Memory leak in ettercap, ec_globals_alloc, src/ec_globals.c #1201

5hadowblad3 opened this issue Oct 28, 2022 · 1 comment · May be fixed by #1205

Comments

@5hadowblad3
Copy link

Hi, there.

There is a memory leak in ettercap in the newest main branch (8e1d6d8).

To reproduce, compile with

CFLAGS="-g -fsanitize=address"

then run:

./src/ettercap

Here is the trace reported by ASAN:

==1812145==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 80 byte(s) in 1 object(s) allocated from:
    #0 0x4c3b37 in calloc  /dependence/llvm11/llvm-11.0.0.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:154:3
    #1 0x7f980abdabe3 in ec_globals_alloc  /benchmark/ettercap/src/ec_globals.c:46:4

Direct leak of 56 byte(s) in 1 object(s) allocated from:
    #0 0x4c3b37 in calloc  /dependence/llvm11/llvm-11.0.0.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:154:3
    #1 0x7f980abdb06c in ec_globals_alloc  /benchmark/ettercap/src/ec_globals.c:55:4

SUMMARY: AddressSanitizer: 136 byte(s) leaked in 2 allocation(s).
Aborted

LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 25, 2022
@LocutusOfBorg
Copy link
Contributor

@5hadowblad3 hello, can you please test #1205?

LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 26, 2022
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Jan 26, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants