diff options
author | Timur Pocheptsov <timur.pocheptsov@theqtcompany.com> | 2016-05-24 12:33:16 +0200 |
---|---|---|
committer | Timur Pocheptsov <timur.pocheptsov@theqtcompany.com> | 2016-05-25 12:15:31 +0000 |
commit | 70b889683339aa3fc7a444e9228f727fa33803fa (patch) | |
tree | 32785ae37e9ccb21a9ac48a50a8e9a05a8ee7bb3 /tools | |
parent | 50e22c765343102c4e0acf1eee8a6ce6f6f39ccf (diff) |
QOpenGLFunctions - move assert inside if statement
Coverity, CIDs 159312-159335, the same pattern:
if (ptr)
ptr->refs.deref();
Q_ASSERT(ptr->reds.load());
Coverity does not like (and rightfully so) potentially invalid (null)
pointer inside Q_ASSERT. Move Q_ASSERT into if-controlled block-statement.
Somehow these CIDs were skipped in the previous similar pass/fix.
Change-Id: I56f92d964ec0b59a7fdff066fa7231ddd71fd0d3
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions