Page MenuHomeFreeBSD
Differential D22126

Fix undefined behavior.
ClosedPublic
Actions

Authored by kib on Oct 23 2019, 3:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 12, 11:50 AM
Unknown Object (File)
Fri, Sep 11, 7:51 AM
Unknown Object (File)
Fri, Sep 4, 7:47 AM
Unknown Object (File)
Mon, Aug 31, 7:32 PM
Unknown Object (File)
Mon, Aug 24, 9:58 PM
Unknown Object (File)
Sat, Aug 22, 12:13 AM
Unknown Object (File)
Thu, Aug 20, 6:38 PM
Unknown Object (File)
Aug 11 2026, 5:21 AM
View All 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
imp accepted this revision.Oct 23 2019, 3:56 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

PathSize
head/
sys/
kern/
12 lines

Diff 63585

View Options

head/sys/kern/vfs_subr.c

Loading...