Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 's' should include 'f'; from Jacqueline Jolicoeur | otto | 2017-11-02 | 1 | -2/+2 |
| | |||||
* | Restore a return that was inadvertently removed from freezero() in r1.234, | jsing | 2017-10-19 | 1 | -1/+2 |
| | | | | | | | which results in an internal double free when internal functions are not in use. ok otto@ | ||||
* | do not return f() where f is a void function; loop var type fix | otto | 2017-10-05 | 1 | -4/+5 |
| | |||||
* | Use dprintf instead of snprintf/write | otto | 2017-10-05 | 1 | -82/+36 |
| | |||||
* | Make delayed free non-optional and make F do an extensive double free check. | otto | 2017-09-23 | 1 | -21/+26 |
| | | | | ok tb@ tedu@ | ||||
* | mapalign returns MAP_FAILED for failuer; from George Koehler | otto | 2017-09-12 | 1 | -2/+2 |
| | |||||
* | check double free before canary for chunks; ok millert@ | otto | 2017-09-11 | 1 | -5/+5 |
| | |||||
* | two MALLOC_STATS only tweaks; one from David CARLIER, the other found by clang | otto | 2017-08-20 | 1 | -8/+4 |
| | |||||
* | one more instance of the previous commit; also initialize ->offset to a | otto | 2017-07-10 | 1 | -2/+3 |
| | | | | definite value in the size == 0 case | ||||
* | Only access offset if canaries are enabled *and* size > 0, otherwise offset | otto | 2017-07-07 | 1 | -2/+2 |
| | | | | is not initialized. Problem spotted by Carlin Bingham; ok phessler@ tedu@ | ||||
* | port the RBT code to userland by making it part of libc. | dlg | 2017-06-19 | 1 | -10/+11 |
| | | | | | | | | | | | | | | src/lib/libc/gen/tree.c is a copy of src/sys/kern/subr_tree.c, but with annotations for symbol visibility. changes to one should be reflected in the other. the malloc debug code that uses RB code is ported to RBT. because libc provides the RBT code, procmap doesn't have to reach into the kernel and build subr_tree.c itself now. mild enthusiasm from many ok guenther@ | ||||
* | - fix bug wrt posix_memalign(3) of blocks between half a page and a page | otto | 2017-05-13 | 1 | -4/+9 |
| | | | | | - document posix_memalign() does not play nice with reacallocarray(3) and freezero(3) | ||||
* | For small allocations (chunk) freezero only validates the given | otto | 2017-04-22 | 1 | -5/+10 |
| | | | | | | | | size if canaries are enabled. In that case we have the exact requested size of the allocation. But we can at least check the given size against the chunk size if C is not enabled. Plus add some braces so my brain doesn't have to scan for dangling else problems when I see this code. | ||||
* | don't forget to fill in canary bytes for posix_memalign(3); reported by | otto | 2017-04-18 | 1 | -1/+4 |
| | | | | and ok jeremy@ | ||||
* | whitespace fixes | otto | 2017-04-17 | 1 | -14/+14 |
| | |||||
* | allow clearing less than allocated and document freezero(3) better | otto | 2017-04-13 | 1 | -5/+5 |
| | |||||
* | Introducing freezero(3) a version of free that guarantees the process | otto | 2017-04-10 | 1 | -12/+72 |
| | | | | | | no longer has access to the content of a memmory object. It does this by either clearing (if the object memory remains cached) or by calling munmap(2). ok millert@, deraadt@, guenther@ | ||||
* | first print size in meta-data then supplied arg size when an inconsistency is | otto | 2017-04-06 | 1 | -3/+3 |
| | | | | detected wrt recallocarray() | ||||
* | small cleanup & optimization; ok deraadt@ millert@ | otto | 2017-03-28 | 1 | -2/+5 |
| | |||||
* | add a helper function to print all pools #ifdef MALLOC_STATS | otto | 2017-03-24 | 1 | -1/+16 |
| | | | | from David CARLIER | ||||
* | move recallocarray to malloc.c and | otto | 2017-03-24 | 1 | -17/+206 |
| | | | | | | | - use internal meta-data to do more consistency checking (especially with option C) - use cheap free if possible ok deraadt@ | ||||
* | Add a NULL test to wrterror() to avoid a NULL deref when called from a | jsg | 2017-02-15 | 1 | -2/+2 |
| | | | | | | free() error path. ok otto@ | ||||
* | fix a comment and rm some dead code as a result of the previous diff | otto | 2017-02-02 | 1 | -8/+5 |
| | |||||
* | Let realloc handle and produce moved pointers for allocations betweenlibressl-v2.5.1 | otto | 2017-02-01 | 1 | -20/+37 |
| | | | | half a page and a page. ok jmatthew@ tb@ | ||||
* | 1. When shrinking a chunk allocation, compare the size of the current | otto | 2017-01-21 | 1 | -46/+87 |
| | | | | | | | | | | allocation to the size of the new allocation (instead of the requested size). 2. Previously realloc takes the easy way and always reallocates if C is active. This commit fixes by carefully updating the recorded requested size in all cases, and writing the canary bytes in the proper location after reallocating. 3. Introduce defines to test if MALLOC_MOVE should be done and to compute the new value. | ||||
* | MALLOC_STATS tweaks, by default not compiled in | otto | 2016-11-04 | 1 | -13/+29 |
| | |||||
* | small tweak to also check canaries if F is in effect | otto | 2016-11-03 | 1 | -3/+5 |
| | |||||
* | remove some old option letters and also make P non-settable. It has | otto | 2016-10-31 | 1 | -24/+6 |
| | | | | | been the default for ages, and I see no valid reason to be able to disable it. ok natano@ | ||||
* | Pages in the malloc cache are either reused quickly or unmapped | otto | 2016-10-28 | 1 | -14/+1 |
| | | | | | | quickly. In both cases it does not make sense to set hints on them. So remove that option, which is just a remainder of old times when malloc used to hold on to pages. ok stefan@ | ||||
* | - fix MALLOC_STATS compile | otto | 2016-10-22 | 1 | -3/+6 |
| | | | | - redundant cast is redundant | ||||
* | fix some void * arithmetic by casting | otto | 2016-10-21 | 1 | -4/+4 |
| | |||||
* | and recommit with fixed GC | otto | 2016-10-21 | 1 | -103/+112 |
| | |||||
* | backout for now; flag combination GC is not ok | otto | 2016-10-20 | 1 | -110/+103 |
| | |||||
* | Also place canaries in > page sized objects (if C is in effect); ok tb@ | otto | 2016-10-20 | 1 | -103/+110 |
| | |||||
* | Wrap _malloc_init() so internal calls go directly | guenther | 2016-10-15 | 1 | -1/+2 |
| | | | | | prodded by otto@ ok kettenis@ otto@ | ||||
* | 0xd0 -> 0xdb; ok deraadt@ millert@ tedu@ | otto | 2016-10-14 | 1 | -3/+3 |
| | |||||
* | optimize canary code a bit by storing offset of sizes table instead of | otto | 2016-10-12 | 1 | -5/+7 |
| | | | | recomputing it all the time | ||||
* | stray tab | otto | 2016-10-07 | 1 | -2/+2 |
| | |||||
* | Beter implementation of chunk canaries: store size in chunk meta data | otto | 2016-10-07 | 1 | -61/+63 |
| | | | | instead of chunk itself; does not change actual allocated size; ok tedu@ | ||||
* | Delete casts to off_t and size_t that are implied by assignments | guenther | 2016-09-21 | 1 | -7/+7 |
| | | | | | | | or prototypes. Ditto for some of the char* and void* casts too. verified no change to instructions on ILP32 (i386) and LP64 (amd64) ok natano@ abluhm@ deraadt@ millert@ | ||||
* | move page junking tp unmap(), right before we stick the region in the cache; | otto | 2016-09-18 | 1 | -6/+6 |
| | | | | ok tedu@ | ||||
* | Less lock contention by using more pools for mult-threaded programs. | otto | 2016-09-01 | 1 | -85/+178 |
| | | | | tested by many (thanks!) ok tedu, guenther@ | ||||
* | black magic for sparc page size can go | tedu | 2016-09-01 | 1 | -4/+2 |
| | |||||
* | wrterror() is fatal, delete dead code; ok tom@ natano@ tedu@ | otto | 2016-08-17 | 1 | -61/+22 |
| | |||||
* | J/j is a three valued option, document and fix code to actuall support that | otto | 2016-07-06 | 1 | -3/+5 |
| | | | | | with a little help from jmc@ for the man page bits ok jca@ and a reluctant tedu@ | ||||
* | adapt S option: add C, rm F (not relevant with 0 cache and disables | otto | 2016-06-30 | 1 | -3/+3 |
| | | | | chunk rnd), rm P: is default | ||||
* | Back out previous; otto saw a potential race that could lead to a | tb | 2016-06-28 | 1 | -32/+23 |
| | | | | | | double unmap and I experienced a much more unstable firefox. discussed with otto on icb | ||||
* | defer munmap to after unlocking malloc. this can (unfortunately) be an | tedu | 2016-06-27 | 1 | -23/+32 |
| | | | | | | | expensive syscall, and we don't want to tie up other threads. there's no need to hold the lock, so defer it to afterwards. from Michael McConville ok deraadt | ||||
* | two times a define to an inline function, from Michael McConville; ok djm@ | otto | 2016-04-12 | 1 | -10/+19 |
| | |||||
* | tweak MALLOC_STATS printing (switched off by default), prodded by | otto | 2016-04-09 | 1 | -14/+14 |
| | | | | Michael McConville |