Fix undefined behavior.
ClosedPublic
Actions

Authored by kib on Oct 23 2019, 3:54 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Feb 28, 6:42 PM

	Unknown Object (File)
	Fri, Feb 27, 10:44 PM

	Unknown Object (File)
	Fri, Feb 27, 4:16 PM

	Unknown Object (File)
	Fri, Feb 27, 7:18 AM

	Unknown Object (File)
	Tue, Feb 24, 5:20 PM

	Unknown Object (File)
	Sat, Feb 7, 10:16 PM

	Unknown Object (File)
	Sat, Feb 7, 4:18 PM

	Unknown Object (File)
	Jan 31 2026, 8:43 AM

View All 74 Files

Subscribers

None

Details

Reviewers

imp

Commits

rS353930: Fix undefined behavior.

Summary

Create a sequence point by ending a full expression for call to vspace() and use of the globals which are modified by vspace.

Reported by: imp

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.Oct 23 2019, 3:54 PM

Harbormaster completed remote builds in B27170: Diff 63584.Oct 23 2019, 3:54 PM

Looks good to me. It's one of the two ways that I saw to fix it, and this is likely faster than the other way (pass the limit to vspace()).

This revision is now accepted and ready to land.Oct 23 2019, 3:56 PM

Closed by commit rS353930: Fix undefined behavior. (authored by kib). · Explain WhyOct 23 2019, 4:07 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rS353930: Fix undefined behavior..

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

vfs_subr.c

12 lines

Diff 63585

View Options

Fix undefined behavior.ClosedPublicActions