Paths

Table of Contentst

Differential D18663

Collapse $HOME into "~" in PS1.
ClosedPublic
Actions

Authored by trasz on Dec 27 2018, 2:57 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 6, 6:36 PM

	Unknown Object (File)
	Thu, Nov 6, 5:46 PM

	Unknown Object (File)
	Wed, Nov 5, 11:31 PM

	Unknown Object (File)
	Mon, Oct 27, 1:11 AM

	Unknown Object (File)
	Mon, Oct 27, 1:11 AM

	Unknown Object (File)
	Mon, Oct 27, 1:11 AM

	Unknown Object (File)
	Mon, Oct 27, 1:11 AM

	Unknown Object (File)
	Sun, Oct 26, 7:40 PM

View All 75 Files

Subscribers

Details

Reviewers

Commits

rS342577: Make sh(1) collapse $HOME into "~" in PS1.

Summary

Make sh(1) collapse $HOME into "~" in PS1.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21740
Build 21018: arc lint + arc unit

Event Timeline

trasz created this revision.Dec 27 2018, 2:57 PM

Herald added a reviewer: jilles. · View Herald TranscriptDec 27 2018, 2:57 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B21739: Diff 52339.Dec 27 2018, 2:57 PM

jilles requested changes to this revision.Dec 27 2018, 3:07 PM

jilles added inline comments.

bin/sh/parser.c
2046–2047	A path `/home/user1` will match `/home/user` here. Also, if `HOME` is `/`, results will be strange and the substitution should probably be disabled.
2050–2052	The buffer length passed to `strlcpy` seems one byte too long. Also, is it valid to calculate `&ps[i + 1]`?

This revision now requires changes to proceed.Dec 27 2018, 3:07 PM

Fix things.

Harbormaster completed remote builds in B21740: Diff 52340.Dec 27 2018, 4:46 PM

trasz marked an inline comment as done.Dec 27 2018, 4:47 PM

trasz added inline comments.

bin/sh/parser.c
2050–2052	Hm, I'm not much of a C lawyer, but I've used that syntax since... ever. What's wrong with it?

jilles accepted this revision.Dec 28 2018, 1:16 PM

jilles added inline comments.

bin/sh/parser.c
2050–2052	The syntax is fine, but I wondered whether `ps[i + 1]` is always within bounds. It turns out that it is because the `i < PROMPTLEN - 1` check above.

This revision is now accepted and ready to land.Dec 28 2018, 1:16 PM

Closed by commit rS342577: Make sh(1) collapse $HOME into "~" in PS1. (authored by trasz). · Explain WhyDec 28 2018, 5:51 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

bin/

sh/

22 lines

Diff 52340

bin/sh/parser.c

Loading...