Page MenuHomeFreeBSD
Differential D30952

veriexec: Fix veriexec -i's confusion between loaded and locked states
ClosedPublic
Actions

Authored by stephane.rochoy_stormshield.eu on Jun 30 2021, 6:16 AM.
Tags
None
Referenced Files
F116095678: D30952.diff
Fri, May 2, 12:16 PM
Unknown Object (File)
Tue, Apr 29, 8:35 PM
Unknown Object (File)
Thu, Apr 17, 9:27 AM
Unknown Object (File)
Sat, Apr 12, 10:26 PM
Unknown Object (File)
Mar 9 2025, 7:19 AM
Unknown Object (File)
Mar 6 2025, 4:18 PM
Unknown Object (File)
Feb 23 2025, 2:10 PM
Unknown Object (File)
Feb 23 2025, 4:12 AM
View All 90 Files
Subscribers
dab
imp

Details

Reviewers
sjg
mw
imp
Commits
rGe900898601b4: veriexec: Fix veriexec -i's confusion between loaded and locked states
rG492bf4fdad4d: veriexec: Fix veriexec -i's confusion between loaded and locked states
Summary

Calling veriexec -i locked return the state of loaded and vice-versa.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 40181
Build 37070: arc lint + arc unit

Event Timeline

stephane.rochoy_stormshield.eu created this revision.Jun 30 2021, 6:16 AM
Herald added a subscriber: imp. · View Herald TranscriptJun 30 2021, 6:16 AM
stephane.rochoy_stormshield.eu requested review of this revision.Jun 30 2021, 6:16 AM
Harbormaster completed remote builds in B40181: Diff 91536.Jun 30 2021, 6:17 AM
stephane.rochoy_stormshield.eu added a reviewer: mw.Jun 30 2021, 6:17 AM
stephane.rochoy_stormshield.eu retitled this revision from veriexec: Fix `veriexec -i`'s confusion between loaded and locked states to veriexec: Fix veriexec -i's confusion between loaded and locked states.Jun 30 2021, 6:19 AM
stephane.rochoy_stormshield.eu added a comment.Jun 30 2021, 10:06 AM

By the way, -i and -z are handled in a very similar fashion. There's probably some room for refactoring while at it.

imp accepted this revision.Jun 30 2021, 10:23 AM
This revision is now accepted and ready to land.Jun 30 2021, 10:23 AM
sjg accepted this revision.Jul 6 2021, 3:00 AM

Oops - thanks

Herald added a subscriber: dab. · View Herald TranscriptJul 6 2021, 3:00 AM
Closed by commit rG492bf4fdad4d: veriexec: Fix veriexec -i's confusion between loaded and locked states (authored by stephane.rochoy_stormshield.eu, committed by wma). · Explain WhySep 23 2021, 8:55 AM
This revision was automatically updated to reflect the committed changes.
wma added a commit: rG492bf4fdad4d: veriexec: Fix veriexec -i's confusion between loaded and locked states.
gbe added a commit: rGe900898601b4: veriexec: Fix veriexec -i's confusion between loaded and locked states.Apr 14 2023, 5:15 AM

Revision Contents
Changeset List

PathSize
sbin/
veriexec/
2 lines

Diff 91536

View Options

sbin/veriexec/veriexec.c

Loading...