Coverity FORWARD_NULL - CID 12458 - src/match.cpp - in function: matchOpsEqual(std::list<MatchOp *, std::allocator<MatchOp *>> &, std::list<MatchOp *, std::allocator<MatchOp *>> &) - Dynamic cast to pointer "dynamic_cast <MatchExpOp *>(*it1)" can return NULL. Assigning: "e1" = "dynamic_cast <MatchExpOp *>(*it1)". Passing null pointer "e1->value" to function "std::operator !=<char, std::char_traits<char>, std::allocator<char> >(std::basic_string<char, std::char_traits<char>, std::allocator<char> > const &, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const &)", which dereferences it.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Confirmed
|
Medium
|
Unassigned | ||
0.9.9 |
Won't Fix
|
High
|
Unassigned |
Bug Description
This bug is exported from the Coverity Integration Manager on Canonical's servers. For information on how this is done please see this website: https:/
CID: 12458
Checker: FORWARD_NULL
Category: dynamic_cast
CWE definition: http://
File: /tmp/buildd/
Function: matchOpsEqual(
Code snippet:
224 return false;
225
226 break;
227 case MatchOp::TypeExp:
CID 12458 - FORWARD_NULL
Dynamic cast to pointer "dynamic_cast <MatchExpOp *>(*it1)" can return NULL.
Assigning: "e1" = "dynamic_cast <MatchExpOp *>(*it1)".
228 e1 = dynamic_
229 e2 = dynamic_
230
CID 12458 - FORWARD_NULL
Passing null pointer "e1->value" to function "std::operator !=<char, std::char_
231 if (e1->value != e2->value)
232 return false;
233
summary: |
- Coverity FORWARD_NULL - CID 12458 + Coverity FORWARD_NULL - CID 12458 - src/match.cpp - in function: + matchOpsEqual(std::list<MatchOp *, std::allocator<MatchOp *>> &, + std::list<MatchOp *, std::allocator<MatchOp *>> &) - Dynamic cast to + pointer "dynamic_cast <MatchExpOp *>(*it1)" can return NULL. Assigning: + "e1" = "dynamic_cast <MatchExpOp *>(*it1)". Passing null pointer + "e1->value" to function "std::operator !=<char, std::char_traits<char>, + std::allocator<char> >(std::basic_string<char, std::char_traits<char>, + std::allocator<char> > const &, std::basic_string<char, + std::char_traits<char>, std::allocator<char> > const &)", which + dereferences it. |
Changed in compiz: | |
milestone: | none → 0.9.10.0 |
Changed in compiz: | |
milestone: | 0.9.10.0 → 0.9.11.0 |
Changed in compiz: | |
status: | New → Confirmed |
importance: | High → Medium |
milestone: | 0.9.11.0 → 0.9.12.1 |
Changed in compiz: | |
milestone: | 0.9.12.1 → 0.9.12.2 |
Source file with Coverity annotations.