)]}' { "log": [ { "commit": "4956c372cf320328952665629b2626e7cbc84c54", "tree": "7a02304c6c2eadc93b1b7a357fb31a143b661a89", "parents": [ "e592dacfa254f59bb097c98d118d0c860b181da1" ], "author": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Thu Dec 19 16:35:51 2019 -0800" }, "committer": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Thu Jan 02 14:09:50 2020 -0800" }, "message": "Move bionic_macros.h from private to platform.\n\nTest: treehugger\nChange-Id: Ie473914f4c8924c7240b3ac22093a9daf42fc948\n" }, { "commit": "ff88fb0d3adbc67a4f94f5ef7e2b5bcc7a96c8f3", "tree": "1a78e188d9e51383579f697cb5532846b841cd69", "parents": [ "1994f28be2c0faf2b70b1ca8fff7d8d2fa68d922" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Nov 04 18:40:00 2019 -0800" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Wed Nov 06 10:42:42 2019 -0800" }, "message": "Fix allocations escaping malloc debug.\n\nWhen using a FILE object for some malloc debug functions, calling\nfprintf will trigger an allocation to be put in the object. The problem\nis that these allocations were not allocated by the malloc debug\nwrapper and they get freed during the fclose as if they are malloc\ndebug allocation. In most cases, the code will detect the bad pointer\nand leak the memory, but it might also cause a crash.\n\nThe fix is to avoid using fprintf so that no allocations are made\nin the object that survive and need to be freed in the fclose call.\n\nChange the MallocXmlElem.h to use a file decsriptor not a FILE object.\n\nAdd new unit and system tests to detect this case.\n\nBug: 143742907\n\nTest: Ran unit and system tests.\nTest: Ran bionic unit tests.\nChange-Id: I524392de822a29483aa5be8f14c680e70033eba2\n" }, { "commit": "8189e77bbb47dbaa1a09d565c62c2fed03e358c3", "tree": "2c8fc5d8fd1ec1ab881d35519670104ad86283e3", "parents": [ "b9a7c651f178323e27752406f89d741cfb143e61" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Tue Apr 09 16:37:23 2019 -0700" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Tue Apr 16 11:22:06 2019 -0700" }, "message": "Remove gMallocLeakZygoteChild.\n\nRemove this global variable and change the setting of it to non-zero\nto a call to android_mallopt.\n\nIn addition, change the initialize function to use pass a bool* instead of\nint*.\n\nBug: 130028357\n\nTest: Ran malloc_debug/malloc_hooks/perfetto tests.\nChange-Id: I20d382bdeaaf38aac6b9dcabea5b3dfab3c945f6\nMerged-In: I20d382bdeaaf38aac6b9dcabea5b3dfab3c945f6\n(cherry picked from commit 5225b342f0810c027df3d09fbbcef4d324b19b93)\n" }, { "commit": "6c619a0da3f96a26d91c1db48fd3e3be156aabe5", "tree": "aa0802d01d19ed0891e6b77a732cc72ed5617dcc", "parents": [ "4e167f35d602f9dcede3b74ff3e09e88f3edb398" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Fri Mar 01 17:59:51 2019 -0800" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Thu Mar 07 08:39:55 2019 -0800" }, "message": "Refactor the malloc_info code.\n\nmalloc_info needs to be per native allocator, but the code treated it\nlike a global function that doesn\u0027t depend on the native memory allocator.\n\nUpdate malloc debug to dump the actual pointers that it has been tracking.\n\nTest: bionic-unit-tests pass.\nTest: malloc debug tests pass.\nTest: malloc hook tests pass.\nChange-Id: I3b0d4d748489dd84c16d16933479dc8b8d79013e\nMerged-In: I3b0d4d748489dd84c16d16933479dc8b8d79013e\n(cherry picked from commit a3656a98b10d2a4a6194a5d9705ad9c2cc5877b0)\n" }, { "commit": "770032ddfa31f6ed18e6ec85013b7939cd291eb0", "tree": "28dc26ae8a6b3623e79d1bb2b19fd5362477843c", "parents": [ "7bd54ade8d12762ef6bdb0096f3838261f14eee5" ], "author": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Wed Jan 02 10:59:48 2019 -0800" }, "committer": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Wed Jan 02 11:04:05 2019 -0800" }, "message": "Fix/suppress bionic google-explicit-constructor warnings\n\n* Add explicit to conversion constructors/operators\n\nBug: 28341362\nTest: make with WITH_TIDY\u003d1 DEFAULT_GLOBAL_TIDY_CHECKS\u003d-*,google-explicit-constructor\nChange-Id: Id1ad0327c1b8c6f094bcbb3ae599bc1f716b3f2f\n" }, { "commit": "5e62b34c0d6fa545b487b9b64fb4a04a0589bc13", "tree": "e08ee44067b3b636ace11b21ec8d63a08d4d2a19", "parents": [ "c89a3971e94c30611c885e4c440eae2fd9937948" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Oct 25 11:00:00 2018 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Oct 25 11:00:00 2018 -0700" }, "message": "Clean up bionic_macros.h a bit.\n\nUse \u003candroid-base/macros.h\u003e instead where possible, and move the bionic\nmacros out of the way of the libbase ones. Yes, there are folks who manage\nto end up with both included at once (thanks OpenGL!), and cleaning that\nup doesn\u0027t seem nearly as practical as just making this change.\n\nBug: N/A\nTest: builds\nChange-Id: I23fc544f39d5addf81dc61471771a5438778895b\n" }, { "commit": "93bdd6ae3ad8322b7be2be0201c8db7227631d14", "tree": "42ee959b96c2c2db42f0569554680dbe22ce39ef", "parents": [ "4c5c45346fcc6f066a89bfc455f287fe2f4e3e41" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Thu Apr 05 11:12:38 2018 -0700" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Thu May 24 08:44:53 2018 -0700" }, "message": "Add support for using the new unwinder.\n\nThis adds a new option backtrace_full, when it is set, then it will use\nlibunwindstack.\n\nModify the dump to file data to dump the extra information from libunwindstack.\nAlong with the new dump file format, change the version to v1.1.\nUpdated document for new format of file data.\n\nAdd unit tests for the new functionality.\n\nBug: 74361929\n\nTest: Ran unit tests.\nChange-Id: I40fff795f5346bba7b9d7fde2e04f269ff4eb7f1\n" }, { "commit": "4da2503d70dc4bc1444454876e3794b69227d90d", "tree": "d904c750ebac96aee76df44baad1320c0db211a3", "parents": [ "a3f6f6c1b9135c90e410f5382b153db9a43a4db0" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Wed Mar 07 13:38:48 2018 -0800" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Apr 02 18:59:23 2018 -0700" }, "message": "Refactor malloc debug.\n\nChanges\n- Refactor the code so that only guards require creating a special header\n for every pointer allocated.\n- Store only a single copy of every backtrace. This saves memory so that\n turning on the backtrace option doesn\u0027t result in 10X memory usage.\n- Added new option track_allocs that only verifies pointers are valid for\n free/malloc_usable_size/realloc.\n- Remove suffix from test names.\n- Add the TRACK_ALLOCS options to all guard options.\n- Add new option verify_pointers that is a lightweight way to verify\n pointers that are passed to allocation routines.\n- Do auto-formatting of the code.\n- Updated documentation for all of these changes.\n\nBug: 74361929\n\nTest: Ran unit tests.\nTest: Ran libmemunreachable unit tests.\nTest: Ran an app with backtrace enabled.\n\nChange-Id: I3246c48ae4f9811f64622d90d0a9b4d9d818702c\n" } ] }