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)
	Sep 5 2025, 11:17 PM

	Unknown Object (File)
	Aug 24 2025, 5:11 PM

	Unknown Object (File)
	Aug 15 2025, 6:39 AM

	Unknown Object (File)
	Jul 25 2025, 8:54 PM

	Unknown Object (File)
	Jul 25 2025, 7:25 PM

	Unknown Object (File)
	Jul 25 2025, 7:06 PM

	Unknown Object (File)
	Jul 5 2025, 1:06 PM

	Unknown Object (File)
	Jul 4 2025, 10:10 AM

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