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)
	Fri, May 15, 6:55 PM

	Unknown Object (File)
	Fri, May 15, 6:42 PM

	Unknown Object (File)
	Sat, May 9, 9:30 AM

	Unknown Object (File)
	Fri, May 8, 6:16 PM

	Unknown Object (File)
	Sat, Apr 25, 7:54 AM

	Unknown Object (File)
	Apr 16 2026, 4:49 AM

	Unknown Object (File)
	Apr 16 2026, 12:56 AM

	Unknown Object (File)
	Apr 15 2026, 12:11 PM

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