Closed Bug 1289278 Opened 4 years ago Closed 4 years ago

Crash in mozilla::net::CacheFileIOManager::GetCacheDirectory

Categories

(Core :: Networking: Cache, defect, critical)

defect
Not set
critical

Tracking

()

VERIFIED FIXED
mozilla50
Tracking Status
firefox50 --- verified

People

(Reporter: ting, Assigned: ting)

Details

(Keywords: crash, Whiteboard: [necko-active])

Crash Data

Attachments

(1 file, 1 obsolete file)

This bug was filed from the Socorro interface and is 
report bp-e7511bff-8451-48aa-bc13-962c92160725.
=============================================================

#4 on Mac with Nightly 20160724. Low volume, but it happens on all platform for null dereferencing.
Attached patch patch v1 (obsolete) — Splinter Review
Attachment #8774608 - Flags: review?(honzab.moz)
Assignee: nobody → janus926
Whiteboard: [necko-active]
Attachment #8774608 - Flags: review?(honzab.moz) → review?(michal.novotny)
Attachment #8774608 - Flags: review?(michal.novotny) → review+
Attached patch patch v2Splinter Review
Reviewer updated.

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=724dfdc6876b
Attachment #8774608 - Attachment is obsolete: true
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/abb0c67c8f6c
Fix null dereferencing in GetCacheDirectory(). r=michal
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/abb0c67c8f6c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
I believe we can safely mark this verified fixe on Fx50, based on crash data from the last ~3 months.

  SIGNATURE   | mozilla::net::CacheFileIOManager::GetCacheDirectory
  -----------------------------------------------------------------
  CRASH STATS | http://tinyurl.com/h9r4z79
  -----------------------------------------------------------------
  OVERVIEW    | 0 crashes on nightly 52
	      | 0 crashes on nightly 51
	      | 0 crashes on aurora 51
	      | 0 crashes on nightly 50
	      | 0 crashes on aurora 50
	      | 0 crashes on beta 50
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.