aboutsummaryrefslogtreecommitdiff
path: root/lib/std/zon/stringify.zig
diff options
context:
space:
mode:
authorRyan Liptak <squeek502@hotmail.com>2025-02-17 17:23:07 -0800
committerAlex Rønne Petersen <alex@alexrp.com>2025-02-20 22:16:05 +0100
commitbbd13ab961ab49e01d5699eec195929cae7bf25a (patch)
tree0ca7c34c7e58e61e1be7d1ebbcc058023d0266a7 /lib/std/zon/stringify.zig
parent253e641ea3ee9f6f1ba14abc5615b861cbc2e347 (diff)
downloadzig-bbd13ab961ab49e01d5699eec195929cae7bf25a.tar.gz
zig-bbd13ab961ab49e01d5699eec195929cae7bf25a.zip
rescanMac: Avoid overallocating/overreading by millions of bytes
readAtLeast is greedy and will read the entire length of the buffer if it can. However, reading past the end of the cert in this case is useless, so reading the full length of the buffer just puts an increasingly large (due to the growth algorithm of ArrayList) collection of wasted bytes after each cert in cb.bytes. In practical terms, this ends up saving potentially millions of bytes of wasted reads/allocations. In my testing, after reading the keychain files on my machine, cb.bytes ends up with these capacities: - Before: cb.bytes.capacity = 32720747 - After: cb.bytes.capacity = 251937 That's a decrease of 99.2% Additionally, swaps to readNoEof since it should be an error to hit EOF without reading the full cert size.
Diffstat (limited to 'lib/std/zon/stringify.zig')
0 files changed, 0 insertions, 0 deletions