If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

aggrNode is null checked after dereference [@ TParseContext::areAllChildConst]

RESOLVED INVALID

Status

()

Core
Graphics
--
critical
RESOLVED INVALID
7 years ago
6 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

({coverity, crash})

Trunk
coverity, crash
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
1026 bool TParseContext::areAllChildConst(TIntermAggregate* aggrNode)
1027 {

derefrences aggrNode:
1028     if (!aggrNode->isConstructor())
1029         return false;

Null checks after dereference:
1035     if (aggrNode) {
(Assignee)

Comment 1

7 years ago
Created attachment 482412 [details] [diff] [review]
assume null check is wanted
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Needs to be filed at http://code.google.com/p/angleproject/ -- please file further ANGLE bugs there.
Status: ASSIGNED → NEW
(Assignee)

Updated

7 years ago
Crash Signature: [@ TParseContext::areAllChildConst]

Comment 3

6 years ago
Our latest ANGLE import has ASSERT(aggrNode != NULL); so this is not a valid patch.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.