diff options
author | Luca Deri <lucaderi@users.noreply.github.com> | 2020-07-13 10:24:23 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-13 10:24:23 +0200 |
commit | 9d8717a0c7a9bdddb4830f69da58d0dba4b425e2 (patch) | |
tree | 5b7ff5a19d286a99d55df2b039c1ee62475c5c99 /example/ndpiReader.c | |
parent | 9d35364ef103be7c7895bb277b601086bb7ff104 (diff) | |
parent | 35f1c362b9c005a1094f19cd4cdf5039e5e887d5 (diff) |
Merge pull request #963 from yskcg/dev
add improved boundary check and check malloc return is NULL
Diffstat (limited to 'example/ndpiReader.c')
-rw-r--r-- | example/ndpiReader.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/example/ndpiReader.c b/example/ndpiReader.c index b0874bc8e..faf5d5508 100644 --- a/example/ndpiReader.c +++ b/example/ndpiReader.c @@ -2943,6 +2943,9 @@ static void ndpi_process_packet(u_char *args, /* allocate an exact size buffer to check overflows */ uint8_t *packet_checked = malloc(header->caplen); + if(packet_checked == NULL){ + return ; + } memcpy(packet_checked, packet, header->caplen); p = ndpi_workflow_process_packet(ndpi_thread_info[thread_id].workflow, header, packet_checked, csv_fp); @@ -3047,7 +3050,10 @@ static void ndpi_process_packet(u_char *args, Leave the free as last statement to avoid crashes when ndpi_detection_giveup() is called above by printResults() */ - free(packet_checked); + if(packet_checked){ + free(packet_checked); + packet_checked = NULL; + } } /** |