Coverity FORWARD_NULL - CID 12459 - 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 *>(*it2)" can return NULL. Assigning: "e2" = "dynamic_cast <MatchExpOp *>(*it2)". Passing null pointer "e2->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
|
High
|
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: 12459
Checker: FORWARD_NULL
Category: dynamic_cast
CWE definition: http://
File: /tmp/buildd/
Function: matchOpsEqual(
Code snippet:
225
226 break;
227 case MatchOp::TypeExp:
228 e1 = dynamic_
CID 12459 - FORWARD_NULL
Dynamic cast to pointer "dynamic_cast <MatchExpOp *>(*it2)" can return NULL.
Assigning: "e2" = "dynamic_cast <MatchExpOp *>(*it2)".
229 e2 = dynamic_
230
CID 12459 - FORWARD_NULL
Passing null pointer "e2->value" to function "std::operator !=<char, std::char_
231 if (e1->value != e2->value)
232 return false;
233
234 break;
summary: |
- Coverity FORWARD_NULL - CID 12459 + Coverity FORWARD_NULL - CID 12459 - 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 *>(*it2)" can return NULL. Assigning: + "e2" = "dynamic_cast <MatchExpOp *>(*it2)". Passing null pointer + "e2->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 |
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.