netlink: do not memcpy 0 bytes from a NULL pointer
AbandonedPublic
Actions

Authored by emaste on Mar 6 2023, 4:34 PM.

Tags

None

Referenced Files

	F151279295: D38927.id118402.diff
	Tue, Apr 7, 7:15 AM

	Unknown Object (File)
	Fri, Apr 3, 6:11 PM

	Unknown Object (File)
	Fri, Apr 3, 10:37 AM

	Unknown Object (File)
	Thu, Apr 2, 5:02 PM

	Unknown Object (File)
	Sun, Mar 22, 12:22 PM

	Unknown Object (File)
	Sun, Mar 22, 3:31 AM

	Unknown Object (File)
	Mar 5 2026, 2:26 PM

	Unknown Object (File)
	Mar 4 2026, 6:12 PM

View All 86 Files

Subscribers

glebius

melifaro

Details

Reviewers

melifaro

Commits

rGd9f07162932b: netlink: do not memcpy 0 bytes from a NULL pointer
rG305aaa93e0c8: netlink: do not memcpy 0 bytes from a NULL pointer

Summary

Although it works in practice it is UB, and might break after a future
compiler update.

Sponsored by: The FreeBSD Foundation

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste created this revision.Mar 6 2023, 4:34 PM

Herald added a reviewer: melifaro. · View Herald TranscriptMar 6 2023, 4:34 PM

Herald added subscribers: glebius, melifaro. · View Herald Transcript

emaste requested review of this revision.Mar 6 2023, 4:34 PM

emaste added inline comments.

sys/netlink/route/iface.c
464	free(NULL) is fine of course, but I think including it in the if body is more clear

Just missed 055776c84a41bb88c6ccd67615ea172864f6804c :)

Although that said I think we ought to include either D38927 or 055776c84a41bb88c6ccd67615ea172864f6804c in 13.2.

emaste added a commit: rG305aaa93e0c8: netlink: do not memcpy 0 bytes from a NULL pointer.Mar 6 2023, 6:12 PM

emaste added a commit: rGd9f07162932b: netlink: do not memcpy 0 bytes from a NULL pointer.Mar 6 2023, 7:38 PM

Revision Contents
Changeset List

Path

Size

sys/

netlink/

route/

iface.c

7 lines

Diff 118402

View Options

netlink: do not memcpy 0 bytes from a NULL pointerAbandonedPublicActions