svn commit: r316112 - head/sys/boot/common
Toomas Soome
tsoome at FreeBSD.org
Tue Mar 28 21:50:13 UTC 2017
Author: tsoome
Date: Tue Mar 28 21:50:11 2017
New Revision: 316112
URL: https://svnweb.freebsd.org/changeset/base/316112
Log:
loader: ls command should display file types properly
With some file system the ls is unable to display file types.
Reviewed by: allanjude
Approved by: allanjude (mentor)
Differential Revision: https://reviews.freebsd.org/D10066
Modified:
head/sys/boot/common/ls.c
Modified: head/sys/boot/common/ls.c
==============================================================================
--- head/sys/boot/common/ls.c Tue Mar 28 21:47:12 2017 (r316111)
+++ head/sys/boot/common/ls.c Tue Mar 28 21:50:11 2017 (r316112)
@@ -60,26 +60,26 @@ command_ls(int argc, char *argv[])
{
int fd;
struct stat sb;
- struct dirent *d;
+ struct dirent *d;
char *buf, *path;
char lbuf[128]; /* one line */
int result, ch;
int verbose;
-
+
result = CMD_OK;
fd = -1;
verbose = 0;
optind = 1;
optreset = 1;
while ((ch = getopt(argc, argv, "l")) != -1) {
- switch(ch) {
+ switch (ch) {
case 'l':
verbose = 1;
break;
case '?':
default:
/* getopt has already reported an error */
- return(CMD_OK);
+ return (CMD_OK);
}
}
argv += (optind - 1);
@@ -91,6 +91,18 @@ command_ls(int argc, char *argv[])
path = argv[1];
}
+ if (stat(path, &sb) == 0 && !S_ISDIR(sb.st_mode)) {
+ if (verbose) {
+ printf(" %c %8d %s\n",
+ typestr[sb.st_mode >> 12],
+ (int)sb.st_size, path);
+ } else {
+ printf(" %c %s\n",
+ typestr[sb.st_mode >> 12], path);
+ }
+ return (CMD_OK);
+ }
+
fd = ls_getdir(&path);
if (fd == -1) {
result = CMD_ERROR;
@@ -102,19 +114,28 @@ command_ls(int argc, char *argv[])
while ((d = readdirfd(fd)) != NULL) {
if (strcmp(d->d_name, ".") && strcmp(d->d_name, "..")) {
- if (verbose) {
+ if (d->d_type == 0 || verbose) {
/* stat the file, if possible */
sb.st_size = 0;
+ sb.st_mode = 0;
buf = malloc(strlen(path) + strlen(d->d_name) + 2);
- sprintf(buf, "%s/%s", path, d->d_name);
- /* ignore return, could be symlink, etc. */
- if (stat(buf, &sb))
- sb.st_size = 0;
- free(buf);
- sprintf(lbuf, " %c %8d %s\n", typestr[d->d_type],
+ if (buf != NULL) {
+ sprintf(buf, "%s/%s", path, d->d_name);
+ /* ignore return, could be symlink, etc. */
+ if (stat(buf, &sb)) {
+ sb.st_size = 0;
+ sb.st_mode = 0;
+ }
+ free(buf);
+ }
+ }
+ if (verbose) {
+ snprintf(lbuf, sizeof(lbuf), " %c %8d %s\n",
+ typestr[d->d_type? d->d_type:sb.st_mode >> 12],
(int)sb.st_size, d->d_name);
} else {
- sprintf(lbuf, " %c %s\n", typestr[d->d_type], d->d_name);
+ snprintf(lbuf, sizeof(lbuf), " %c %s\n",
+ typestr[d->d_type? d->d_type:sb.st_mode >> 12], d->d_name);
}
if (pager_output(lbuf))
goto out;
@@ -124,9 +145,8 @@ command_ls(int argc, char *argv[])
pager_close();
if (fd != -1)
close(fd);
- if (path != NULL)
- free(path);
- return(result);
+ free(path); /* ls_getdir() did allocate path */
+ return (result);
}
/*
@@ -145,6 +165,11 @@ ls_getdir(char **pathp)
/* one extra byte for a possible trailing slash required */
path = malloc(strlen(*pathp) + 2);
+ if (path == NULL) {
+ snprintf(command_errbuf, sizeof (command_errbuf),
+ "out of memory");
+ goto out;
+ }
strcpy(path, *pathp);
/* Make sure the path is respectable to begin with */
@@ -153,7 +178,7 @@ ls_getdir(char **pathp)
"bad path '%s'", path);
goto out;
}
-
+
/* If there's no path on the device, assume '/' */
if (*cp == 0)
strcat(path, "/");
@@ -176,12 +201,12 @@ ls_getdir(char **pathp)
}
*pathp = path;
- return(fd);
+ return (fd);
out:
free(path);
*pathp = NULL;
if (fd != -1)
close(fd);
- return(-1);
+ return (-1);
}
More information about the svn-src-head
mailing list