Closed Bug 1531022 Opened 9 months ago Closed 9 months ago

SymbolLoader::LoadSymbols called from GLContext::LoadMoreSymbols() without error handling

Categories

(Core :: Graphics, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: ehsan, Assigned: jgilbert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Its error return value isn't being checked here. Coverity notes that we check its error return value in most other call sites.

https://searchfox.org/mozilla-central/rev/dbddac86aadf1d4871fb350bbe66db43728a9f81/gfx/gl/GLContext.cpp#1450

Jeff, should we be checking the return value here?

Flags: needinfo?(jgilbert)
Priority: -- → P3

Nope, we should (void) it.

Assignee: nobody → jgilbert
Flags: needinfo?(jgilbert)
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.