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
F168528030: D30952.id95553.diff
Fri, Aug 28, 8:52 PM
Unknown Object (File)
Wed, Aug 26, 10:44 PM
Unknown Object (File)
Tue, Aug 18, 6:44 PM
Unknown Object (File)
Sat, Aug 15, 10:05 AM
Unknown Object (File)
Fri, Aug 14, 10:53 PM
Unknown Object (File)
Wed, Aug 12, 9:16 PM
Unknown Object (File)
Sun, Aug 2, 4:55 AM
Unknown Object (File)
Sun, Aug 2, 1:52 AM
View All 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...