-
Notifications
You must be signed in to change notification settings - Fork 5.3k
memory corruption in histogram change #3223
Copy link
Copy link
Closed
Description
There is a memory corruption issue somewhere in the following commit range:
b9e7961...a328bc5
The crash signatures are not consistent, but when it first happened I was pretty sure it was somehow related to e293ffb.
However, I reverted that, and we are still seeing crashes. (I also spent 2 hours looking at the code and it seems fine.)
Given the changes in the above range, I think the next most likely candidate is the histogram change: 9a1e49f
I'm going to spend time today digging into this, but I wanted to open this in case anyone else wants to scan through the diffs and help with bug spotting.
cc @envoyproxy/maintainers @jmarantz @mrice32 @ramaraochavali
Reactions are currently unavailable