Differential D33953

ktls: Zero out TLS_GET_RECORD control messages
ClosedPublic
Actions

Authored by markj on Jan 19 2022, 8:18 PM.

Details

Reviewers

jhb

Commits

rG6be8944d96d2: ktls: Zero out TLS_GET_RECORD control messages

Summary

Otherwise we end up copying uninitialized bytes into the socket buffer.

Reported by: KMSAN

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 43991
Build 40879: arc lint + arc unit

Event Timeline

markj created this revision.Jan 19 2022, 8:18 PM

Herald added subscribers: ae, imp. · View Herald TranscriptJan 19 2022, 8:18 PM

markj requested review of this revision.Jan 19 2022, 8:18 PM

Harbormaster completed remote builds in B43991: Diff 101652.Jan 19 2022, 8:18 PM

Huh, we should be setting all of the fields? Is there a padding byte I guess?

In D33953#768089, @jhb wrote:

Huh, we should be setting all of the fields? Is there a padding byte I guess?

Yes, sorry the description isn't quite right. There is a single padding byte.

jhb accepted this revision.Jan 20 2022, 8:36 PM

This revision is now accepted and ready to land.Jan 20 2022, 8:36 PM

Closed by commit rG6be8944d96d2: ktls: Zero out TLS_GET_RECORD control messages (authored by markj). · Explain WhyJan 20 2022, 8:44 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG6be8944d96d2: ktls: Zero out TLS_GET_RECORD control messages.

Revision Contents
Changeset List

Path

Size

sys/

dev/

cxgbe/

tom/

t4_tls.c

1 line

kern/

uipc_ktls.c

1 line

Diff 101652

View Options

sys/dev/cxgbe/tom/t4_tls.c