Paths

Table of Contentst

Differential D54177

lio: Avoid out-of-bounds read or write MAC address
ClosedPublic
Actions

Authored by zlei on Dec 11 2025, 4:24 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jul 17, 9:37 PM

	Unknown Object (File)
	Thu, Jul 16, 10:22 AM

	Unknown Object (File)
	Wed, Jul 15, 6:58 AM

	Unknown Object (File)
	Wed, Jul 15, 6:58 AM

	Unknown Object (File)
	Mon, Jul 13, 7:22 AM

	Unknown Object (File)
	Sun, Jul 12, 11:59 PM

	Unknown Object (File)
	Tue, Jul 7, 8:24 AM

	Unknown Object (File)
	Sun, Jul 5, 6:09 AM

View All 81 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGf119719571b8: lio: Avoid out-of-bounds read or write MAC address
rGe8de565f6bb8: lio: Avoid out-of-bounds read or write MAC address
rGc162d7febbc8: lio: Avoid out-of-bounds read or write MAC address
rG094626d3a500: lio: Avoid out-of-bounds read or write MAC address

Summary

MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Dec 11 2025, 4:24 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 11 2025, 4:24 PM

zlei requested review of this revision.Dec 11 2025, 4:24 PM

Good find.

sys/dev/liquidio/lio_main.c
1375–1376	Change 6 to ETHER_ADDR_LEN too?

This revision is now accepted and ready to land.Dec 14 2025, 5:15 PM

zlei added inline comments.Dec 15 2025, 9:36 AM

sys/dev/liquidio/lio_main.c
1375–1376	Actually this loop copying can be shortened by `memcpy()`. I'll update for better readability.

Replaced loop copying with memcpy() for better readability.

This revision now requires review to proceed.Dec 15 2025, 3:32 PM

markj accepted this revision as: markj.Dec 15 2025, 9:26 PM

This revision is now accepted and ready to land.Dec 15 2025, 9:26 PM

Closed by commit rG094626d3a500: lio: Avoid out-of-bounds read or write MAC address (authored by zlei). · Explain WhyDec 16 2025, 4:42 AM

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rG094626d3a500: lio: Avoid out-of-bounds read or write MAC address.

In D54177#1238745, @markj wrote:

Good find.

Forgot to mention that I found this while hacking on a multicast filter bug related to qlnxe https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=290973 . The driver qlnxe read multicast ethernet address with a length ETHER_HDR_LEN which is unnecessary long.

zlei added a commit: rGc162d7febbc8: lio: Avoid out-of-bounds read or write MAC address.Jan 31 2026, 5:05 PM

zlei added a commit: rGe8de565f6bb8: lio: Avoid out-of-bounds read or write MAC address.Jan 31 2026, 6:31 PM

zlei added a commit: rGf119719571b8: lio: Avoid out-of-bounds read or write MAC address.Jan 31 2026, 6:44 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

liquidio/

4 lines

10 lines

Diff 168163

sys/dev/liquidio/lio_ioctl.c

Loading...

sys/dev/liquidio/lio_main.c

Loading...