)]}' { "log": [ { "commit": "acc9af61966e60c447bdd34344ffad1cd6a9f54b", "tree": "0d9cfd7e87245b2b561b5b555fb94a2a694b5c39", "parents": [ "fe19a10caf9e06006b53d358f1e914a0d4e1514b" ], "author": { "name": "Brian Cain", "email": "brian.cain@oss.qualcomm.com", "time": "Mon Mar 30 09:26:16 2026 -0500" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Mar 30 07:30:45 2026 -0700" }, "message": "[compiler-rt] Enable GWP-ASan for Hexagon (#188410)\n\nAdd Hexagon to ALL_GWP_ASAN_SUPPORTED_ARCH, add struct layout\nassertions.\n\nGitOrigin-RevId: a33acdb0b2dca00db08630050b10b09a7546235a\n" }, { "commit": "fe19a10caf9e06006b53d358f1e914a0d4e1514b", "tree": "7b42fe30a28bbf9b969d7bea81b14d4587ac1eb2", "parents": [ "c2c0aa68c88cd2a1fa1a36a5a8f5703b30386d8c" ], "author": { "name": "Austin Jiang", "email": "austinjiangboyu@gmail.com", "time": "Tue Jan 13 11:52:46 2026 -0500" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 13 08:55:53 2026 -0800" }, "message": "Fix typos and spelling errors across codebase (#156270)\n\nCorrected various spelling mistakes such as \u0027occurred\u0027, \u0027receiver\u0027,\n\u0027initialized\u0027, \u0027length\u0027, and others in comments, variable names,\nfunction names, and documentation throughout the project. These\nchanges improve code readability and maintain consistency in naming\nand documentation.\n\nCo-authored-by: Louis Dionne \u003cldionne.2@gmail.com\u003e\nGitOrigin-RevId: e6cdfb75ac88719c3bb6f3cab17ec0c8a781a34f\n" }, { "commit": "c2c0aa68c88cd2a1fa1a36a5a8f5703b30386d8c", "tree": "418c07d225b7a2c049491617bcb605240f029111", "parents": [ "bb3b0aae4a6d29009f5d8367ab81085794d43790" ], "author": { "name": "Roland McGrath", "email": "mcgrathr@google.com", "time": "Mon Sep 08 11:30:23 2025 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Sep 08 11:35:59 2025 -0700" }, "message": "[gwp_asan] Use anonymous namespace for test helper code (#157205)\n\nTests can be at top-level or inside an anonymous namespace,\ndoesn\u0027t matter. But putting their helper code inside anonymous\nnamespaces both makes the code compatible with compiling using\n-Wmissing-declarations and might let the compiler optimize the\ntest good a bit better.\n\nGitOrigin-RevId: 1f39435567b35a6797d92cd9a24e503e6b0302fd\n" }, { "commit": "bb3b0aae4a6d29009f5d8367ab81085794d43790", "tree": "f5cabf98ffe1051ddc6e8d109e03eed2464e2c80", "parents": [ "ff5bc9a472dc5be4de26fb554dfb5409fe568708" ], "author": { "name": "Fabio D\u0027Urso", "email": "fdurso@google.com", "time": "Tue Aug 26 15:11:48 2025 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Aug 26 06:13:24 2025 -0700" }, "message": "[GWP-ASan] Include \u003cunistd.h\u003e for sysconf(_SC_PAGESIZE) (#155261)\n\nThis fixes build failures on Fuchsia that started with #153860\n\nGitOrigin-RevId: c05567842b73a2286ba6d84cac09693d41f0c838\n" }, { "commit": "ff5bc9a472dc5be4de26fb554dfb5409fe568708", "tree": "dc88061393017b7afa4b164d6216ad8f45c1fe66", "parents": [ "a6da01e612ed0c87b977b3b6ea3d939cd2431823" ], "author": { "name": "Peter Collingbourne", "email": "pcc@google.com", "time": "Thu Aug 14 12:24:12 2025 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Aug 15 14:45:43 2025 -0700" }, "message": "compiler-rt: Make the tests pass on AArch64 and with page size !\u003d 4096.\n\nThis makes the tests pass on my AArch64 machine with 16K pages.\n\nNot sure why some of the AArch64-specific test failures don\u0027t seem to\noccur on sanitizer-aarch64-linux. I could also reproduce them by running\nbuildbot_cmake.sh on my machine.\n\nPull Request: https://github.com/llvm/llvm-project/pull/153860\n\nGitOrigin-RevId: 19cfc30b33df5bf121bac96d4c1aac6028df0670\n" }, { "commit": "a6da01e612ed0c87b977b3b6ea3d939cd2431823", "tree": "8abb1e08f31d77da31aae2b4050b8c94b446c6f0", "parents": [ "e9fd72c3535f99c7ff6bae02128e04cbb3596ec5" ], "author": { "name": "Caslyn Tonelli", "email": "6718161+Caslyn@users.noreply.github.com", "time": "Tue Jan 07 10:03:31 2025 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 07 10:05:42 2025 -0800" }, "message": "[gwp_asan] Soft-transition ZXTEST_USE_STREAMABLE_MACROS removal (#121887)\n\nSoft-transition the removal of setting ZXTEST_USE_STREAMABLE_MACROS, by\nonly setting the macro if not already defined. A future PR will remove\nsetting the macro entirely in harness.h.\n\nGitOrigin-RevId: 478648e2c0adbafa5e4cb10eb53aadbdf7de97d9\n" }, { "commit": "e9fd72c3535f99c7ff6bae02128e04cbb3596ec5", "tree": "6db984283f8ed2ae30169c8295001498a0d1e782", "parents": [ "2fee62a7c93b3f14502f9b616c79d7d0708963ad" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Jul 29 11:35:20 2024 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Jul 29 03:36:50 2024 -0700" }, "message": "Log errno (or fuchsia equivalent) on map failures (#95391)\n\nA feature requested by Android, we should log errno (or the\ncorresponding fuchsia error status) as part of the message when\nmmap/mprotect/etc. fails.\n\nGitOrigin-RevId: 0a94511aec7a41194c0e61d88801312542ff70ce\n" }, { "commit": "2fee62a7c93b3f14502f9b616c79d7d0708963ad", "tree": "a8cc84f626669de7af10aa646ef9de09399f03c3", "parents": [ "baaa534a19094c2528b12b4c3e7bd80a43264a73" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Jun 25 09:22:29 2024 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jun 25 01:26:18 2024 -0700" }, "message": "[NFC] [GWP-ASan] Rename Check() to check() (#96605)\n\nChange this function to be LLVM-style in name.\n\nGitOrigin-RevId: 11e12bd82b0358de053cfd0dc762e9b30948c8a1\n" }, { "commit": "baaa534a19094c2528b12b4c3e7bd80a43264a73", "tree": "91aa40fdb6a1c7842b10c5da68faaf78eff928a2", "parents": [ "4b71d2c81a62e1a489462f59c12529024456a1d4" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jun 13 09:21:31 2024 +0100" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jun 13 01:25:31 2024 -0700" }, "message": "[GWP-ASan] Various test fixes. (#94938)\n\nWhen running some tests with --gtest_repeat\u003d100 --gtest_shuffle, I\nencountered some problems because the allocator wasn\u0027t torn down\ncompletely, and the singleton pointer ended up pointing to a\nuse-after-scope\u0027d object.\n\nThis patch has a couple of fixes and niceties:\n 1. Removing the once-init stuff from tests, now that it\u0027s implicitly\n done in GuardedPoolAllocator::installAtFork() anyway.\n 2. Calling uninitTestOnly() in the late_init test.\n 3. Resetting the HasReportedBadPoolAccess when the signal handlers are\n installed (allowing for --gtest_repeat w/ recoverable mode).\n 4. Adding a check and resetting the singleton pointer in\n uninitTestOnly().\n\nGitOrigin-RevId: 7adb7aa494247f2492f6207289ad90cb48807517\n" }, { "commit": "4b71d2c81a62e1a489462f59c12529024456a1d4", "tree": "33a7a4061f97b355b27826481b2876fe655f9c14", "parents": [ "b71867cbdc5a73139394e896d176f9d998b908d9" ], "author": { "name": "Benjamin Kramer", "email": "benny.kra@googlemail.com", "time": "Thu Sep 14 10:58:59 2023 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Sep 14 02:05:38 2023 -0700" }, "message": "Add GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST to gwp_asan unittests\n\nThis keeps it working after the gtest update in a866ce789eb99da4d7a486eeb60a53be6c75f4fd\n\nGitOrigin-RevId: 4f1f171f330545c7fad254a672c61eaffcd9344c\n" }, { "commit": "b71867cbdc5a73139394e896d176f9d998b908d9", "tree": "2c089a6e665f52dc57357f364eb12835b23d3166", "parents": [ "fdc0cff65d4482218f748d44369cfe4b876fcca8" ], "author": { "name": "Ian Anderson", "email": "iana@apple.com", "time": "Wed Aug 23 13:17:42 2023 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Aug 30 11:45:38 2023 -0700" }, "message": "[Headers][Modules] Make separate headers for the stdarg.h and stddef.h pieces so that they can be modularized\n\nstdarg.h and stddef.h have to be textual headers in their upcoming modules to support their `__needs_xxx` macros. That means that they won\u0027t get precompiled into their modules\u0027 pcm, and instead their declarations will go into every other pcm that uses them. For now that\u0027s ok since the type merger can handle the declarations in these headers, but it\u0027s suboptimal at best. Make separate headers for all of the pieces so that they can be properly modularized.\n\nReviewed By: aaron.ballman, ChuanqiXu\n\nDifferential Revision: https://reviews.llvm.org/D158709\n\nGitOrigin-RevId: 680da4b5d7ec05a1e6656f2a7603a7b843268bab\n" }, { "commit": "fdc0cff65d4482218f748d44369cfe4b876fcca8", "tree": "879ba1eca534358e35886a24a42564188dbcba51", "parents": [ "e6f09ab8606bb76f8acd0c17bb29469f722bed99" ], "author": { "name": "Caslyn Tonelli", "email": "caslyn@google.com", "time": "Wed Apr 26 22:15:21 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Apr 27 14:56:04 2023 -0700" }, "message": "[gwp_asan] Exclude recoverable tests on Fuchsia\n\nSignal handlers used by recoverable tests are unsupported on Fuchsia.\n\nExclude the set of tests that test recoverable code paths (i.e.\nBacktraceGuardedPoolAllocator tests in recoverable.cpp) and always set\nthe `Recoverable` testing bool to `false` on the Fuchsia platform.\n\nDifferential Revision: https://reviews.llvm.org/D149311\n\nGitOrigin-RevId: 43ba32109657b697c6dd31a30563a66dade5f254\n" }, { "commit": "e6f09ab8606bb76f8acd0c17bb29469f722bed99", "tree": "3ea74d8d51de282bb97feca13d653b4630247f9f", "parents": [ "e7541948abb500e78daa26dd81f144adbc13b502" ], "author": { "name": "Caslyn Tonelli", "email": "caslyn@google.com", "time": "Thu Mar 30 00:03:05 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Apr 18 15:03:34 2023 -0700" }, "message": "[gwp_asan] Employ EXPECT_DEATH for zxtest compatibility\n\nEmploy a similar tactic introduced by https://reviews.llvm.org/D94362\nfor gwp_asan tests. zxtest `ASSERT_DEATH` syntax differs from gtest in\nthat it expects a lambda.\n\nzxtest does not have `EXPECT_DEATH`, so it introduced for Fuchsia builds\nand wraps the expression with a lambda to create a compatible syntax\nbetween zxtest and gtest for death tests.\n\nAn example of where this compatiblity is needed is in\n`never_allocated.cpp`.\n\nDifferential Revision: https://reviews.llvm.org/D147189\n\nGitOrigin-RevId: f2a1726d2199150918430a84f29cf8835bfe8e5a\n" }, { "commit": "e7541948abb500e78daa26dd81f144adbc13b502", "tree": "fe3d3da933dca07b219a713c467b1893ec4ef20f", "parents": [ "66fafe4adc77686aa6a19c8f99607e5526091b36" ], "author": { "name": "Alex Brachet", "email": "abrachet@google.com", "time": "Wed Mar 01 18:54:05 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Mar 01 10:56:58 2023 -0800" }, "message": "[GWP-ASan][Fuchsia] Use more modern zxtest testing macros\n\nGitOrigin-RevId: 21fc42b2d594986a4ae28475b10c75bfe10d3bd2\n" }, { "commit": "66fafe4adc77686aa6a19c8f99607e5526091b36", "tree": "26b2d53f02b7880ef3238569a39f2e2481343026", "parents": [ "83ecb6a9613dd07572b64bbe537d5c25dd2b2b90" ], "author": { "name": "Alex Brachet", "email": "abrachet@google.com", "time": "Wed Mar 01 18:49:18 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Mar 01 10:56:51 2023 -0800" }, "message": "[GWP-ASan] Stop using type aliases for ::testing:: types\n\nGitOrigin-RevId: b27f657f89c605f635f79ebcbacd8a729f8e1bff\n" }, { "commit": "83ecb6a9613dd07572b64bbe537d5c25dd2b2b90", "tree": "b8c325c1b2ab4d6fb3c03d85f33691a9a4b6fbe7", "parents": [ "f36df953a3bab8b8e54e1ce7214cb710a5605a1a" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Feb 28 14:05:42 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Feb 28 14:21:08 2023 -0800" }, "message": "[GWP-ASan] Handle wild touches of the guarded pool.\n\nAllocMeta could be null when returned from __gwp_asan_get_metadata() for\na bad access into the GuardedPagePool that was never allocated.\nCurrently, then we dereference the null pointer, oops.\n\nHoist the check up and print a message (only once in recoverable mode)\nabout the bad memory access.\n\nReviewed By: fmayer\n\nDifferential Revision: https://reviews.llvm.org/D144973\n\nGitOrigin-RevId: bc949f923ee37e6c71b69d0c0337d0e6b499832d\n" }, { "commit": "f36df953a3bab8b8e54e1ce7214cb710a5605a1a", "tree": "29001fa60f7a2caac205f2aa96e2ed20534430cd", "parents": [ "130625efe629f3aa66b700a2b16e92baea0df6ad" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Feb 02 15:37:33 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Feb 02 15:42:52 2023 -0800" }, "message": "[GWP-ASan] Remove thread clamping in tests.\n\nIt\u0027s better and easier for us to just have threads contend against each\nother in the tests if it\u0027s more than the maximum supported number of\nhardware threads available.\n\nSpecifically, the recoverable test fails on Android because the\nGTEST_SKIP in a called function, and it only properly works from the\nTEST_* harness function. Android tests run on cuttlefish, which can be a\nsingle core with two hyperthreads.\n\nReviewed By: fmayer\n\nDifferential Revision: https://reviews.llvm.org/D143221\n\nGitOrigin-RevId: 10939d1d580b9d3c9c2f3539c6bdb39f408179c0\n" }, { "commit": "130625efe629f3aa66b700a2b16e92baea0df6ad", "tree": "9d601feb76fce8c579d4fb7eb741589d01237f3d", "parents": [ "ab06386d19dd37004640593c2edc402f351ce831" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jan 12 16:01:06 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 17 10:25:58 2023 -0800" }, "message": "Reland: [GWP-ASan] Add recoverable mode.\n\nThe GWP-ASan recoverable mode allows a process to continue to function\nafter a GWP-ASan error is detected. The error will continue to be\ndumped, but GWP-ASan now has APIs that a signal handler (like the\nexample optional crash handler) can call in order to allow the\ncontinuation of a process.\n\nWhen an error occurs with an allocation, the slot used for that\nallocation will be permanently disabled. This means that free() of that\npointer is a no-op, and use-after-frees will succeed (writing and\nreading the data present in the page).\n\nFor heap-buffer-overflow/underflow, the guard page is marked as accessible\nand buffer-overflows will succeed (writing and reading the data present\nin the now-accessible guard page). This does impact adjacent\nallocations, buffer-underflow and buffer-overflows from adjacent\nallocations will no longer touch an inaccessible guard page. This could\nbe improved in future by having two guard pages between each adjacent\nallocation, but that\u0027s out of scope of this patch.\n\nEach allocation only ever has a single error report generated. It\u0027s\nwhatever came first between invalid-free, double-free, use-after-free or\nheap-buffer-overflow, but only one.\n\nReviewed By: eugenis, fmayer\n\nDifferential Revision: https://reviews.llvm.org/D140173\n\nGitOrigin-RevId: 35b5499d7259ac3e5c648a711678290695703a87\n" }, { "commit": "ab06386d19dd37004640593c2edc402f351ce831", "tree": "6621be5ceea5fb6b6b140df475af32100be7b626", "parents": [ "4a2c1cad91e6722b1c21e0e1ae706f5e12d64a12" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jan 12 15:58:36 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 12 16:06:09 2023 -0800" }, "message": "Revert \"[GWP-ASan] Add recoverable mode.\"\n\nThis reverts commit 90a9beb7cc9755791caa23dfc4e36bc544e98ed3.\n\nReason: Broke an internal build.\nGitOrigin-RevId: 7cc12cb6c838e928d24d8ccb8449ccab415ac4ad\n" }, { "commit": "4a2c1cad91e6722b1c21e0e1ae706f5e12d64a12", "tree": "438b2d7f68e6be4852b5c866503a0ca1c09dfa57", "parents": [ "a2937b081cb83a74836f93850e8b2ddc8153bf89" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jan 12 15:58:10 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 12 16:06:02 2023 -0800" }, "message": "Revert \"[GWP-ASan] Fix test to work with Fuchsia\u0027s zxtest\"\n\nThis reverts commit 64138725e4cf0ab5e3f9c0dba38e11ce3d3b4017.\n\nParent change https://reviews.llvm.org/D140173 broke an internal build.\nI\u0027ll land this bit as part of the re-land for\nhttps://reviews.llvm.org/D140173.\n\nGitOrigin-RevId: d99120939ecbe686d418fc29fdac9bc6574b5f30\n" }, { "commit": "a2937b081cb83a74836f93850e8b2ddc8153bf89", "tree": "0f396389aa89ed9b74a892d5ef18280e0ee2ad4f", "parents": [ "5cd0466acd74fc10f074c2076abdf122022c9c3f" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jan 12 15:57:27 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jan 12 16:05:55 2023 -0800" }, "message": "Revert \"[GWP-ASan] Fix 6413872\"\n\nThis reverts commit df87e62cbd4b5090b5349247d29457706d2ac4e8.\n\nParent change https://reviews.llvm.org/D140173 broke an internal build.\nI\u0027ll integrate this patch into the re-land for\nhttps://reviews.llvm.org/D140173.\n\nGitOrigin-RevId: 4c0b3964cbc0f8366903e5fb5784af01b79545a1\n" }, { "commit": "5cd0466acd74fc10f074c2076abdf122022c9c3f", "tree": "0ba76118c4a2ab5a384da80daabce8c57f732fbc", "parents": [ "b00d6c0e34a702911568e17932b852e523d46822" ], "author": { "name": "Alex Brachet", "email": "abrachet@google.com", "time": "Wed Jan 11 23:42:19 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 11 15:45:53 2023 -0800" }, "message": "[GWP-ASan] Fix 6413872\n\nUse testing not zxtest in non-Fuchsia case\n\nGitOrigin-RevId: df87e62cbd4b5090b5349247d29457706d2ac4e8\n" }, { "commit": "b00d6c0e34a702911568e17932b852e523d46822", "tree": "0f396389aa89ed9b74a892d5ef18280e0ee2ad4f", "parents": [ "f4272dd5b0ef82680f02152168b8ec19f6048f12" ], "author": { "name": "Alex Brachet", "email": "abrachet@google.com", "time": "Wed Jan 11 23:16:19 2023 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 11 15:20:34 2023 -0800" }, "message": "[GWP-ASan] Fix test to work with Fuchsia\u0027s zxtest\n\nGitOrigin-RevId: 64138725e4cf0ab5e3f9c0dba38e11ce3d3b4017\n" }, { "commit": "f4272dd5b0ef82680f02152168b8ec19f6048f12", "tree": "438b2d7f68e6be4852b5c866503a0ca1c09dfa57", "parents": [ "4079a1c1f02525415bea11cdaf804fb7635c7de8" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Jan 11 12:55:38 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Jan 11 13:15:28 2023 -0800" }, "message": "[GWP-ASan] Add recoverable mode.\n\nThe GWP-ASan recoverable mode allows a process to continue to function\nafter a GWP-ASan error is detected. The error will continue to be\ndumped, but GWP-ASan now has APIs that a signal handler (like the\nexample optional crash handler) can call in order to allow the\ncontinuation of a process.\n\nWhen an error occurs with an allocation, the slot used for that\nallocation will be permanently disabled. This means that free() of that\npointer is a no-op, and use-after-frees will succeed (writing and\nreading the data present in the page).\n\nFor heap-buffer-overflow/underflow, the guard page is marked as accessible\nand buffer-overflows will succeed (writing and reading the data present\nin the now-accessible guard page). This does impact adjacent\nallocations, buffer-underflow and buffer-overflows from adjacent\nallocations will no longer touch an inaccessible guard page. This could\nbe improved in future by having two guard pages between each adjacent\nallocation, but that\u0027s out of scope of this patch.\n\nEach allocation only ever has a single error report generated. It\u0027s\nwhatever came first between invalid-free, double-free, use-after-free or\nheap-buffer-overflow, but only one.\n\nReviewed By: eugenis, fmayer\n\nDifferential Revision: https://reviews.llvm.org/D140173\n\nGitOrigin-RevId: 90a9beb7cc9755791caa23dfc4e36bc544e98ed3\n" }, { "commit": "4079a1c1f02525415bea11cdaf804fb7635c7de8", "tree": "6621be5ceea5fb6b6b140df475af32100be7b626", "parents": [ "ebca419199eb7fe574da5bc4975ddd0e4ccc2640" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Jan 10 10:29:49 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 10 10:36:27 2023 -0800" }, "message": "[GWP-ASan] Fix up bad report for in-page underflow w/ UaF\n\nComplex scenario, but reports when there\u0027s both a use-after-free and\nbuffer-underflow that is in-page (i.e. doesn\u0027t touch the guard page)\nended up generating a pretty bad report:\n\n\u0027Use After Free at 0x7ff392e88fef (18446744073709551615 bytes into a\n1-byte allocation at 0x7ff392e88ff0) by thread 3836722 here:\u0027\n\n(note the 2^64-bytes-into-alloc, very cool and good!)\n\nFix up that case, and add a diagnostic about when you have both a\nuse-after-free and a buffer-overflow that it\u0027s probably a bogus report\n(assuming the developer didn\u0027t *really* screw up and have a uaf+overflow\nbug at the same time).\n\nReviewed By: vitalybuka\n\nDifferential Revision: https://reviews.llvm.org/D139885\n\nGitOrigin-RevId: dcf23e13615f88bdd4975058595ee60cf1d5811c\n" }, { "commit": "ebca419199eb7fe574da5bc4975ddd0e4ccc2640", "tree": "3b88f574c4d1094febe64359390d9dcaca6b5e9c", "parents": [ "2ad1a5e48efec766678f661b5206e35fa9ab5008" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Jan 10 10:16:03 2023 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 10 10:19:15 2023 -0800" }, "message": "[GWP-ASan] Fix atfork handlers being installed multiple times in tests\n\nWe incorrectly install the atfork handlers multiple times in the test\nharness, tracked down to the default parameter used by\nCheckLateInitIsOK. This manifested in a hang if running the tests with\n--gtest_repeat\u003d{\u003e\u003d2} as the atfork handler ran multiple times, causing\ndouble-lock and double-unlock, which on my machine hung.\n\nAdd a check-fail for this case as well to prevent this from happening\nagain (it was difficult to track down and is an easy mistake to make).\n\nDifferential Revision: https://reviews.llvm.org/D139731\n\nGitOrigin-RevId: 296f7fbbb5c48a961aa31b2ffa1d048e6bfbd7a1\n" }, { "commit": "2ad1a5e48efec766678f661b5206e35fa9ab5008", "tree": "bab21c8c44ff3125d7e8975f719004e77283b2ac", "parents": [ "7d5a8d77a1923bbed636f279a79800c9faaffb5a" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Dec 12 13:53:08 2022 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Dec 12 14:12:23 2022 -0800" }, "message": "[GWP-ASan] Fix sanitizer backtrace util using wrong print.\n\nThe sanitizer backtrace is used in tests, and calling StackTrace.Print()\nmeans that it uses the sanitizer\u0027s Printf(), rather than GWP-ASan\u0027s\nPrintf(). In the current code, GWP-ASan\u0027s Printf() *is* the sanitizer\nprint, but this isn\u0027t guaranteed to be the case, and will change in an\nupcoming patch.\n\nReviewed By: eugenis\n\nDifferential Revision: https://reviews.llvm.org/D139748\n\nGitOrigin-RevId: f529a9f324a8eb4a0f2c5e662b6c57a6a775d62f\n" }, { "commit": "7d5a8d77a1923bbed636f279a79800c9faaffb5a", "tree": "781b75facb55a670ed29327fb94fbe621f28790f", "parents": [ "01a219d9755d4fe99366d311a8a8ef16a13cb863" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed May 18 14:10:12 2022 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed May 18 14:14:35 2022 -0700" }, "message": "[NFCI] clang-format gwp-asan files.\n\nGitOrigin-RevId: f9a3c43eaa97d125595d83565a5d4aafd8dac734\n" }, { "commit": "01a219d9755d4fe99366d311a8a8ef16a13cb863", "tree": "3604a44356958c7994a2a046f85574d568be3a24", "parents": [ "2713bce2358c1e70ed17d6734135c30efb93eaa8" ], "author": { "name": "Gulfem Savrun Yeniceri", "email": "gulfem@google.com", "time": "Sat Feb 19 01:05:38 2022 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 18 17:21:16 2022 -0800" }, "message": "[gwp_asan] Add missing \u003calgorithm\u003e include\n\nAfter https://reviews.llvm.org/D119667, \u003calgorithm\u003e is no longer\ntransitively included from various headers. This patch adds the\n\u003calgorithm\u003e include into gwp_asan.\n\nDifferential Revision: https://reviews.llvm.org/D120172\n\nGitOrigin-RevId: 4961bb477d0526a7b965609cd42d44caa8413d45\n" }, { "commit": "2713bce2358c1e70ed17d6734135c30efb93eaa8", "tree": "03997def045aae39a0b4f80340113f5a911d4ca1", "parents": [ "cb1ba0f0858aa7277f0da748b14df0ec2097bfa9" ], "author": { "name": "David Blaikie", "email": "dblaikie@gmail.com", "time": "Sat Oct 16 22:43:57 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sat Oct 16 22:50:22 2021 -0700" }, "message": "Fix a few warnings (signed/unsigned comparison in gtest, and missing field initializers)\n\nGitOrigin-RevId: 6176fda3f992b5086302b3826aa0636135cc4cc0\n" }, { "commit": "cb1ba0f0858aa7277f0da748b14df0ec2097bfa9", "tree": "18545ad58e6e07d59651839c5eb1d531f12b2b4f", "parents": [ "9d985453dc072362dc57e0837d0c9cef214cd57e" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Mon Sep 27 12:31:59 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Sep 27 13:55:21 2021 -0700" }, "message": "[gwp-asan] Initialize AllocatorVersionMagic at runtime\n\nGWP-ASan\u0027s `AllocatorState` was recently extended with a\n`AllocatorVersionMagic` structure required so that GWP-ASan bug reports\ncan be understood by tools at different versions.\n\nOn Fuchsia, this in included in the `scudo::Allocator` structure, and\nby having non-zero initializers, this effectively moved the static\nallocator structure from the `.bss` segment to the `.data` segment, thus\nincreasing (significantly) the size of the libc.\n\nThis CL proposes to initialize the structure with its magic numbers at\nruntime, allowing for the allocator to go back into the `.bss` segment.\n\nI will work on adding a test on the Scudo side to ensure that this type\nof changes get detected early on. Additional work is also needed to\nreduce the footprint of the (large) memory-tagging related structures\nthat are currently part of the allocator.\n\nDifferential Revision: https://reviews.llvm.org/D110575\n\nGitOrigin-RevId: 04f5913395de23a5f6745156021ab10a4a0039de\n" }, { "commit": "9d985453dc072362dc57e0837d0c9cef214cd57e", "tree": "ffa992c172293bc5b3561e2ecff3be99dd8d2072", "parents": [ "d580434ff5f52ba30c08211f8f9f7364038c68f9" ], "author": { "name": "Frederic Cambus", "email": "fred@statdns.com", "time": "Fri Sep 24 19:04:16 2021 +0530" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Sep 24 06:46:33 2021 -0700" }, "message": "[compiler-rt] Use portable \"#!/usr/bin/env bash\" shebang for tests.\n\nIn build_symbolizer.sh we can safely remove the -eu argument from the shebang (which is an unportable construct), as the scripts sets **-e** and **-u** already.\n\nDifferential Revision: https://reviews.llvm.org/D110039\n\nGitOrigin-RevId: 626e2a6c6236d2fd7582928a0363d381c55eb43d\n" }, { "commit": "d580434ff5f52ba30c08211f8f9f7364038c68f9", "tree": "56d6c54e038095e3bd0fc1fa5aca367e2423fed6", "parents": [ "61c099ca0862f75aad8f3084bb53f254f3a34983" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jul 29 14:52:33 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 29 19:35:13 2021 -0700" }, "message": "[GWP-ASan] Add version header.\n\nAdds magic version header to AllocatorState. This can be used by\nout-of-process crash handlers, like Crashpad on Fuchsia, to do offline\nreconstruction of GWP-ASan crash metadata.\n\nCrashpad on Fuchsia is intending on dumping the AllocationMetadata pool\nand the AllocatorState directly into the minidump. Then, using the\nversion number, they can unpack the data on serverside using a versioned\nunpack tool.\n\nAlso add some asserts to make sure the version number gets bumped if the\ninternal structs get changed.\n\nReviewed By: eugenis, mcgrathr\n\nDifferential Revision: https://reviews.llvm.org/D106690\n\nGitOrigin-RevId: 8e167f66b27fe9d2573eb149f736700302675297\n" }, { "commit": "61c099ca0862f75aad8f3084bb53f254f3a34983", "tree": "e2ccbc41278f65c7bb11667e44128d4bd24763b3", "parents": [ "868b9a2250cb6cd990af6bda6a7b04b26d15a7a6" ], "author": { "name": "Alexander Belyaev", "email": "pifon@google.com", "time": "Mon Jul 19 14:17:13 2021 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 29 19:35:08 2021 -0700" }, "message": "[rt][nfc] Rewrite #ifndef as #if defined().\n\nGitOrigin-RevId: f58a1f65e74686b7a59ae2fa849ce4c70d520e89\n" }, { "commit": "868b9a2250cb6cd990af6bda6a7b04b26d15a7a6", "tree": "7fa28c6881699e88882d3b028a92ba6d22213acb", "parents": [ "0894fec50135a5eea00429fb6722a2b384d29297" ], "author": { "name": "David Spickett", "email": "david.spickett@linaro.org", "time": "Mon Jul 19 10:43:21 2021 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 29 19:35:03 2021 -0700" }, "message": "[compiler-rt][GWP-ASAN] Disable 2 tests on Armv7 Linux\n\nThese have been failing on our bots for a while due to\nincomplete backtraces. (you don\u0027t get the names of the\nfunctions that did the access, just the reporter frames)\n\nSee:\nhttps://lab.llvm.org/buildbot/#/builders/170/builds/180\nGitOrigin-RevId: 3d5c1a81738147104a2146e2457dc2f4cc8f3e05\n" }, { "commit": "0894fec50135a5eea00429fb6722a2b384d29297", "tree": "44d5e668f96836ab16ce97ede7016d82e058015f", "parents": [ "c5ac5d72557a6361acfd6b250fa5eb2248511ae1" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jul 08 13:06:34 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 15 20:05:17 2021 -0700" }, "message": "[GWP-ASan] Change unreachable -\u003e trap to work around DCE bug.\n\ntrapOnAddress is designed to SEGV on a specific address. Unfortunately,\nwith an IR change, __builtin_unreachable() ends up doing DCE on things\nthat have side effects, like the load that causes the trap.\n\nChange to __builtin_trap() to avoid the optimisation.\n\nRoot cause is still an LLVM bug, and tracked in\nhttps://bugs.llvm.org/show_bug.cgi?id\u003d47480.\n\nReviewed By: eugenis\n\nDifferential Revision: https://reviews.llvm.org/D105654\n\nGitOrigin-RevId: d458f379324967c3c408be06e21aad9bc92c54cb\n" }, { "commit": "c5ac5d72557a6361acfd6b250fa5eb2248511ae1", "tree": "befcdc2eff7e6effcaa605e5c9c274d677094a34", "parents": [ "fc8018b0923a8e4d10c48ab19d66f28eed4436ea" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Fri Jul 02 13:50:52 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 15 20:05:12 2021 -0700" }, "message": "Name all DEATH tests using \u0027DeathTest\u0027 suffix.\n\ngtest highly recommends this prefix, and runs death tests first\n(https://github.com/google/googletest/blob/master/docs/advanced.md#death-test-naming).\nThis may help with some spurious bot failures like\nhttps://lab.llvm.org/buildbot/#/builders/169/builds/1290/steps/25/logs/stdio.\n\nReviewed By: cryptoad, vitalybuka\n\nDifferential Revision: https://reviews.llvm.org/D105371\n\nGitOrigin-RevId: dd1c4bd09dd7a48c744f58847862f2e2bd633477\n" }, { "commit": "fc8018b0923a8e4d10c48ab19d66f28eed4436ea", "tree": "96eb1e160ea72074c95084b4d543a9e773f2e69a", "parents": [ "eb4d3dc5d899248eb9b9b421c61025a94e47d548" ], "author": { "name": "Vitaly Buka", "email": "vitalybuka@google.com", "time": "Wed Jun 30 14:27:53 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Jul 15 20:05:04 2021 -0700" }, "message": "[gwp_asan] Improve symbolizer script\n\nShow inlined functions.\nHide unhelpful DWARF related warnings.\n\nReviewed By: hctim\n\nDifferential Revision: https://reviews.llvm.org/D105230\n\nGitOrigin-RevId: 7d207472030a244d540fa53e47a734675b289a94\n" }, { "commit": "eb4d3dc5d899248eb9b9b421c61025a94e47d548", "tree": "a3d813d83e9d0a92f3bd0581eb08abe709b21922", "parents": [ "ae91f88371030bcf3e8a1759e3e5af93fd839eaa" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon May 10 12:19:19 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue May 25 19:12:25 2021 -0700" }, "message": "[Scudo] Use GWP-ASan\u0027s aligned allocations and fixup postalloc hooks.\n\nThis patch does a few cleanup things:\n 1. The non-standalone scudo has a problem where GWP-ASan allocations\n may not meet alignment requirements where Scudo was requested to have\n alignment \u003e\u003d 16. Use the new GWP-ASan API to fix this.\n 2. The standalone variant loses some debugging information inside of\n GWP-ASan because we ask GWP-ASan to allocate an aligned size in the\n frontend. This means reports end up with \u0027UaF on a 16-byte allocation\u0027\n for a 1-byte allocation with 16-byte alignment. Also use the new API to\n fix this.\n 3. Add post-alloc hooks for GWP-ASan intercepted allocations, and add\n stats tracking for GWP-ASan allocations.\n 4. Add a small test that checks the alignment of the frontend\n allocator, so that it can be used under GWP-ASan torture mode.\n 5. Add GWP-ASan torture mode as a testing configuration to catch these\n regressions.\n\nDepends on D94830, D95889.\n\nReviewed By: cryptoad\n\nDifferential Revision: https://reviews.llvm.org/D95884\n\nGitOrigin-RevId: e78b64df98878d1da56275e0c272ed58364da3ad\n" }, { "commit": "ae91f88371030bcf3e8a1759e3e5af93fd839eaa", "tree": "e3e17db130946db9ad91e34711517e9417701850", "parents": [ "9a4e8c989e3deaae6fe97aed00b87fe139944055" ], "author": { "name": "Roland McGrath", "email": "mcgrathr@google.com", "time": "Wed Apr 28 16:45:20 2021 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Sun May 09 19:10:34 2021 -0700" }, "message": "[gwp_asan] Use __sanitizer_fast_backtrace on Fuchsia\n\nReviewed By: phosek, cryptoad, hctim\n\nDifferential Revision: https://reviews.llvm.org/D101407\n\nGitOrigin-RevId: 3341324d82f1278f91b8704048aff414abeaacdb\n" }, { "commit": "9a4e8c989e3deaae6fe97aed00b87fe139944055", "tree": "f4400529af50df8e9f781d598f40928fd43eaecc", "parents": [ "154a61bdbcab3a5d1408e4151226c46aa956c3f4" ], "author": { "name": "Fangrui Song", "email": "i@maskray.me", "time": "Wed Feb 17 20:03:16 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 04:06:44 2021 -0800" }, "message": "[GWP-ASan] Change sys/cdefs.h to features.h\n\nsys/cdefs.h is a glibc internal header which is not supposed to be included by applications.\n(Some libc implementations provide this file for compatibility.)\nAndroid features.h includes sys/cdefs.h, so we can include features.h instead.\n\nThis change makes `ninja gwp_asan` build on musl.\n\nGitOrigin-RevId: da59c2e4dc1c7b502cc2098247b2ee38ac297e0c\n" }, { "commit": "154a61bdbcab3a5d1408e4151226c46aa956c3f4", "tree": "79d20641fbfcf1fa976e0c5d46e978650446d0f2", "parents": [ "bedb30c43523824366b4ce370d5098afce81ea6e" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 10 08:38:34 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 19 04:06:40 2021 -0800" }, "message": "[GWP-ASan] Add back some headers removed by IWYU.\n\nThese headers are required for Android.\n\nReviewed By: eugenis\n\nDifferential Revision: https://reviews.llvm.org/D96374\n\nGitOrigin-RevId: b93786907c54df4ca6658c569624a6460f72ce1e\n" }, { "commit": "bedb30c43523824366b4ce370d5098afce81ea6e", "tree": "f2992d22f30625d30cfaafdd230dfc317595d5d4", "parents": [ "30fc6a50bc04d2125056b95011acf84d2717d0c0" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Feb 08 10:46:45 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Feb 08 14:43:24 2021 -0800" }, "message": "[GWP-ASan] Add aligned allocations.\n\nAdds a new allocation API to GWP-ASan that handles size+alignment\nrestrictions.\n\nReviewed By: cryptoad, eugenis\n\nDifferential Revision: https://reviews.llvm.org/D94830\n\nGitOrigin-RevId: 3d8823b8e48a3f064b1e2dd52881b3ac581f6f2b\n" }, { "commit": "30fc6a50bc04d2125056b95011acf84d2717d0c0", "tree": "6513b2fa0cb1271497896ef490e58b4bf48b7367", "parents": [ "e825c8bf2b952d77d355546e56714a22462473a2" ], "author": { "name": "Roland McGrath", "email": "mcgrathr@google.com", "time": "Wed Feb 03 10:31:45 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 05 03:26:22 2021 -0800" }, "message": "[gwp_asan] Use zx_system_get_page_size() on Fuchsia\n\nFuchsia is migrating to a variable page size.\n\nReviewed By: hctim\n\nDifferential Revision: https://reviews.llvm.org/D95920\n\nGitOrigin-RevId: 09fe23a61c62da68ff776ec0b69cb9c7477c9ad5\n" }, { "commit": "e825c8bf2b952d77d355546e56714a22462473a2", "tree": "58abcb18253d97f4d582d693032dfaeb1e878c20", "parents": [ "4cd24926383436a663b21da8d93cabd33d471621" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Feb 02 14:46:25 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 05 03:26:19 2021 -0800" }, "message": "Revert \"[GWP-ASan] Add aligned allocations.\"\n\nThis reverts commit 9dc06762470cb5a6cde8de5833cb75262e1bacb0.\n\nShould not have been committed, was a bad rebase. Review still\ncontinuing on D94830.\n\nGitOrigin-RevId: 0dcf3324cfb4429e85b54e857f9bb86f423ffc5e\n" }, { "commit": "4cd24926383436a663b21da8d93cabd33d471621", "tree": "7c39d4b28fb97396012a53c49702847103f9e5ed", "parents": [ "726be38d7400cf81fa9c0fbcd3106b770db1ee70" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Feb 02 14:32:28 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 05 03:26:16 2021 -0800" }, "message": "[GWP-ASan] Add locking around unwinder for atfork protection.\n\nUnwinders (like libc\u0027s backtrace()) can call their own locks (like the\nlibdl lock). We need to let the unwinder release the locks before\nforking. Wrap a new lock around the unwinder for atfork protection.\n\nReviewed By: eugenis\n\nDifferential Revision: https://reviews.llvm.org/D95889\n\nGitOrigin-RevId: 30973f6fe01cc0a9624147466f0c54b91a1b61d7\n" }, { "commit": "726be38d7400cf81fa9c0fbcd3106b770db1ee70", "tree": "12d80ee1c77c98ecc1208ad219bffe586cea74b5", "parents": [ "aea9a377213b45e26f9ca69a89bc05fc11d3889a" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Jan 14 15:16:56 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Fri Feb 05 03:26:12 2021 -0800" }, "message": "[GWP-ASan] Add aligned allocations.\n\nAdds a new allocation API to GWP-ASan that handles size+alignment\nrestrictions.\n\nDifferential Revision: https://reviews.llvm.org/D94830\n\nGitOrigin-RevId: 9dc06762470cb5a6cde8de5833cb75262e1bacb0\n" }, { "commit": "aea9a377213b45e26f9ca69a89bc05fc11d3889a", "tree": "2350fe1687fc18bec52e37a3d62adc42e8461943", "parents": [ "b3132884bdb15c09d5f2a3931eaae1fbee932a26" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Fri Jan 15 12:57:00 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 19 17:25:28 2021 -0800" }, "message": "[GWP-ASan] Add inbuilt options parser.\n\nAdds a modified options parser (shamefully pulled from Scudo, which\nshamefully pulled it from sanitizer-common) to GWP-ASan. This allows\ncustomers (Android) to parse options strings in a common way.\n\nDepends on D94117.\n\nAOSP side of these patches is staged at:\n\n - sepolicy (sysprops should only be settable by the shell, in both root and\n unrooted conditions):\n https://android-review.googlesource.com/c/platform/system/sepolicy/+/1517238\n\n - zygote updates:\n https://android-review.googlesource.com/c/platform/frameworks/base/+/1515009\n\n - bionic changes to add `gwp_asan.\u003cprocess_name\u003e` system property, and\n GWP_ASAN_OPTIONS environment variable:\n https://android-review.googlesource.com/c/platform/bionic/+/1514989\n\nReviewed By: eugenis\n\nDifferential Revision: https://reviews.llvm.org/D92696\n\nGitOrigin-RevId: 6a42cbf6d2116b52cb59aa3e23bef93a30cf2dc8\n" }, { "commit": "b3132884bdb15c09d5f2a3931eaae1fbee932a26", "tree": "27fae005c2546dbfc08ee39d09df60c2bd8f0856", "parents": [ "3a2adbe34caad3c8503d7ec0077cd73ba668c0fe" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Jan 13 17:28:10 2021 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Jan 19 17:25:25 2021 -0800" }, "message": "[GWP-ASan] Minor refactor of optional components.\n\nIn preparation for the inbuilt options parser, this is a minor refactor\nof optional components including:\n - Putting certain optional elements in the right header files,\n according to their function and their dependencies.\n - Cleaning up some old and mostly-dead code.\n - Moving some functions into anonymous namespaces to prevent symbol\n export.\n\nReviewed By: cryptoad, eugenis\n\nDifferential Revision: https://reviews.llvm.org/D94117\n\nGitOrigin-RevId: a8520f6970fd4d44ceae1ec5969563a0dbe30f54\n" }, { "commit": "3a2adbe34caad3c8503d7ec0077cd73ba668c0fe", "tree": "1ab91f11a8e8a4d95c04bb4d2aeca3a172d91f1e", "parents": [ "adf85b2db027f61a43c6bd44efaa27e22cf03644" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Dec 10 12:41:56 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Dec 15 00:12:18 2020 -0800" }, "message": "[GWP-ASan] IWYU \u0026 clang-format\n\nRun an IWYU pass and clang-format GWP-ASan code.\n\nReviewed By: eugenis, mcgrathr\n\nDifferential Revision: https://reviews.llvm.org/D92688\n\nGitOrigin-RevId: 61a038f8528f12c0c2ee5a9794c257fdae626d29\n" }, { "commit": "adf85b2db027f61a43c6bd44efaa27e22cf03644", "tree": "ecc26f6c58fc6fdb94880afc5b79e941828cf4c4", "parents": [ "a0c7a18d299c2e250498a5d9a60453b0c1a231ec" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Tue Dec 01 11:46:23 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Tue Dec 08 19:23:03 2020 -0800" }, "message": "[GWP-ASan] Fix flaky test on Fuchsia\n\nThe LateInit test might be reusing some already initialized thread\nspecific data if run within the main thread. This means that there\nis a chance that the current value will not be enough for the 100\niterations, hence the test flaking.\n\nFix this by making the test run in its own thread.\n\nDifferential Revision: https://reviews.llvm.org/D92415\n\nGitOrigin-RevId: c904c32b9c9243b11ffc18e46b7350f000e9c088\n" }, { "commit": "a0c7a18d299c2e250498a5d9a60453b0c1a231ec", "tree": "c5088cb85851729a4e532611660199d5fa51e842", "parents": [ "93825cbc29457da75326e5e179696e0f5aa9daec" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Mon Nov 16 15:34:14 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 30 05:26:40 2020 -0800" }, "message": "[GWP-ASan] Port tests to Fuchsia\n\nThis modifies the tests so that they can be run on Fuchsia:\n- add the necessary includes for `set`/`vector` etc\n- do the few modifications required to use zxtest instead og gtest\n\n`backtrace.cpp` requires stacktrace support that Fuchsia doesn\u0027t have\nyet, and `enable_disable.cpp` currently uses `fork()` which Fuchsia\ndoesn\u0027t support yet. I\u0027ll revisit this later.\n\nI chose to use `harness.h` to hold my \"platform-specific\" include and\nnamespace, and using this header in tests rather than `gtest.h`,\nwhich I am open to change if someone would rather go another direction.\n\nDifferential Revision: https://reviews.llvm.org/D91575\n\nGitOrigin-RevId: 5556616b5b5223f95607ad94053a55f0deaf2762\n" }, { "commit": "93825cbc29457da75326e5e179696e0f5aa9daec", "tree": "ca79aabc316c3b1426680b789b29cb846f926f5d", "parents": [ "48a09ebdd63bd5c0f03c008b2d8d245550774e46" ], "author": { "name": "Roland McGrath", "email": "mcgrathr@google.com", "time": "Wed Nov 18 09:29:30 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 30 05:26:37 2020 -0800" }, "message": "[GWP-ASan] Respect GWP_ASAN_DEFAULT_ENABLED compile-time macro\n\nIf the containing allocator build uses -DGWP_ASAN_DEFAULT_ENABLED\u003dfalse\nthen the option will default to false. For e.g. Scudo, this is simpler\nand more efficient than using -DSCUDO_DEFAULT_OPTIONS\u003d... to set gwp-asan\noptions that have to be parsed from the string at startup.\n\nReviewed By: hctim\n\nDifferential Revision: https://reviews.llvm.org/D91463\n\nGitOrigin-RevId: 7810d837864933b49f5e20a970a8e51ea091fa8e\n" }, { "commit": "48a09ebdd63bd5c0f03c008b2d8d245550774e46", "tree": "84a22c9c077a6a72abdc920865d52a9e075bf290", "parents": [ "7dee7eee495bf937c1d6941155585e53c6df65f4" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Sat Oct 31 11:09:29 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 02 20:49:11 2020 -0800" }, "message": "[GWP-ASan] Stub out backtrace/signal functions on Fuchsia\n\nThe initial version of GWP-ASan on Fuchsia doesn\u0027t support crash and\nsignal handlers, so this just adds empty stubs to be able to compile\nthe project on the platform.\n\nDifferential Revision: https://reviews.llvm.org/D90537\n\nGitOrigin-RevId: 98fe39df939ee6f1fd20b75edcc9fe0860b52dba\n" }, { "commit": "7dee7eee495bf937c1d6941155585e53c6df65f4", "tree": "d9d5a10be89c41cabf11b7621df0789c0c66bece", "parents": [ "118bc3f105f9061c5a8e8cb7e68d253e9ca26190" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Fri Oct 30 10:42:28 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Mon Nov 02 11:22:14 2020 -0800" }, "message": "[GWP-ASan] Fuchsia specific mapping \u0026 utilities functions\n\nThis CL introduces the Fuchsia versions of the existing platform\nspecific functions.\n\nFor Fuchsia, we need to track the VMAR (https://fuchsia.dev/fuchsia-src/reference/kernel_objects/vm_address_region)\nof the Guarded Pool mapping, and for this purpose I added some platform\nspecific data structure that remains empty on POSIX platforms.\n\n`getThreadID` is not super useful for Fuchsia so it\u0027s just left as a\nstub for now.\n\nWhile testing the changes in my Fuchsia tree, I realized that\n`guarded_pool_allocator_tls.h` should have closed the namespace before\nincluding `GWP_ASAN_PLATFORM_TLS_HEADER`, otherwise drama ensues.\n\nThis was tested in g3, upstream LLVM, and Fuchsia (with local changes).\n\nDifferential Revision: https://reviews.llvm.org/D90483\n\nGitOrigin-RevId: 63ad0876567d7a782daedfe9f564942a37f063f5\n" }, { "commit": "118bc3f105f9061c5a8e8cb7e68d253e9ca26190", "tree": "3a1d51106cb0e6556af6c4c4cf4bf958f8de5d82", "parents": [ "952b37c94587a8da34ae593f4164aed3ce6b517f" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Wed Oct 28 16:04:02 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 19:43:20 2020 -0700" }, "message": "[GWP-ASan] Add mutexes for Fuchsia\n\nMitch expressed a preference to not have `#ifdef`s in platform agnostic\ncode, this change tries to accomodate this.\n\nI am not attached to the method this CL proposes, so if anyone has a\nsuggestion, I am open.\n\nWe move the platform specific member of the mutex into its own platform\nspecific class that the main `Mutex` class inherits from. Functions are\nimplemented in their respective platform specific compilation units.\n\nFor Fuchsia, we use the sync APIs, as those are also the ones being\nused in Scudo.\n\nDifferential Revision: https://reviews.llvm.org/D90351\n\nGitOrigin-RevId: 3591721ada99810aeddfa3d4c83eea0dbeed7f18\n" }, { "commit": "952b37c94587a8da34ae593f4164aed3ce6b517f", "tree": "727679c50df05259c7f3e750dc941352a4050d67", "parents": [ "738cb91c0b3ddd1589a4bdbb1c92dc365cb382a3" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Mon Oct 26 14:54:22 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:57 2020 -0700" }, "message": "[GWP-ASan] Abstract the thread local variables access\n\nIn a similar fashion to D87420 for Scudo, this CL introduces a way to\nget thread local variables via a platform-specific reserved TLS slot,\nsince Fuchsia doesn\u0027t support ELF TLS from the libc itself.\n\nIf needing to use this, a platform will have to define\n`GWP_ASAN_HAS_PLATFORM_TLS_SLOT` and provide `gwp_asan_platform_tls_slot.h`\nwhich will define a `uint64_t *getPlatformGwpAsanTlsSlot()` function\nthat will return the TLS word of storage.\n\nI snuck in a couple of cleanup items as well, moving some static\nfunctions to anonymous namespace for consistency.\n\nDifferential Revision: https://reviews.llvm.org/D90195\n\nGitOrigin-RevId: 90678f65ae47523586bd34392ed3cd1369cf5e9b\n" }, { "commit": "738cb91c0b3ddd1589a4bdbb1c92dc365cb382a3", "tree": "9989621e2dda945a6f0e7e8c309b75346f64f8e0", "parents": [ "7155761857beb7a82840cb8795369966d3e656eb" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Thu Oct 22 15:53:50 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:50 2020 -0700" }, "message": "[GWP-ASan] Refactor memory mapping functions\n\nIn preparation for Fuchsia support, this CL refactors the memory\nmapping functions.\n\nThe new functions are as follows:\n- for Freeslots and Metadata:\n `void *map(size_t Size, const char *Name) const;`\n `void unmap(void *Ptr, size_t Size) const;`\n- for the Pool:\n `void *reservePool(size_t Size);`\n `void commitPool(void *Ptr, size_t Size) const;`\n `void decommitPool(void *Ptr, size_t Size) const;`\n `void unreservePool();`\n Note that those don\u0027t need a `Name` parameter as those are fixed per\n function. `{reserve,unreserve}Pool` are not `const` because they will\n modify platform specific class member on Fuchsia.\n\nI added a plethora of `assert()` as the initial code was not enforcing\npage alignment for sizes and addresses, which caused problem in the\ninitial Fuchsia draft. All sizes should now be properly rounded up to\na page.\n\nDifferential Revision: https://reviews.llvm.org/D89993\n\nGitOrigin-RevId: 612e02ee8c3e8f204378796af2eb526cf5e348f0\n" }, { "commit": "7155761857beb7a82840cb8795369966d3e656eb", "tree": "365d10bae0f590110d7b3fbbbcc92f00ef8bbc28", "parents": [ "7edfc998e9f00e283ab4f7af79cf5b68608d554d" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Thu Oct 22 13:40:12 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:48 2020 -0700" }, "message": "[GWP-ASan] Move random-related code in the allocator (redo)\n\nThis is a redo of D89908, which triggered some `-Werror\u003dconversion`\nerrors with GCC due to assignments to the 31-bit variable.\n\nThis CL adds to the original one a 31-bit mask variable that is used\nat every assignment to silence the warning.\n\nDifferential Revision: https://reviews.llvm.org/D89984\n\nGitOrigin-RevId: 3580a45014e90a9895cc2dec28ab2e7f78c47470\n" }, { "commit": "7edfc998e9f00e283ab4f7af79cf5b68608d554d", "tree": "f432a7089082355a6ce27d7def6c0e1216ddf615", "parents": [ "72384c39dab1bd9bd98bc08a895c283bb128f09a" ], "author": { "name": "Nikita Popov", "email": "nikita.ppv@gmail.com", "time": "Thu Oct 22 21:56:37 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:44 2020 -0700" }, "message": "Revert \"[GWP-ASan] Move random-related code in the allocator\"\n\nThis reverts commit 9903b0586cfb76ef2401c342501e61e1bd3daa0f.\n\nCauses build failures (on GCC 10.2) with the following error:\n\nIn file included from /home/nikic/llvm-project/compiler-rt/lib/scudo/standalone/combined.h:29,\n from /home/nikic/llvm-project/compiler-rt/lib/scudo/standalone/allocator_config.h:12,\n from /home/nikic/llvm-project/compiler-rt/lib/scudo/standalone/wrappers_cpp.cpp:14:\n/home/nikic/llvm-project/compiler-rt/lib/scudo/standalone/../../gwp_asan/guarded_pool_allocator.h: In member function ‘bool gwp_asan::GuardedPoolAllocator::shouldSample()’:\n/home/nikic/llvm-project/compiler-rt/lib/scudo/standalone/../../gwp_asan/guarded_pool_allocator.h:82:69: error: conversion from ‘uint32_t’ {aka ‘unsigned int’} to ‘unsigned int:31’ may change value [-Werror\u003dconversion]\n 82 | (getRandomUnsigned32() % (AdjustedSampleRatePlusOne - 1)) + 1;\n | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~\n\nGitOrigin-RevId: 04e42f62548d4c0367664188a938b609435718e2\n" }, { "commit": "72384c39dab1bd9bd98bc08a895c283bb128f09a", "tree": "e5d41eaae806ccfb614baae8ea8e01ed080d987e", "parents": [ "bac1f66b620854da534710b67b3baa235fcdfdd5" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Wed Oct 21 13:04:09 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:42 2020 -0700" }, "message": "[GWP-ASan] Move random-related code in the allocator\n\nWe need to have all thread specific data packed into a single `uintptr_t`\nfor the upcoming Fuchsia support. We can move the `RandomState` into the\n`ThreadLocalPackedVariables`, reducing the size of `NextSampleCounter`\nto 31 bits (or we could reduce `RandomState` to 31 bits).\n\nWe move `getRandomUnsigned32` into the platform agnostic part of the\nclass, and `initPRNG` in the platform specific part.\n\n`ScopedBoolean` is replaced by actual assignments since non-const\nreferences to bitfields are prohibited.\n\n`random.{h,cpp}` are removed.\n\nDifferential Revision: https://reviews.llvm.org/D89908\n\nGitOrigin-RevId: 9903b0586cfb76ef2401c342501e61e1bd3daa0f\n" }, { "commit": "bac1f66b620854da534710b67b3baa235fcdfdd5", "tree": "f432a7089082355a6ce27d7def6c0e1216ddf615", "parents": [ "b2b29ff375e9fa41b3c3ca7aad094e730f43ee87" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Tue Oct 20 11:15:52 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:38 2020 -0700" }, "message": "[GWP-ASan] Rework utilities (NFC)\n\nFew changes wrt utilities:\n- split `Check` into a platform agnostic condition test and a platform\n specific termination, for which we introduce the function `die`.\n- add a platform agnostic `utilities.cpp` that gets the allocation\n alignment functions original in the platform specific file, as they\n are reusable by all platforms.\n\nDifferential Revision: https://reviews.llvm.org/D89811\n\nGitOrigin-RevId: be8e4de7240eb0f7dc2f7fd6d07898184c04c72b\n" }, { "commit": "b2b29ff375e9fa41b3c3ca7aad094e730f43ee87", "tree": "195749991e36dfda3459df5cf86ca518c57ade84", "parents": [ "ed9d4b9ed96fbfb4e626f3f4c185dfa76c3cce02" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Mon Oct 19 11:54:43 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Thu Oct 29 11:47:34 2020 -0700" }, "message": "[GWP-ASan] Cleanup (NFC)\n\nCleaning up some of the GWP-ASan code base:\n- lots of headers didn\u0027t have the correct file name\n- adding `#ifdef` guard to `utilities.h`\n- correcting an `#ifdef` guard based on actual file name\n- removing an extra `;`\n- clang-format\u0027ing the code (`-style\u003dllvm`)\n\nDifferential Revision: https://reviews.llvm.org/D89721\n\nGitOrigin-RevId: ae9d0400283229032fb3795b7b50920bf9be38dc\n" }, { "commit": "ed9d4b9ed96fbfb4e626f3f4c185dfa76c3cce02", "tree": "78b2f69162a751f9028dee971ece345ba3ac7f31", "parents": [ "df841da2f78ef5dca876a2fde67e1e39f4e1ca24" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Jul 27 10:51:53 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:10 2020 -0700" }, "message": "[GWP-ASan] Crash handler API returns sizeof(collected trace)\n\nSummary:\nFix up a slight bug with the crash handler API, where we say that we\nreturn the size of the collected trace (instead of the size of the trace\nthat\u0027s returned) when the return buffer is too small, and the result is\ntruncated.\n\nAlso, as a result, patch up a small uninitialized memory bug.\n\nReviewers: morehouse, eugenis\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers\n\nTags: #sanitizers\n\nDifferential Revision: https://reviews.llvm.org/D84652\n\nGitOrigin-RevId: d19af2f2476b5e13a65d5283cce9859e2c1ef763\n" }, { "commit": "df841da2f78ef5dca876a2fde67e1e39f4e1ca24", "tree": "324841a4e185fe7c4c7a8fe52af673005778356e", "parents": [ "32dc40fd1771f7f8cea82d6ef68d98e1919405aa" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Jul 27 05:46:43 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:08 2020 -0700" }, "message": "[GWP-ASan] Fix uninitialized memory use in sigaction.\n\nFix up a small bug where we used a partially-uninitialized sigaction\nstruct in the optional signal handler. Shouldn\u0027t be a user-visible\nchange.\n\nGitOrigin-RevId: 005eee8b3ef7f244daf12f574a191079bfc1918b\n" }, { "commit": "32dc40fd1771f7f8cea82d6ef68d98e1919405aa", "tree": "0ead835d3731dce0a047e601911dd98cec33d766", "parents": [ "b56903608c7926cb28cff6b01a938a5888007cbd" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Jul 21 08:19:02 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:06 2020 -0700" }, "message": "[GWP-ASan] Split the unwinder into segv/non-segv.\n\nNote: Resubmission with frame pointers force-enabled to fix builds with\n-DCOMPILER_RT_BUILD_BUILTINS\u003dFalse\n\nSummary:\nSplits the unwinder into a non-segv (for allocation/deallocation traces) and a\nsegv unwinder. This ensures that implementations can select an accurate, slower\nunwinder in the segv handler (if they choose to use the GWP-ASan provided one).\nThis is important as fast frame-pointer unwinders (like the sanitizer unwinder)\ndon\u0027t like unwinding through signal handlers.\n\nReviewers: morehouse, cryptoad\n\nReviewed By: morehouse, cryptoad\n\nSubscribers: cryptoad, mgorny, eugenis, pcc, #sanitizers\n\nTags: #sanitizers\n\nDifferential Revision: https://reviews.llvm.org/D83994\n\nGitOrigin-RevId: 4f029d1be4e19270416637327f56668744e64b5c\n" }, { "commit": "b56903608c7926cb28cff6b01a938a5888007cbd", "tree": "afe0e24515564b56b96ba8f8eacae61ba2671a37", "parents": [ "82049f7ed2d21950c67db7475527a9761225a3e0" ], "author": { "name": "Hans Wennborg", "email": "hans@chromium.org", "time": "Tue Jul 21 11:06:33 2020 +0200" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:05 2020 -0700" }, "message": "Revert 502f0cc0e38 \"[GWP-ASan] Split the unwinder into segv/non-segv.\"\n\nIt was causing tests to fail in -DCOMPILER_RT_BUILD_BUILTINS\u003dOFF builds:\n\n GwpAsan-Unittest :: ./GwpAsan-x86_64-Test/BacktraceGuardedPoolAllocator.DoubleFree\n GwpAsan-Unittest :: ./GwpAsan-x86_64-Test/BacktraceGuardedPoolAllocator.UseAfterFree\n\nsee comment on the code review.\n\n\u003e Summary:\n\u003e Splits the unwinder into a non-segv (for allocation/deallocation traces) and a\n\u003e segv unwinder. This ensures that implementations can select an accurate, slower\n\u003e unwinder in the segv handler (if they choose to use the GWP-ASan provided one).\n\u003e This is important as fast frame-pointer unwinders (like the sanitizer unwinder)\n\u003e don\u0027t like unwinding through signal handlers.\n\u003e\n\u003e Reviewers: morehouse, cryptoad\n\u003e\n\u003e Reviewed By: morehouse, cryptoad\n\u003e\n\u003e Subscribers: cryptoad, mgorny, eugenis, pcc, #sanitizers\n\u003e\n\u003e Tags: #sanitizers\n\u003e\n\u003e Differential Revision: https://reviews.llvm.org/D83994\n\nThis reverts commit 502f0cc0e3889229e923e187f38dda91324ae139.\n\nGitOrigin-RevId: ab6263c9258ccd0e3c9cb4f675979f22cfba6131\n" }, { "commit": "82049f7ed2d21950c67db7475527a9761225a3e0", "tree": "0ead835d3731dce0a047e601911dd98cec33d766", "parents": [ "fedc3e223ffdfc93c8a1abcac768ade05fb8f09b" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Fri Jul 17 12:59:09 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:03 2020 -0700" }, "message": "[GWP-ASan] Split the unwinder into segv/non-segv.\n\nSummary:\nSplits the unwinder into a non-segv (for allocation/deallocation traces) and a\nsegv unwinder. This ensures that implementations can select an accurate, slower\nunwinder in the segv handler (if they choose to use the GWP-ASan provided one).\nThis is important as fast frame-pointer unwinders (like the sanitizer unwinder)\ndon\u0027t like unwinding through signal handlers.\n\nReviewers: morehouse, cryptoad\n\nReviewed By: morehouse, cryptoad\n\nSubscribers: cryptoad, mgorny, eugenis, pcc, #sanitizers\n\nTags: #sanitizers\n\nDifferential Revision: https://reviews.llvm.org/D83994\n\nGitOrigin-RevId: 502f0cc0e3889229e923e187f38dda91324ae139\n" }, { "commit": "fedc3e223ffdfc93c8a1abcac768ade05fb8f09b", "tree": "afe0e24515564b56b96ba8f8eacae61ba2671a37", "parents": [ "9204c4eb5d0fa2cab106a7fade1da9ea35cb043f" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Mar 26 10:09:57 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:43:00 2020 -0700" }, "message": "[GWP-ASan] Use functions in backtrace test, not line numbers.\n\nSummary:\nThere\u0027s no unwinding functionality on Android that allows for line\nnumbers to be retrieved in-process. As a result, we can\u0027t have\nthis backtrace test run on Android.\n\nCleanup the test to use optnone functions instead, which is more stable\nthan line numbers anyway.\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers, morehouse, cferris\n\nTags: #sanitizers\n\nDifferential Revision: https://reviews.llvm.org/D76807\n\nGitOrigin-RevId: 1216f4c0ea0c232d545ce3ab8f858c696e085d47\n" }, { "commit": "9204c4eb5d0fa2cab106a7fade1da9ea35cb043f", "tree": "dd09b8de6a6bc3fa4f760ea00fbf62a79e557bb5", "parents": [ "9fdbe7d2ed60e4702f94ade0c50b4e950ea5519d" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Tue Mar 24 14:37:07 2020 -0700" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:59 2020 -0700" }, "message": "[GWP-ASan] Only pack frames that are stored.\n\nSummary:\nBacktrace() returns the number of frames that are *available*, rather\nthan the number of frames stored. When we pack, we supply the number of\nframes we have stored. The number of available frames can exceed the\nnumber of stored frames, leading to stack OOB read.\n\nFix up this problem.\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers, morehouse, cferris, pcc\n\nTags: #sanitizers\n\nDifferential Revision: https://reviews.llvm.org/D76722\n\nGitOrigin-RevId: a4e8d89704d2584d5c56cb27745beab25c7b9b36\n" }, { "commit": "9fdbe7d2ed60e4702f94ade0c50b4e950ea5519d", "tree": "ad6379488d50a195525213cb8fc8986d2614fad9", "parents": [ "06ff811d2509a88c1435c8d5cd3f0131d28fd5a2" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Mar 05 10:27:08 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:57 2020 -0700" }, "message": "[GWP-ASan] Fix thread ID.\n\nSummary:\nGWP-ASan currently reports \u003cunknown\u003e thread ID, as the crash handler\nmerge dropped the include. Oops.\n\nReviewers: morehouse\n\nReviewed By: morehouse\n\nSubscribers: eugenis, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D75693\n\nGitOrigin-RevId: 85c0b34fc5a5e19dee4b5055eaafbb1c2589c233\n" }, { "commit": "06ff811d2509a88c1435c8d5cd3f0131d28fd5a2", "tree": "336981201e4a9b75a8c979e78cff06c0ef143760", "parents": [ "21bad50eb21d1b3d7b4766385137008fe91bd004" ], "author": { "name": "Kostya Kortchinsky", "email": "kostyak@google.com", "time": "Thu Feb 13 12:55:56 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:55 2020 -0700" }, "message": "[GWP-ASan] Silence gcc error\n\nSummary: It complains about reaching the end of a non-void returning function.\n\nReviewers: eugenis, hctim, morehouse\n\nSubscribers: #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D74578\n\nGitOrigin-RevId: ef7488ef205ceb018af463b82386f5ee6a365445\n" }, { "commit": "21bad50eb21d1b3d7b4766385137008fe91bd004", "tree": "7577ed192af6d5801c92e8fa10e38c300a7e932a", "parents": [ "ae3f1f24e8d3a179673352d87a5905522fd9ade9" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 12 16:06:42 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:53 2020 -0700" }, "message": "[GWP-ASan] [NFC] Change enum from ANDROID-\u003eBIONIC.\n\nGitOrigin-RevId: 91e194d1ff8d6954644774315ffa2fa9e2f62aec\n" }, { "commit": "ae3f1f24e8d3a179673352d87a5905522fd9ade9", "tree": "6225e84c7274040e43948d98aeea1ba00c46cea5", "parents": [ "595c0bdcaac4e8fa626e3ab655978939ddbbf4f3" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 12 15:24:55 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:51 2020 -0700" }, "message": "[GWP-ASan] Update alignment on Android.\n\nSummary:\nAndroid has different alignment requirements. You can read more about\nthem here\n(https://cs.android.com/android/platform/superproject/+/master:bionic/tests/malloc_test.cpp;l\u003d808),\nbut the general gist is that for malloc(x \u003c\u003d 8), we do malloc(8), and\nfor everything else, we do 16-byte alignment.\n\nReviewers: eugenis, morehouse, cferris\n\nReviewed By: eugenis, morehouse\n\nSubscribers: #sanitizers, llvm-commits, pcc\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D74364\n\nGitOrigin-RevId: 5f2a74c87abc4ed926e93e75d1c3178723a47322\n" }, { "commit": "595c0bdcaac4e8fa626e3ab655978939ddbbf4f3", "tree": "ca5a86a275b28b9211f577c08ba0b114196350bc", "parents": [ "02fce31b137e91121c43f260de8b71f7d998833f" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Feb 06 10:35:42 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:49 2020 -0700" }, "message": "[GWP-ASan] Use weak abort message definition.\n\nSummary:\nNew shard out of the Check() function for GWP-ASan uses\nandroid_set_abort_message. This is happily present on bionic Android,\nbut not Android for glibc host x86. Fix up to use the weak definition\nalways, so we don\u0027t have to worry.\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers, llvm-commits, pcc, cferris\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D74142\n\nGitOrigin-RevId: 8d19af685c9b1ba50df0ed739dbfffd0aa7686ee\n" }, { "commit": "02fce31b137e91121c43f260de8b71f7d998833f", "tree": "7b5df7932f47960ae5814146dccd0b117099f464", "parents": [ "bbcf5e0f1e79ea4375fa599e412ca255295f1322" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Thu Feb 06 10:07:08 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:47 2020 -0700" }, "message": "[GWP-ASan] Fix PRNG to use IE TLS.\n\nSummary:\nGWP-ASan\u0027s PRNG didn\u0027t use Initial-Exec TLS. Fix that to ensure that we don\u0027t\nhave infinite recursion, and also that we don\u0027t allocate a DTV on Android when\nGWP-ASan is touched.\n\nTest coverage ensuring that the sample counter is UINT32_MAX for an\nuninitialised GWP-ASan is provided by gwp_asan/tests/late_init.cpp.\n\nReviewers: pcc, cferris\n\nReviewed By: pcc\n\nSubscribers: #sanitizers, llvm-commits, rprichard, eugenis\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D74135\n\nGitOrigin-RevId: 25de3f98b8a7436404dbc185040645f2549a8a8f\n" }, { "commit": "bbcf5e0f1e79ea4375fa599e412ca255295f1322", "tree": "eaa50d74665f358c3b00d96d05a8d3ed3eb106e5", "parents": [ "5f42f0721a599a14026bad6f7a72d3307cb3d54f" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 05 17:31:27 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:45 2020 -0700" }, "message": "[GWP-ASan] Fix unused variables from crash handler + clang-format\n\nSummary: NFC - See title\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: merge_guards_bot, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D74100\n\nGitOrigin-RevId: 0bfc4890931943388b6c9f5aae3d036668093c92\n" }, { "commit": "5f42f0721a599a14026bad6f7a72d3307cb3d54f", "tree": "ccffb2c8a38e34140e1646910b47c3566d558f6d", "parents": [ "bc9183af0a6bebb9de04bafa1ac2137d19bd6d8f" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 05 16:46:51 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:44 2020 -0700" }, "message": "[GWP-ASan] Change crash handler constant.\n\nConstant is too large to fit into uintptr_t on 32-bit.\n\nGitOrigin-RevId: 0c3b2986ac6b71abc649811c3ec9cb0bf073c7d8\n" }, { "commit": "bc9183af0a6bebb9de04bafa1ac2137d19bd6d8f", "tree": "b18139bf448c54e9ed513a85a069e8705786755f", "parents": [ "4f2299db6add597e5549d31bd606a3e80eba7cdd" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Wed Feb 05 15:39:03 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:42 2020 -0700" }, "message": "[GWP-ASan] Crash Handler API.\n\nSummary:\nForewarning: This patch looks big in #LOC changed. I promise it\u0027s not that bad, it just moves a lot of content from one file to another. I\u0027ve gone ahead and left inline comments on Phabricator for sections where this has happened.\n\nThis patch:\n 1. Introduces the crash handler API (crash_handler_api.h).\n 2. Moves information required for out-of-process crash handling into an AllocatorState. This is a trivially-copied POD struct that designed to be recovered from a deceased process, and used by the crash handler to create a GWP-ASan report (along with the other trivially-copied Metadata struct).\n 3. Implements the crash handler API using the AllocatorState and Metadata.\n 4. Adds tests for the crash handler.\n 5. Reimplements the (now optionally linked by the supporting allocator) in-process crash handler (i.e. the segv handler) using the new crash handler API.\n 6. Minor updates Scudo \u0026 Scudo Standalone to fix compatibility.\n 7. Changed capitalisation of errors (e.g. /s/Use after free/Use After Free).\n\nReviewers: cryptoad, eugenis, jfb\n\nReviewed By: eugenis\n\nSubscribers: merge_guards_bot, pcc, jfb, dexonsmith, mgorny, cryptoad, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73557\n\nGitOrigin-RevId: a62586846fa90054bd9224912b07095d2fca662c\n" }, { "commit": "4f2299db6add597e5549d31bd606a3e80eba7cdd", "tree": "1c7a3a03896ff992ce02a3741d1db0f8ef6e9b8a", "parents": [ "1f1cdc9f78fb6684fbb5942f3798af05be1fa797" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Feb 03 12:25:51 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:40 2020 -0700" }, "message": "[GWP-ASan] Allow late initialisation if single-threaded.\n\nSummary:\nThis patch allows for late initialisation of the GWP-ASan allocator. Previously, if late initialisation occurred, the sample counter was never updated, meaning we would end up having to wait for 2^32 allocations before getting a sampled allocation.\n\nNow, we initialise the sampling mechanism in init() as well. We require init() to be called single-threaded, so this isn\u0027t a problem.\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: merge_guards_bot, mgorny, #sanitizers, llvm-commits, cferris\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73896\n\nGitOrigin-RevId: 0d6fccb460e515a78c33dcd97cda459332ddd63a\n" }, { "commit": "1f1cdc9f78fb6684fbb5942f3798af05be1fa797", "tree": "30b6d12735aa8d674365efb2b4aaf1adf38e5807", "parents": [ "ab1bee318f745e3421c6321764be4c1b529ba5cd" ], "author": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Thu Jan 23 16:12:15 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:38 2020 -0700" }, "message": "[gwp-asan] Implement malloc_iterate.\n\nSummary:\nThis is an Android-specific interface for iterating over all live\nallocations in a memory range.\n\nReviewers: hctim, cferris\n\nSubscribers: mgorny, mgrang, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73305\n\nGitOrigin-RevId: 46044a6900e81f7f3f0954266016eb49d1c099ab\n" }, { "commit": "ab1bee318f745e3421c6321764be4c1b529ba5cd", "tree": "5f7ab77ed9776d7bb602d3564a3d16ce0e62adfb", "parents": [ "885b397f97046b1b9d0180cff7510e0d3382636e" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Fri Jan 24 15:01:47 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:36 2020 -0700" }, "message": "[GWP-ASan] Add names to anonymous mappings.\n\nSummary:\nAdds names to anonymous GWP-ASan mappings. This helps Android with debugging\nvia. /proc/maps, as GWP-ASan-allocated mappings are now easily identifyable.\n\nReviewers: eugenis, cferris\n\nReviewed By: eugenis\n\nSubscribers: merge_guards_bot, #sanitizers, llvm-commits, cryptoad, pcc\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73373\n\nGitOrigin-RevId: e1440f594c1409867c4c037b6cda9cdeccdda8fb\n" }, { "commit": "885b397f97046b1b9d0180cff7510e0d3382636e", "tree": "b80f3beb8d871c95b4abbcfdcfe20ac2e197098b", "parents": [ "b05579b03d74f6faf0c4c98f016ac2b5136eddfe" ], "author": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Fri Jan 10 16:01:01 2020 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:34 2020 -0700" }, "message": "[GWP-ASan] enable/disable and fork support.\n\nSummary:\n* Implement enable() and disable() in GWP-ASan.\n* Setup atfork handler.\n* Improve test harness sanity and re-enable GWP-ASan in Scudo.\n\nScudo_standalone disables embedded GWP-ASan as necessary around fork().\nStandalone GWP-ASan sets the atfork handler in init() if asked to. This\nrequires a working malloc(), therefore GWP-ASan initialization in Scudo\nis delayed to the post-init callback.\n\nTest harness changes are about setting up a single global instance of\nthe GWP-ASan allocator so that pthread_atfork() does not create\ndangling pointers.\n\nTest case shamelessly stolen from D72470.\n\nReviewers: cryptoad, hctim, jfb\n\nSubscribers: mgorny, jfb, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D73294\n\nGitOrigin-RevId: 596d06145a2b94fa9549f0e8cadc641586dfab1d\n" }, { "commit": "b05579b03d74f6faf0c4c98f016ac2b5136eddfe", "tree": "3d5e42ea25f9413bcbb9716600842454207d195d", "parents": [ "6b6df79f695232d7e975ff43197eb3c69666f233" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Dec 09 08:19:39 2019 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:32 2020 -0700" }, "message": "[GWP-ASan] 32-bit test pointers, allow multi-init for test.\n\nSummary:\nGWP-ASan test currently fail on 32-bit platforms, as some of the pointers are\nlarger than `uintptr_t` on 32-bit platforms. Fix up all those instances.\n\nAlso add an uncompress varint test where the result is an underflow.\n\nFurthermore, allow multi-init for testing. Each gtest when running\n`check-gwp_asan` apparently runs in its own instance, but when integrating\nthese tests into Android, this behaviour isn\u0027t the same. We remove the\nglobal multi-init check here, to allow for testing to work elsewhere, and we\u0027re\nnot really worried about multi-init anyway as it\u0027s part of our contract with\nthe allocator.\n\nReviewers: eugenis, vlad.tsyrklevich\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers, llvm-commits, pcc\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D71121\n\nGitOrigin-RevId: 26fd95680bcf96870fbe9187aaa7e460709ce487\n" }, { "commit": "6b6df79f695232d7e975ff43197eb3c69666f233", "tree": "9840e20c36c439a683d1a903ed4347483bcb5f82", "parents": [ "11c6adadc7f79ae759db206367afd344fcee0a68" ], "author": { "name": "Mitch Phillips", "email": "31459023+hctim@users.noreply.github.com", "time": "Mon Nov 25 12:25:43 2019 -0800" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:30 2020 -0700" }, "message": "[GWP-ASan] Add GWP_ASAN_ prefix to macros.\n\nSummary:\nWhen platforms use their own `LIKELY()` definitions, it can be quite\ntroublesome to ensure they don\u0027t conflict with the GWP-ASan internal\ndefinitions. Just force the GWP_ASAN_ prefix to help this issue.\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: #sanitizers, llvm-commits, cferris, pcc\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D70683\n\nGitOrigin-RevId: 19edfb37282ade36c502d8fcea9c915c443474c9\n" }, { "commit": "11c6adadc7f79ae759db206367afd344fcee0a68", "tree": "e1905e763f821ab318bc5d520ce072c2fbb37aa0", "parents": [ "145ebb5828afeba16376e352621ea2716dc99260" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Tue Aug 27 18:28:07 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:28 2020 -0700" }, "message": "Add GWP-ASan fuzz target to compiler-rt/tools.\n\nSummary:\n@eugenis to approve addition of //compiler-rt/tools.\n@pree-jackie please confirm that this WFY.\n\nD66494 introduced the GWP-ASan stack_trace_compressor_fuzzer. Building fuzz\ntargets in compiler-rt is a new affair, and has some challenges:\n- If the host compiler doesn\u0027t have compiler-rt, the -fsanitize\u003dfuzzer may not\n be able to link against `libclang_rt.fuzzer*`.\n- Things in compiler-rt generally aren\u0027t built when you want to build with\n sanitizers using `-DLLVM_USE_SANITIZER`. This tricky to work around, so\n we create the new tools directory so that we can build fuzz targets with\n sanitizers. This has the added bonus of fixing the problem above as well, as\n we can now just guard the fuzz target build to only be done with\n `-DLLVM_USE_SANITIZE_COVERAGE\u003dOn`.\n\nReviewers: eugenis, pree-jackie\n\nReviewed By: eugenis, pree-jackie\n\nSubscribers: dberris, mgorny, #sanitizers, llvm-commits, eugenis, pree-jackie, lebedev.ri, vitalybuka, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D66776\n\nllvm-svn: 370094\nGitOrigin-RevId: ae56e593b90c61eda160d03398e953cc5c6037b3\n" }, { "commit": "145ebb5828afeba16376e352621ea2716dc99260", "tree": "cb10da3b8c6768caee7d9cff616c4ef46a6886d3", "parents": [ "cadb0ef2ebd46f4217028ecb4d69a094e2fcf025" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Fri Aug 23 23:23:48 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:27 2020 -0700" }, "message": "[GWP-ASan] Split options_parser and backtrace_sanitizer_common.\n\nSummary:\noptional/options_parser and optional/backtrace_sanitizer_common are logically\nseparate components. They both use sanitizer-common to power their\nfunctionality, but there was an unstated implicit dependency that in order for\nbacktrace_sanitizer_common to function correctly, one had to also use\noptions_parser.\n\nThis was because options_parser called __sanitizer::InitialiseCommonFlags. This\nis a requirement for backtrace_sanitizer_common to work, as the sanitizer\nunwinder uses the sanitizer_common flags and will SEGV on a null page if\nthey\u0027re not initialised correctly.\n\nThis patch removes this hidden dependency. You can now use\nbacktrace_sanitizer_common without the requirements of options_parser.\n\nThis patch also makes the GWP-ASan unit tests only have a soft dependency on\nsanitizer-common. The unit tests previously explicitly used\n__sanitizer::Printf, which is now provided under\ntests/optional/printf_sanitizer_common. This allows Android to build the unit\ntests using their own signal-safe printf().\n\nReviewers: eugenis\n\nReviewed By: eugenis\n\nSubscribers: srhines, mgorny, #sanitizers, llvm-commits, vlad.tsyrklevich, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D66684\n\nllvm-svn: 369825\nGitOrigin-RevId: 27d69b2f4f6bf9b09755ae2428a4703eb4a4663b\n" }, { "commit": "cadb0ef2ebd46f4217028ecb4d69a094e2fcf025", "tree": "ad53e5ba3c734cf55acc2d6df6cace5bec3f71f6", "parents": [ "c8e6197c6e14ce9d086802fd13b04fa93058eb63" ], "author": { "name": "Petr Hosek", "email": "phosek@chromium.org", "time": "Thu Aug 22 07:03:38 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:25 2020 -0700" }, "message": "Revert \"[GWP-ASan] Remove c++ standard lib dependency.\"\n\nThis reverts commit r369606: this doesn\u0027t addressed the underlying\nproblem and it\u0027s not the correct solution.\n\nllvm-svn: 369623\nGitOrigin-RevId: 028b5499fffa536236cb5714e6d14b786883a4b4\n" }, { "commit": "c8e6197c6e14ce9d086802fd13b04fa93058eb63", "tree": "8e39955e32b14390c2968f43a7828549bc339fb4", "parents": [ "bc944c315bccc74e8b06d979aebd0327cf7153f3" ], "author": { "name": "Petr Hosek", "email": "phosek@chromium.org", "time": "Thu Aug 22 00:22:56 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:23 2020 -0700" }, "message": "[GWP-ASan] Remove c++ standard lib dependency.\n\nRemove c++ standard library dependency for now for @phosek. They have a\ncomplicated build system that breaks with the fuzzer target here.\n\nAlso added a todo to remedy later.\n\nDifferential Revision: https://reviews.llvm.org/D66568\n\nllvm-svn: 369606\nGitOrigin-RevId: 3dd22585199195d4656154205ab631b8dd406143\n" }, { "commit": "bc944c315bccc74e8b06d979aebd0327cf7153f3", "tree": "ad53e5ba3c734cf55acc2d6df6cace5bec3f71f6", "parents": [ "eea622aa95b85cf136d40b0a0a0068b89ecb1189" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Wed Aug 21 17:53:51 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:21 2020 -0700" }, "message": "[GWP-ASan] Add public-facing documentation [6].\n\nSummary:\nNote: Do not submit this documentation until Scudo support is reviewed and submitted (should be #[5]).\n\nSee D60593 for further information.\n\nThis patch introduces the public-facing documentation for GWP-ASan, as well as updating the definition of one of the options, which wasn\u0027t properly merged. The document describes the design and features of GWP-ASan, as well as how to use GWP-ASan from both a user\u0027s standpoint, and development documentation for supporting allocators.\n\nReviewers: jfb, morehouse, vlad.tsyrklevich\n\nReviewed By: morehouse, vlad.tsyrklevich\n\nSubscribers: kcc, dexonsmith, kubamracek, cryptoad, jfb, #sanitizers, llvm-commits, vlad.tsyrklevich, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D62875\n\nllvm-svn: 369552\nGitOrigin-RevId: c776f3f3c26f69012456117b5487df2b81ad51e7\n" }, { "commit": "eea622aa95b85cf136d40b0a0a0068b89ecb1189", "tree": "982c789f59db6b84f85bc29f54380314cdbb305b", "parents": [ "5931d04aff5fe22f0ed9935fda45c3f47dec10a3" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Tue Aug 20 20:16:11 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:19 2020 -0700" }, "message": "[GWP-ASan] Fix typos.\n\nSummary:\nFix two spelling typos and de-indent a guarded #define so that it\u0027s\nconsistent with clang-format.\n\nReviewers: vitalybuka\n\nReviewed By: vitalybuka\n\nSubscribers: #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D66311\n\nllvm-svn: 369433\nGitOrigin-RevId: 8f5e1755ca385566c0352a9bd292218cebfd3d0b\n" }, { "commit": "5931d04aff5fe22f0ed9935fda45c3f47dec10a3", "tree": "85582b6271ee122138288a677fba1c59bb099687", "parents": [ "1af060d54935e4e80d5da2e82da23f90db3ee710" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Thu Aug 15 21:09:09 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:17 2020 -0700" }, "message": "[GWP-ASan] Implement stack frame compression.\n\nSummary:\nThis patch introduces stack frame compression to GWP-ASan. Each stack frame is\nvariable-length integer encoded as the difference between frame[i] and\nframe[i - 1]. Furthermore, we use zig-zag encoding on the difference to ensure\nthat negative differences are also encoded into a relatively small number of\nbytes.\n\nExamples of what the compression looks like can be seen in\n`gwp_asan/tests/compression.cpp`.\n\nThis compression can reduce the memory consumption cost of stack traces by\n~50%.\n\nReviewers: vlad.tsyrklevich\n\nReviewed By: vlad.tsyrklevich\n\nSubscribers: mgorny, #sanitizers, llvm-commits, eugenis, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D66189\n\nllvm-svn: 369048\nGitOrigin-RevId: be8a2f75657b27ea113517c3279f0489a7f4018c\n" }, { "commit": "1af060d54935e4e80d5da2e82da23f90db3ee710", "tree": "89d36a77e36f5c40c1bd2ba577a62980ecdf9f61", "parents": [ "548f9b97ec2411f1cae5f904ff0e9fa83a800843" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Mon Aug 12 21:36:44 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:16 2020 -0700" }, "message": "[GWP-ASan] Update backtrace function signature.\n\nSummary:\nUpdates the function signature and comments for backtracing (and printing\nbacktraces). This update brings GWP-ASan in line with future requirements for\nstack frame compression, wherein the length of the trace is provided\nexplicitly, rather than relying on nullptr-termination.\n\nReviewers: vlad.tsyrklevich\n\nReviewed By: vlad.tsyrklevich\n\nSubscribers: #sanitizers, llvm-commits, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D66099\n\nllvm-svn: 368619\nGitOrigin-RevId: 352d1b59c0c892f4d642a7355e0dee664bb19479\n" }, { "commit": "548f9b97ec2411f1cae5f904ff0e9fa83a800843", "tree": "1964f0bda67478d8836c0219bd9f05ff43fdbdef", "parents": [ "13bcee480f6adf5627ce12b76c5d0abaee97243c" ], "author": { "name": "Nico Weber", "email": "nicolasweber@gmx.de", "time": "Thu Aug 01 17:53:25 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:14 2020 -0700" }, "message": "Remove a few straggler \".cc\"s in compiler-rt/lib\n\nllvm-svn: 367589\nGitOrigin-RevId: 4a1a113a99082914734b78e434a2ffea78a57dc4\n" }, { "commit": "13bcee480f6adf5627ce12b76c5d0abaee97243c", "tree": "f3322bf873768574ffb757d006bd473a9d74a873", "parents": [ "c41bec042f21843da0e2b34810005c5ba03cbc9a" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Tue Jul 16 20:06:17 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:12 2020 -0700" }, "message": "[GWP-ASan] Add thread ID to PRNG seed.\n\nSummary:\nAdds thread ID to PRNG seed for increased entropy. In particular, this allows\nmultiple runs in quick succession that will have different PRNG seeds, allowing\nfor better demos/testing.\n\nReviewers: kcc\n\nReviewed By: kcc\n\nSubscribers: kubamracek, #sanitizers, llvm-commits\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D64453\n\nllvm-svn: 366253\nGitOrigin-RevId: b157dcacb5b96fd64900906911832a8ac3bb189e\n" }, { "commit": "c41bec042f21843da0e2b34810005c5ba03cbc9a", "tree": "d588a10a4a09af25269515861e00d6bcf6b5fd08", "parents": [ "e773f8e263dc11236baf54a65323e45e02a2bf22" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Thu Jul 11 18:07:03 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:10 2020 -0700" }, "message": "Explicitly define __STDC_FORMAT_MACROS for PRIu64\n\nSummary:\nBuilds are failing on RHEL machines because of PRIu64.\n\nlvm/projects/compiler-rt/lib/gwp_asan/guarded_pool_allocator.cpp:420:50: error: expected \u0027)\u0027\n\n`snprintf(ThreadBuffer, kThreadBufferLen, \"%\" PRIu64, ThreadID);`\ninttypes.h in RHEL uses PRIu64 macros only when __STDC_FORMAT_MACROS is defined.\n\nAuthor: DTharun\n\nReviewers: hctim\n\nReviewed By: hctim\n\nDifferential Revision: https://reviews.llvm.org/D64388\n\nllvm-svn: 365801\nGitOrigin-RevId: af3dc759e786324c8a38fa7849da0791b99e67b0\n" }, { "commit": "e773f8e263dc11236baf54a65323e45e02a2bf22", "tree": "767240eaecfe77c004945c0df553fcab923f875b", "parents": [ "1702b1443e465162041c9679bfa5aba9ffcfc252" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Tue Jul 02 16:04:52 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:08 2020 -0700" }, "message": "[GWP-ASan] Add generic unwinders and structure backtrace output.\n\nSummary:\nAdds two flavours of generic unwinder and all the supporting cruft. If the\nsupporting allocator is okay with bringing in sanitizer_common, they can use\nthe fast frame-pointer based unwinder from sanitizer_common. Otherwise, we also\nprovide the backtrace() libc-based unwinder as well. Of course, the allocator\ncan always specify its own unwinder and unwinder-symbolizer.\n\nThe slightly changed output format is exemplified in the first comment on this\npatch. It now better incorporates backtrace information, and displays\nallocation details on the second line.\n\nReviewers: eugenis, vlad.tsyrklevich\n\nReviewed By: eugenis, vlad.tsyrklevich\n\nSubscribers: srhines, kubamracek, mgorny, cryptoad, #sanitizers, llvm-commits, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D63841\n\nllvm-svn: 364941\nGitOrigin-RevId: 7339ca278c3fdb1d932b776005b2687ec11eadae\n" }, { "commit": "1702b1443e465162041c9679bfa5aba9ffcfc252", "tree": "1124fb797a64c8f41d2be2e4781e7a29c1281002", "parents": [ "169282c48030ec3126a79daa4e3bf1efb1665421" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Tue Jun 25 22:29:05 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:06 2020 -0700" }, "message": "[GWP-ASan] Guard against recursive allocs. Pack TLS for perf.\n\nSummary:\nAdd a recursivity guard for GPA::allocate(). This means that any\nrecursive allocations will fall back to the supporting allocator. In future\npatches, we will introduce stack trace collection support. The unwinder will be\nprovided by the supporting allocator, and we can\u0027t guarantee they don\u0027t call\nmalloc() (e.g. backtrace() on posix may call dlopen(), which may call malloc().\n\nFurthermore, this patch packs the new TLS recursivity guard into a thread local\nstruct, so that TLS variables should be hopefully not fall across cache lines.\n\nReviewers: vlad.tsyrklevich, morehouse, eugenis\n\nReviewed By: eugenis\n\nSubscribers: kubamracek, #sanitizers, llvm-commits, eugenis\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D63736\n\nllvm-svn: 364356\nGitOrigin-RevId: 05d1a2bd1bf3c0172334bbdf21cdf97135ab2797\n" }, { "commit": "169282c48030ec3126a79daa4e3bf1efb1665421", "tree": "40eb64f36558dc70b834862d567b43d92256a925", "parents": [ "9c54dd1befaf2d0850c5f28f087f7502c48639bd" ], "author": { "name": "Douglas Yung", "email": "douglas.yung@sony.com", "time": "Thu Jun 06 08:04:33 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:04 2020 -0700" }, "message": "Fixup files added in r362636 to build with gcc 5.4. NFCI\n\nllvm-svn: 362682\nGitOrigin-RevId: dbceb9b2203b58f829e911c8d1610b95d9abf192\n" }, { "commit": "9c54dd1befaf2d0850c5f28f087f7502c48639bd", "tree": "264710a9301ba24bd12638e38a88d804e429dddd", "parents": [ "3115e17eacc3779fc0b6462906e4b1eebd4c3506" ], "author": { "name": "Mitch Phillips", "email": "mitchphillips@outlook.com", "time": "Wed Jun 05 19:42:48 2019 +0000" }, "committer": { "name": "Copybara-Service", "email": "copybara-worker@google.com", "time": "Wed Sep 30 11:42:02 2020 -0700" }, "message": "[GWP-ASan] Core Guarded Pool Allocator [4].\n\nSummary:\nSee D60593 for further information.\n\nThis patch introduces the core of GWP-ASan, being the guarded pool allocator. This class contains the logic for creating and maintaining allocations in the guarded pool. Its public interface is to be utilised by supporting allocators in order to provide sampled guarded allocation behaviour.\n\nThis patch also contains basic functionality tests of the allocator as unittests. The error-catching behaviour will be tested in upcoming patches that use Scudo as an implementing allocator.\n\nReviewers: vlad.tsyrklevich, eugenis, jfb\n\nReviewed By: vlad.tsyrklevich\n\nSubscribers: dexonsmith, kubamracek, mgorny, cryptoad, jfb, #sanitizers, llvm-commits, morehouse\n\nTags: #sanitizers, #llvm\n\nDifferential Revision: https://reviews.llvm.org/D62872\n\nllvm-svn: 362636\nGitOrigin-RevId: a95edb9dc1ddaf70761e8c90be175f144a28f757\n" } ], "next": "3115e17eacc3779fc0b6462906e4b1eebd4c3506" }