diff options
author | Romain Forlot <romain.forlot@iot.bzh> | 2019-01-18 14:42:34 +0100 |
---|---|---|
committer | Romain Forlot <romain.forlot@iot.bzh> | 2019-05-17 07:42:59 +0000 |
commit | 994b37a0c027b86f5bc257347dd403f71f3ab694 (patch) | |
tree | 285f626d677af6892a40935405b520cc80a054bf /low-can-binding | |
parent | ea8e3ef5b673f0d2b7d096a6bd229e9edf5dec54 (diff) |
Fix: double free issuehalibut_7.99.1halibut/7.99.17.99.1
handle is already freed during the active_diagnostic_request destruction.
Bug-AGL: SPEC-2415
Change-Id: Ia26d5c9d2974ca34d411c3182b218141a84c8f21
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Diffstat (limited to 'low-can-binding')
-rw-r--r-- | low-can-binding/diagnostic/diagnostic-manager.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/low-can-binding/diagnostic/diagnostic-manager.cpp b/low-can-binding/diagnostic/diagnostic-manager.cpp index 55e61084..ba4b0c87 100644 --- a/low-can-binding/diagnostic/diagnostic-manager.cpp +++ b/low-can-binding/diagnostic/diagnostic-manager.cpp @@ -188,8 +188,6 @@ void diagnostic_manager_t::find_and_erase(active_diagnostic_request_t* entry, st void diagnostic_manager_t::cancel_request(active_diagnostic_request_t* entry) { entry->get_socket().close(); - if(entry->get_handle()) - delete(entry->get_handle()); delete entry; entry = nullptr; } |