Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

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
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
4a1c87e 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
@LocutusOfBorg LocutusOfBorg linked a pull request Nov 25, 2022 that will close this issue
Open
@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
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
d30ca2e 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
@LocutusOfBorg
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
9e909ba 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
@LocutusOfBorg
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
ceb0943 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Nov 29, 2022
@LocutusOfBorg
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
b8371b4 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
@koeppea koeppea mentioned this issue Jan 21, 2023
Merged
LocutusOfBorg added a commit to LocutusOfBorg/ettercap that referenced this issue Jan 26, 2023
@LocutusOfBorg
Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det…
dffb0a3 
…ected by CFLAGS="-g -fsanitize=address"

Fixes: Ettercap#1201
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix two memory leaks on CALLOC/FREE of ec_globals_alloc function, det… LocutusOfBorg/ettercap
2 participants
@LocutusOfBorg @5hadowblad3