Page MenuHomeFreeBSD
Differential D20387

ficl pfopen: verify file
ClosedPublic
Actions

Authored by sjg on May 23 2019, 8:10 PM.
Tags
None
Referenced Files
F150091285: D20387.id57856.diff
Sun, Mar 29, 7:47 AM
F150067622: D20387.diff
Sun, Mar 29, 3:17 AM
Unknown Object (File)
Fri, Mar 20, 4:24 AM
Unknown Object (File)
Thu, Mar 19, 7:45 PM
Unknown Object (File)
Sat, Mar 14, 6:58 PM
Unknown Object (File)
Wed, Mar 11, 1:15 PM
Unknown Object (File)
Thu, Mar 5, 12:53 PM
Unknown Object (File)
Mon, Mar 2, 6:35 AM
View All Files
Subscribers
imp

Details

Reviewers
kd
stevek
Commits
rS348449: ficl pfopen: verify file
rS348249: ficl pfopen: verify file
Summary

If the file is verified - do not allow write
otherwise do not allow read.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sjg created this revision.May 23 2019, 8:10 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 23 2019, 8:10 PM
Harbormaster completed remote builds in B24447: Diff 57796.May 23 2019, 8:10 PM
sjg updated this revision to Diff 57797.May 23 2019, 8:14 PM

Add ficl.mk

Harbormaster completed remote builds in B24448: Diff 57797.May 23 2019, 8:14 PM
sjg added reviewers: kd, stevek.May 23 2019, 8:15 PM
stevek added inline comments.May 23 2019, 9:27 PM
stand/ficl/loader.c
509 ↗(On Diff #57797)

I believe this should be written like so:
if ((mode & O_ACCMODE) != O_WRONLY) {

515 ↗(On Diff #57797)

Similar thing here:
if ((mode & O_ACCMODE) != O_RDONLY) {

sjg updated this revision to Diff 57803.May 23 2019, 9:33 PM

Use O_ACCMODE

Harbormaster completed remote builds in B24452: Diff 57803.May 23 2019, 9:33 PM
sjg marked 2 inline comments as done.May 23 2019, 9:33 PM

Good point

sjg updated this revision to Diff 57805.May 23 2019, 9:43 PM

stand.h needs to define O_ACCMODE

Harbormaster completed remote builds in B24454: Diff 57805.May 23 2019, 9:43 PM
kd accepted this revision.May 24 2019, 7:33 AM

Did some basic testing and it looks ok.

This revision is now accepted and ready to land.May 24 2019, 7:33 AM
stevek accepted this revision.May 24 2019, 5:58 PM
Closed by commit rS348249: ficl pfopen: verify file (authored by sjg). · Explain WhyMay 24 2019, 7:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
stand/
4 lines
ficl/
17 lines
libsa/
1 line

Diff 57856

View Options

head/stand/ficl.mk

Loading...
View Options

head/stand/ficl/loader.c

Loading...
View Options

head/stand/libsa/stand.h

Loading...