Page Menu
Home
FreeBSD
Search
Configure Global Search
Log In
Files
F144925100
D55146.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Flag For Later
Award Token
Size
3 KB
Referenced Files
None
Subscribers
None
D55146.diff
View Options
diff --git a/bin/pwd/pwd.1 b/bin/pwd/pwd.1
--- a/bin/pwd/pwd.1
+++ b/bin/pwd/pwd.1
@@ -29,7 +29,7 @@
.\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
.\" SUCH DAMAGE.
.\"
-.Dd May 22, 2023
+.Dd Februrary 6, 2026
.Dt PWD 1
.Os
.Sh NAME
@@ -60,7 +60,7 @@
.El
.Pp
If no options are specified, the
-.Fl P
+.Fl L
option is assumed.
.Sh ENVIRONMENT
Environment variables used by
diff --git a/bin/pwd/pwd.c b/bin/pwd/pwd.c
--- a/bin/pwd/pwd.c
+++ b/bin/pwd/pwd.c
@@ -3,6 +3,7 @@
*
* Copyright (c) 1991, 1993, 1994
* The Regents of the University of California. All rights reserved.
+ * Copyright (c) 2026 Dag-Erling Smørgrav
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
@@ -31,40 +32,74 @@
#include <sys/param.h>
#include <sys/stat.h>
-#include <sys/types.h>
#include <err.h>
-#include <errno.h>
+#include <stdbool.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
-static char *getcwd_logical(void);
-void usage(void);
+static char *
+getcwd_logical(void)
+{
+ struct stat log, phy;
+ char *pwd, *p, *q;
+
+ /* $PWD is set and absolute */
+ if ((pwd = getenv("PWD")) == NULL || *pwd != '/')
+ return (NULL);
+ /* $PWD does not contain /./ or /../ */
+ for (p = pwd; *p; p = q) {
+ for (q = ++p; *q && *q != '/'; q++)
+ /* nothing */;
+ if ((*p == '.' && q == ++p) ||
+ (*p == '.' && q == ++p))
+ return (NULL);
+ }
+ /* $PWD refers to the current directory */
+ if (stat(pwd, &log) != 0 || stat(".", &phy) != 0 ||
+ log.st_dev != phy.st_dev || log.st_ino != phy.st_ino)
+ return (NULL);
+ return (pwd);
+}
+
+static char *
+getcwd_physical(void)
+{
+ static char pwd[MAXPATHLEN];
+
+ return (getcwd(pwd, sizeof(pwd)));
+}
+
+static void
+usage(void)
+{
+ (void)fprintf(stderr, "usage: pwd [-L | -P]\n");
+ exit(1);
+}
int
main(int argc, char *argv[])
{
- int physical;
- int ch;
- char *p;
+ char *pwd;
+ int opt;
+ bool logical;
- physical = 1;
- while ((ch = getopt(argc, argv, "LP")) != -1)
- switch (ch) {
+ logical = true;
+ while ((opt = getopt(argc, argv, "LP")) != -1) {
+ switch (opt) {
case 'L':
- physical = 0;
+ logical = true;
break;
case 'P':
- physical = 1;
+ logical = false;
break;
- case '?':
default:
usage();
}
+ }
argc -= optind;
argv += optind;
-
if (argc != 0)
usage();
@@ -72,40 +107,10 @@
* If we're trying to find the logical current directory and that
* fails, behave as if -P was specified.
*/
- if ((!physical && (p = getcwd_logical()) != NULL) ||
- (p = getcwd(NULL, 0)) != NULL)
- printf("%s\n", p);
+ if ((logical && (pwd = getcwd_logical()) != NULL) ||
+ (pwd = getcwd_physical()) != NULL)
+ printf("%s\n", pwd);
else
err(1, ".");
-
exit(0);
}
-
-void __dead2
-usage(void)
-{
-
- (void)fprintf(stderr, "usage: pwd [-L | -P]\n");
- exit(1);
-}
-
-static char *
-getcwd_logical(void)
-{
- struct stat lg, phy;
- char *pwd;
-
- /*
- * Check that $PWD is an absolute logical pathname referring to
- * the current working directory.
- */
- if ((pwd = getenv("PWD")) != NULL && *pwd == '/') {
- if (stat(pwd, &lg) == -1 || stat(".", &phy) == -1)
- return (NULL);
- if (lg.st_dev == phy.st_dev && lg.st_ino == phy.st_ino)
- return (pwd);
- }
-
- errno = ENOENT;
- return (NULL);
-}
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Sun, Feb 15, 4:29 AM (5 h, 46 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
28726731
Default Alt Text
D55146.diff (3 KB)
Attached To
Mode
D55146: pwd: Clean up and adopt POSIX semantics
Attached
Detach File
Event Timeline
Log In to Comment