Page MenuHomeFreeBSD
Differential D17236

Improve loader passwords
ClosedPublic
Actions

Authored by trasz on Sep 19 2018, 1:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 26 2024, 9:51 AM
Unknown Object (File)
Dec 25 2024, 4:56 AM
Unknown Object (File)
Nov 26 2024, 1:09 AM
Unknown Object (File)
Nov 23 2024, 6:11 PM
Unknown Object (File)
Nov 14 2024, 4:02 AM
Unknown Object (File)
Nov 13 2024, 5:44 AM
Unknown Object (File)
Oct 24 2024, 4:23 PM
Unknown Object (File)
Oct 24 2024, 4:23 PM
View All 37 Files
Subscribers
imp

Details

Reviewers
kevans
Commits
rS338886: Improve loader passwords:
Summary

Improve loader passwords:

  1. Be clear about which password is being requested
  2. Remove extraneous whitespace between the prompt and the cursor
  3. Move the twiddle to where the prompt is, instead of two characters to the right
  4. Fix erasing the 'incorrect password' message when retrying; previously it was erased partially
  5. Remove the unneeded exclamation mark

Diff Detail

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

Event Timeline

trasz created this revision.Sep 19 2018, 1:50 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 19 2018, 1:50 PM
Harbormaster completed remote builds in B19667: Diff 48210.Sep 19 2018, 1:50 PM
trasz edited the summary of this revision. (Show Details)Sep 19 2018, 1:52 PM
trasz added a reviewer: kevans.
kevans accepted this revision.Sep 19 2018, 4:25 PM

I don't have time to do a thorough review/test, but nothing here offends my greater sensibilities so it looks good to me if you've tested it. =)

This revision is now accepted and ready to land.Sep 19 2018, 4:25 PM
Closed by commit rS338886: Improve loader passwords: (authored by trasz). · Explain WhySep 22 2018, 1:15 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
stand/
lua/
19 lines

Diff 48355

View Options

head/stand/lua/password.lua

Show First 20 Lines • Show All 47 Lines • ▼ Show 20 Lines
.Fa fd .Fa fd
is a descriptor to be operated on by is a descriptor to be operated on by
.Fa cmd .Fa cmd
as described below. as described below.
Depending on the value of Depending on the value of
.Fa cmd , .Fa cmd ,
.Fn fcntl .Fn fcntl
can take an additional third argument can take an additional third argument
.Fa "long arg" . .Fa arg .
jhbUnsubmitted
Done
Inline Actions

Maybe just say the type of the 3rd arg depends on the operation (and document the type for each operation that takes the third arg) instead of trying to assume it's a single type? In CheriBSD we have a wrapper in libc to deal with this and have fcntl() as a truly var-args function. On existing platforms var-args is close enough to work for passing syscall arguments, but not all ABIs require this. (var-args are passed as a block in a bounded pointer on CHERI platforms and not just a continuation of register arguments).

jhb: Maybe just say the type of the 3rd arg depends on the operation (and document the type for each…
kibAuthorUnsubmitted
Done
Inline Actions

I pushed everything but this change, the review is updated with the man page changes.

kib: I pushed everything but this change, the review is updated with the man page changes.
Unless otherwise noted below for a specific operation,
emasteUnsubmitted
Done
Inline Actions

maybe "with type dependent on the specific operation"

or maybe

can take an additional third argument
.Fa arg.
Unless otherwise noted below for a specific operation,
.Fa arg
has type
.Vt int .
emaste: maybe "with type dependent on the specific operation" or maybe ``` can take an additional…
.Fa arg
has type
.Vt int .
.Bl -tag -width F_DUP2FD_CLOEXEC .Bl -tag -width F_DUP2FD_CLOEXEC
.It Dv F_DUPFD .It Dv F_DUPFD
Return a new descriptor as follows: Return a new descriptor as follows:
.Pp .Pp
.Bl -bullet -compact -offset 4n .Bl -bullet -compact -offset 4n
.It .It
Lowest numbered available descriptor greater than or equal to Lowest numbered available descriptor greater than or equal to
.Fa arg . .Fa arg .
▲ Show 20 Lines • Show All 121 Lines • ▼ Show 20 Lines
.It Dv F_GET_SEALS .It Dv F_GET_SEALS
Get seals associated with the file, if the underlying filesystem supports seals. Get seals associated with the file, if the underlying filesystem supports seals.
.It Dv F_ISUNIONSTACK .It Dv F_ISUNIONSTACK
Check if the vnode is part of a union stack (either the "union" flag from Check if the vnode is part of a union stack (either the "union" flag from
.Xr mount 2 .Xr mount 2
or unionfs). or unionfs).
This is a hack not intended to be used outside of libc. This is a hack not intended to be used outside of libc.
.It Dv F_KINFO .It Dv F_KINFO
Fills a Fills a
markjUnsubmitted
Done
Inline Actions
.It Dv F_KINFO
- Fills
+ Fills a
.Vt struct kinfo_file
markj:
.Vt struct kinfo_file .Vt struct kinfo_file
for the file referenced by the specified file descriptor. for the file referenced by the specified file descriptor.
The The
.Fa arg .Fa arg
argument should point to the storage for argument should point to the storage for
.Vt struct kinfo_file . .Vt struct kinfo_file .
The The
.Va kf_structsize .Va kf_structsize
▲ Show 20 Lines • Show All 78 Lines • ▼ Show 20 Lines
Seals are on a per-inode basis and require support by the underlying filesystem. Seals are on a per-inode basis and require support by the underlying filesystem.
If the underlying filesystem does not support seals, If the underlying filesystem does not support seals,
.Dv F_ADD_SEALS .Dv F_ADD_SEALS
and and
.Dv F_GET_SEALS .Dv F_GET_SEALS
will fail and return will fail and return
.Dv EINVAL . .Dv EINVAL .
.Pp .Pp
Several commands are available for doing advisory file locking; Several operations are available for doing advisory file locking;
emasteUnsubmitted
Done
Inline Actions

Is "command" or "operation" the better term, in general? I think we should be consistent between this sentence and the added text above/below

emaste: Is "command" or "operation" the better term, in general? I think we should be consistent…
they all operate on the following structure: they all operate on the following structure:
.Bd -literal .Bd -literal
struct flock { struct flock {
off_t l_start; /* starting offset */ off_t l_start; /* starting offset */
off_t l_len; /* len = 0 means until end of file */ off_t l_len; /* len = 0 means until end of file */
pid_t l_pid; /* lock owner */ pid_t l_pid; /* lock owner */
short l_type; /* lock type: read/write, etc. */ short l_type; /* lock type: read/write, etc. */
short l_whence; /* type of l_start */ short l_whence; /* type of l_start */
int l_sysid; /* remote system id or zero for local */ int l_sysid; /* remote system id or zero for local */
}; };
.Ed .Ed
These advisory file locking operations take a pointer to
emasteUnsubmitted
Done
Inline Actions

These commands take a pointer to struct flock as the third argument arg.
or
These advisory file locking commands take a pointer to struct flock as the third argument arg.

emaste: These commands take a pointer to struct flock as the third argument arg. or These advisory file…
emasteUnsubmitted
Done
Inline Actions

Maybe "operations" here too for consistency?
But from the context this one is clear so I don't mind either way.

emaste: Maybe "operations" here too for consistency? But from the context this one is clear so I don't…
.Vt struct flock
as the third argument
.Fa arg .
The commands available for advisory record locking are as follows: The commands available for advisory record locking are as follows:
.Bl -tag -width F_SETLKWX .Bl -tag -width F_SETLKWX
.It Dv F_GETLK .It Dv F_GETLK
Get the first lock that blocks the lock description pointed to by the Get the first lock that blocks the lock description pointed to by the
third argument, third argument,
.Fa arg , .Fa arg ,
taken as a pointer to a taken as a pointer to a
.Fa "struct flock" .Fa "struct flock"
▲ Show 20 Lines • Show All 477 Lines • Show Last 20 Lines