Closed
Bug 979242
Opened 11 years ago
Closed 11 years ago
Intermittent | valgrind-test | 21,008 (20,992 direct, 16 indirect) bytes in 1 blocks are definitely lost at calloc / xcb_connect_to_fd / xcb_connect_to_display_with_auth_info / _XConnectXCB
Categories
(Core :: General, defect)
Tracking
()
RESOLVED
FIXED
mozilla30
Tracking | Status | |
---|---|---|
firefox28 | --- | unaffected |
firefox29 | --- | unaffected |
firefox30 | --- | fixed |
firefox-esr24 | --- | unaffected |
People
(Reporter: cbook, Assigned: n.nethercote)
References
()
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
924 bytes,
patch
|
gkw
:
review+
|
Details | Diff | Splinter Review |
Linux x86-64 mozilla-inbound valgrind on 2014-03-03 20:36:46 PST for push af11210cba85
slave: bld-linux64-spot-364
https://tbpl.mozilla.org/php/getParsedLog.php?id=35576351&tree=Mozilla-Inbound
TEST-UNEXPECTED-FAIL | valgrind-test | 21,008 (20,992 direct, 16 indirect) bytes in 1 blocks are definitely lost at calloc / xcb_connect_to_fd / xcb_connect_to_display_with_auth_info / _XConnectXCB
![]() |
Assignee | |
Comment 1•11 years ago
|
||
Attachment #8387381 -
Flags: review?(gary)
![]() |
Assignee | |
Updated•11 years ago
|
Assignee: nobody → n.nethercote
Status: NEW → ASSIGNED
![]() |
||
Comment 2•11 years ago
|
||
Comment on attachment 8387381 [details] [diff] [review]
Suppress leak in xcb_connect_to_fd detected by Valgrind.
Is this diff'ed against the correct changeset?
I checked that it does not seem to patch correctly at http://hg.mozilla.org/mozilla-central/annotate/b0e7f63c2138/build/valgrind/x86_64-redhat-linux-gnu.sup which is tip right now. (Moreover I couldn't find the bug 850134 in the source tree (from the context).)
Nonetheless, the changed lines itself are r=me once you get a patch that can actually land on m-c tip properly.
Attachment #8387381 -
Flags: review?(gary) → review+
![]() |
Assignee | |
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Updated•11 years ago
|
status-firefox28:
--- → unaffected
status-firefox29:
--- → unaffected
status-firefox30:
--- → fixed
status-firefox-esr24:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•