Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(93)

Side by Side Diff: build/sanitizers/lsan_suppressions.cc

Issue 924923002: Add default options/suppressions for LeakSanitizer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « build/sanitizers/OWNERS ('k') | build/sanitizers/sanitizer_options.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 // This file contains the default suppressions for LeakSanitizer.
6 // You can also pass additional suppressions via LSAN_OPTIONS:
7 // LSAN_OPTIONS=suppressions=/path/to/suppressions. Please refer to
8 // http://dev.chromium.org/developers/testing/leaksanitizer for more info.
9
10 #if defined(LEAK_SANITIZER)
11
12 // Please make sure the code below declares a single string variable
13 // kLSanDefaultSuppressions which contains LSan suppressions delimited by
14 // newlines. See http://dev.chromium.org/developers/testing/leaksanitizer
15 // for the instructions on writing suppressions.
16 char kLSanDefaultSuppressions[] =
17 // Intentional leak used as sanity test for Valgrind/memcheck.
18 "leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody\n"
19
20 // ================ Leaks in third-party code ================
21
22 // False positives in libfontconfig. http://crbug.com/39050
23 "leak:libfontconfig\n"
24
25 // Leaks in Nvidia's libGL.
26 "leak:libGL.so\n"
27
28 // A small string is leaked here (57 bytes per process). http://crbug.com/46571# c9
29 "leak:blink::V8GCController::collectGarbage\n"
30
31 // http://crbug.com/270180
32 "leak:net::ProxyResolverV8::Context::ResolveProxy\n"
33
34 // TODO(earthdok): revisit NSS suppressions after the switch to BoringSSL
35 // NSS leaks in CertDatabaseNSSTest tests. http://crbug.com/51988
36 "leak:net::NSSCertDatabase::ImportFromPKCS12\n"
37 "leak:net::NSSCertDatabase::ListCerts\n"
38 "leak:net::NSSCertDatabase::DeleteCertAndKey\n"
39 "leak:crypto::ScopedTestNSSDB::ScopedTestNSSDB\n"
40 // Another leak due to not shutting down NSS properly. http://crbug.com/124445
41 "leak:error_get_my_stack\n"
42 // The NSS suppressions above will not fire when the fast stack unwinder is
43 // used, because it can't unwind through NSS libraries. Apply blanket
44 // suppressions for now.
45 "leak:libnssutil3\n"
46 "leak:libnspr4\n"
47 "leak:libnss3\n"
48 "leak:libplds4\n"
49 "leak:libnssckbi\n"
50
51 // XRandR has several one time leaks.
52 "leak:libxrandr\n"
53
54 // Skia leaks GrGpuGL::ProgramCache::Entry. http://crbug.com/262934
55 "leak:GrGpuGL::flushGraphicsState\n"
56
57 // xrandr leak. http://crbug.com/119677
58 "leak:XRRFindDisplay\n"
59
60 // V8 may leak this by design in unit tests. http://crbug.com/323149
61 "leak:v8::internal::Genesis::Genesis\n"
62
63 // Suppressions for objects which can be owned by the V8 heap. This is a
64 // temporary workaround until LeakSanitizer supports the V8 heap.
65 // Those should only fire in (browser)tests. If you see one of them in Chrome,
66 // then it's a real leak.
67 // http://crbug.com/328552
68 "leak:WTF::StringImpl::createUninitialized\n"
69 "leak:WTF::StringImpl::create8BitIfPossible\n"
70 "leak:blink::MouseEvent::create\n"
71 "leak:blink::WindowProxy::initializeIfNeeded\n"
72 "leak:blink::*::*GetterCallback\n"
73 "leak:blink::CSSComputedStyleDeclaration::create\n"
74 "leak:blink::V8PerIsolateData::ensureDomInJSContext\n"
75 "leak:gin/object_template_builder.h\n"
76 "leak:gin::internal::Dispatcher\n"
77 "leak:blink::LocalDOMWindow::getComputedStyle\n"
78
79 // http://crbug.com/356785
80 "leak:content::RenderViewImplTest_DecideNavigationPolicyForWebUI_Test::TestBody\ n"
81
82 // ================ Leaks in Chromium code ================
83 // PLEASE DO NOT ADD SUPPRESSIONS FOR NEW LEAKS.
84 // Instead, commits that introduce memory leaks should be reverted. Suppressing
85 // the leak is acceptable in some cases when reverting is impossible, i.e. when
86 // enabling leak detection for the first time for a test target with
87 // pre-existing leaks.
88
89 // Small test-only leak in ppapi_unittests. http://crbug.com/258113
90 "leak:ppapi::proxy::PPP_Instance_Private_ProxyTest_PPPInstancePrivate_Test\n"
91
92 // http://crbug.com/318221
93 "leak:base::EnsureProcessTerminated\n"
94
95 // http://crbug.com/322671
96 "leak:content::SpeechRecognitionBrowserTest::SetUpOnMainThread\n"
97
98 // http://crbug.com/355641
99 "leak:TrayAccessibilityTest\n"
100
101 // http://crbug.com/354644
102 "leak:CertificateViewerUITest::ShowModalCertificateViewer\n"
103
104 // http://crbug.com/356306
105 "leak:content::SetProcessTitleFromCommandLine\n"
106
107 // PLEASE READ ABOVE BEFORE ADDING NEW SUPPRESSIONS.
108
109 // End of suppressions.
110 ; // Please keep this semicolon.
111
112 #endif // LEAK_SANITIZER
OLDNEW
« no previous file with comments | « build/sanitizers/OWNERS ('k') | build/sanitizers/sanitizer_options.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698