Skip to content

Commit 5050bc2

Browse files
authored
[CWS] zero related events when getting from pool instead of before put (#44331)
### What does this PR do? `TestPutBackPoolEvent` has been quite flaky lately, this PR moves the zeroing of the event to the Get side of the related events pool operations, ensuring the event is in know state when we use it. ### Motivation ### Describe how you validated your changes ### Additional Notes Co-authored-by: paul.cacheux <paul.cacheux@datadoghq.com>
1 parent 0198fc5 commit 5050bc2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/security/probe/probe_ebpf.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1046,14 +1046,14 @@ func (p *EBPFProbe) zeroEvent() *model.Event {
10461046

10471047
func (p *EBPFProbe) getPoolEvent() *model.Event {
10481048
event := p.eventPool.Get()
1049+
relatedEventZeroer(event)
10491050
event.FieldHandlers = p.fieldHandlers
10501051
return event
10511052
}
10521053

10531054
var relatedEventZeroer = model.NewEventZeroer()
10541055

10551056
func (p *EBPFProbe) putBackPoolEvent(event *model.Event) {
1056-
relatedEventZeroer(event)
10571057
p.eventPool.Put(event)
10581058
}
10591059

0 commit comments

Comments
 (0)