Closed Bug 1359565 Opened 7 years ago Closed 2 years ago

Fix and re-enable test_formdataparsing.html and test_formdataparsing_sw_reroute.html on ASAN

Categories

(Core :: DOM: Service Workers, defect, P3)

defect

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox-esr52 --- wontfix
firefox-esr91 --- wontfix
firefox53 --- wontfix
firefox54 --- wontfix
firefox55 --- wontfix
firefox95 --- wontfix
firefox96 --- wontfix
firefox97 --- fixed

People

(Reporter: RyanVM, Assigned: RyanVM)

References

Details

(Keywords: memory-leak, Whiteboard: [MemShrink:P2])

Attachments

(1 file)

Over in bug 1325942, test_formdataparsing.html and test_formdataparsing_sw_reroute.html had to be disabled on ASAN because they were triggering permaleaks with LeakSanitizer.

To make tracking easier, this bug is being filed to track finding and fixing the root cause of the leaks so those tests can be re-enabled.

The leaks took the form of:

==1866==ERROR: LeakSanitizer: detected memory leaks
Direct leak of 13824 byte(s) in 96 object(s) allocated from:
    #0 0x4b2e4b in malloc /builds/slave/moz-toolchain/src/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:52:3
    #1 0x7fe4dff45eee in maybe_pod_malloc<unsigned char> /src/obj-firefox/dist/include/mozilla/AllocPolicy.h:80:28
    #2 0x7fe4dff45eee in pod_malloc<unsigned char> /src/obj-firefox/dist/include/mozilla/AllocPolicy.h:101
    #3 0x7fe4dff45eee in convertToHeapStorage /src/obj-firefox/dist/include/mozilla/Vector.h:936
    #4 0x7fe4dff45eee in mozilla::Vector<unsigned char, 0ul, mozilla::MallocAllocPolicy>::growStorageBy(unsigned long) /src/obj-firefox/dist/include/mozilla/Vector.h:1027
    #5 0x7fe4e03c2cb8 in append<char> /src/obj-firefox/dist/include/mozilla/Vector.h:1353:9
    #6 0x7fe4e03c2cb8 in append<char> /src/obj-firefox/dist/include/mozilla/Vector.h:1414
    #7 0x7fe4e03c2cb8 in mozilla::net::nsStreamLoader::WriteSegmentFun(nsIInputStream*, void*, char const*, unsigned int, unsigned int, unsigned int*) /src/netwerk/base/nsStreamLoader.cpp:137
    #8 0x7fe4e010039c in nsPipeInputStream::ReadSegments(nsresult (*)(nsIInputStream*, void*, char const*, unsigned int, unsigned int, unsigned int*), void*, unsigned int, unsigned int*) /src/xpcom/io/nsPipe3.cpp:1431:12
    #9 0x7fe4e03c31c9 in mozilla::net::nsStreamLoader::OnDataAvailable(nsIRequest*, nsISupports*, nsIInputStream*, unsigned long, unsigned int) /src/netwerk/base/nsStreamLoader.cpp:153:10
Priority: -- → P3
Whiteboard: [MemShrink]
Flags: needinfo?(catalin.badea392)
Whiteboard: [MemShrink] → [MemShrink:P2]
Flags: needinfo?(catalin.badea392) → needinfo?(mdaly)
Backlog.
Flags: needinfo?(mdaly)

NI myself to see if we can re-enable these now.

Flags: needinfo?(ryanvm)
Assignee: nobody → ryanvm
Status: NEW → ASSIGNED
Pushed by rvandermeulen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/52bfbf43dfa4
Re-enable test_formdataparsing.html and test_formdataparsing_sw_reroute.html on ASAN. r=dom-storage-reviewers,janv
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: