diff options
author | Dario Lombardo <lomato@gmail.com> | 2018-01-16 17:28:56 +0100 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2018-01-16 18:56:11 +0000 |
commit | d77d5c9d206a1c395ac4762938d6ddf19e92f314 (patch) | |
tree | 12cb9cb034e6506b5ef54c5d12a18ac9a727273e /ui/qt/utils | |
parent | 107a91a8cc283f648552a4924fade2216670fb23 (diff) |
Qt: add more sanity checks in utils (found by clang).
Ensure the pointer exist before dereference them.
Change-Id: I7c2881c9c74b88517d01d9306507c14ff2a7b257
Reviewed-on: https://code.wireshark.org/review/25345
Petri-Dish: Dario Lombardo <lomato@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/qt/utils')
-rw-r--r-- | ui/qt/utils/frame_information.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/qt/utils/frame_information.cpp b/ui/qt/utils/frame_information.cpp index 71a828f4f6..3402454aca 100644 --- a/ui/qt/utils/frame_information.cpp +++ b/ui/qt/utils/frame_information.cpp @@ -47,7 +47,7 @@ FrameInformation::FrameInformation(CaptureFile * capfile, frame_data * fi, QObje void FrameInformation::loadFrameTree() { - if ( ! fi_ || ! cap_file_ ) + if ( ! fi_ || ! cap_file_ || !cap_file_->capFile()) return; if (!cf_read_record(cap_file_->capFile(), fi_)) |