open-telemetry / opentelemetry-ebpf-profiler

The production-scale datacenter profiler (C/C++, Go, Rust, Python, Java, NodeJS, .NET, PHP, Ruby, Perl, ...)
Apache License 2.0
2.42k stars 262 forks source link

BPF: dotnet: make nibble map reading more compliant #142

Closed fabled closed 2 months ago

fabled commented 2 months ago

Remove the non-obvious reading past end-of-array to next member, and add modulo arithmetic to ensure that bpf verifier is happy.

fixes #141