diff options
author | Guy Harris <guy@alum.mit.edu> | 2015-01-18 13:26:12 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2015-01-18 21:28:04 +0000 |
commit | 7940bbc192d91ee156dda5a91cd7dca754836882 (patch) | |
tree | 8623ea4d5f5f1b6beb75ab92d455529ba8929b9c /tools | |
parent | d2d4cf27e60abea9ace02f125fc5c398103c8884 (diff) |
Nobody uses se_ allocations, so remove the code for them.
Don't check for se_ routines in checkAPIs; the failure to compile or
link will suffice to catch attempts to use them.
Update comments in another test script to reflect the disappearance of
the session allocator.
Change-Id: If50d953c7130f48b696bc2dd9d327bea2af49bd4
Reviewed-on: https://code.wireshark.org/review/6638
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/checkAPIs.pl | 19 | ||||
-rw-r--r-- | tools/test-common.sh | 6 |
2 files changed, 3 insertions, 22 deletions
diff --git a/tools/checkAPIs.pl b/tools/checkAPIs.pl index df7086f3d5..94843572a6 100755 --- a/tools/checkAPIs.pl +++ b/tools/checkAPIs.pl @@ -81,7 +81,7 @@ my %APIs = ( # The MSDN page for ZeroMemory recommends SecureZeroMemory # instead. 'ZeroMemory', - # use ep_*, se_*, or g_* functions instead of these: + # use wmem_*, ep_*, or g_* functions instead of these: # (One thing to be aware of is that space allocated with malloc() # may not be freeable--at least on Windows--with g_free() and # vice-versa.) @@ -162,23 +162,6 @@ my %APIs = ( 'ep_stack_push', 'ep_stack_pop', 'ep_stack_peek', - 'se_alloc', - 'se_new', - 'se_alloc0', - 'se_new0', - 'se_strdup', - 'se_strndup', - 'se_memdup', - 'se_strdup_vprintf', - 'se_strdup_printf', - 'se_alloc_array', - 'se_tree_create', - 'se_tree_insert32', - 'se_tree_lookup32', - 'se_tree_lookup32_le', - 'se_tree_insert32_array', - 'se_tree_lookup32_array', - 'se_tree_lookup32_array_le', 'emem_tree_insert32', 'emem_tree_lookup32', 'emem_tree_lookup32_le', diff --git a/tools/test-common.sh b/tools/test-common.sh index 17ac66b57a..8d81b3ca5b 100644 --- a/tools/test-common.sh +++ b/tools/test-common.sh @@ -90,12 +90,10 @@ fi ############################################################################## ### Set up environment variables for fuzz testing ### ############################################################################## -# Initialize (ep_ and se_) allocated memory to 0xBADDCAFE and freed memory +# Initialize ep_ allocated memory to 0xBADDCAFE and freed memory # to 0xDEADBEEF export WIRESHARK_DEBUG_SCRUB_MEMORY= -# Use canaries in se_ allocations (off by default due to the memory usage) -export WIRESHARK_DEBUG_SE_USE_CANARY= -# Verify that ep_ and se_ allocated memory is not passed to certain routines +# Verify that ep_ allocated memory is not passed to certain routines # which need the memory to be persistent. export WIRESHARK_EP_VERIFY_POINTERS= export WIRESHARK_SE_VERIFY_POINTERS= |