diff options
author | Dominik Maier <domenukk@gmail.com> | 2020-04-13 16:57:19 +0200 |
---|---|---|
committer | Dominik Maier <domenukk@gmail.com> | 2020-04-13 16:57:19 +0200 |
commit | 326ab632c378878c96f44586a6e1f3dfa3db2276 (patch) | |
tree | af7203c4502671c04b6b87545a9e29b4e2cca87d /src/afl-sharedmem.c | |
parent | dda096da03cae528dee9fd53e64896e93efe8f4a (diff) | |
download | afl++-326ab632c378878c96f44586a6e1f3dfa3db2276.tar.gz |
fixed uaf and warnings
Diffstat (limited to 'src/afl-sharedmem.c')
-rw-r--r-- | src/afl-sharedmem.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/afl-sharedmem.c b/src/afl-sharedmem.c index 9db84e77..eea1cc95 100644 --- a/src/afl-sharedmem.c +++ b/src/afl-sharedmem.c @@ -67,6 +67,7 @@ list_t shm_list = {.element_prealloc_count = 0}; void afl_shm_deinit(sharedmem_t *shm) { + // TODO: clang reports a potential UAF in this function/makro(?) list_remove(&shm_list, shm); #ifdef USEMMAP @@ -93,14 +94,6 @@ void afl_shm_deinit(sharedmem_t *shm) { } -/* At exit, remove all leftover maps */ - -void afl_shm_atexit(void) { - - LIST_FOREACH(&shm_list, sharedmem_t, { afl_shm_deinit(el); }); - -} - /* Configure shared memory. Returns a pointer to shm->map for ease of use. */ @@ -207,7 +200,6 @@ u8 *afl_shm_init(sharedmem_t *shm, size_t map_size, unsigned char dumb_mode) { #endif list_append(&shm_list, shm); - atexit(afl_shm_atexit); return shm->map; |