[SERVER-15660] PlanRanker::hasStage should protect itself against NULL pointers Created: 14/Oct/14 Updated: 10/Dec/14 Resolved: 06/Dec/14 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | 2.7.7 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Samantha Ritter (Inactive) | Assignee: | J Rassi |
| Resolution: | Done | Votes: | 0 |
| Labels: | 28qa | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Operating System: | ALL |
| Participants: |
| Description |
|
In the case that a null PlanStageStats* gets pushed onto the tree of children stats passed to PlanRanker::hasStage(), this method will attempt to access 'stageType' on this null pointer and seg fault. I imagine that in theory, null PlanStageStats pointers should never be passed to this method. However, the MockStage currently returns NULL as its stats, and that is able to propagate through to this point and crash the system. |