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

	Unknown Object (File)
	Mon, Jan 5, 5:22 PM

	Unknown Object (File)
	Sun, Jan 4, 9:01 PM

	Unknown Object (File)
	Tue, Dec 30, 8:08 PM

	Unknown Object (File)
	Nov 21 2025, 1:28 PM

	Unknown Object (File)
	Nov 14 2025, 12:02 AM

	Unknown Object (File)
	Nov 5 2025, 12:35 AM

	Unknown Object (File)
	Nov 5 2025, 12:33 AM

	Unknown Object (File)
	Nov 5 2025, 12:29 AM

View All 77 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