diff --git a/commands/readlink.c b/commands/readlink.c index a19c8e0..2dfa566 100644 --- a/commands/readlink.c +++ b/commands/readlink.c @@ -44,17 +44,21 @@ if (argc < optind + 2) return COMMAND_ERROR_USAGE; - if (readlink(argv[optind], realname, PATH_MAX - 1) < 0) - goto err; - if (canonicalize) { - char *buf = canonicalize_path(realname); + char *buf = canonicalize_path(argv[optind]); + struct stat s; if (!buf) goto err; + if (stat(dirname(argv[optind]), &s) || !S_ISDIR(s.st_mode)) { + free(buf); + goto err; + } setenv(argv[optind + 1], buf); free(buf); } else { + if (readlink(argv[optind], realname, PATH_MAX - 1) < 0) + goto err; setenv(argv[optind + 1], realname); } @@ -65,15 +69,16 @@ } BAREBOX_CMD_HELP_START(readlink) -BAREBOX_CMD_HELP_TEXT("Read value of a symbolic link and store it into VARIABLE.") +BAREBOX_CMD_HELP_TEXT("Read value of a symbolic link or canonical file name and store it into VARIABLE.") BAREBOX_CMD_HELP_TEXT("") BAREBOX_CMD_HELP_TEXT("Options:") -BAREBOX_CMD_HELP_OPT ("-f", "canonicalize by following first symlink"); +BAREBOX_CMD_HELP_OPT("-f", "canonicalize by following symlinks;") +BAREBOX_CMD_HELP_OPT("", "final component need not exist"); BAREBOX_CMD_HELP_END BAREBOX_CMD_START(readlink) .cmd = do_readlink, - BAREBOX_CMD_DESC("read value of a symbolic link") + BAREBOX_CMD_DESC("read value of a symbolic link or canonical file name") BAREBOX_CMD_OPTS("[-f] FILE VARIABLE") BAREBOX_CMD_GROUP(CMD_GRP_FILE) BAREBOX_CMD_HELP(cmd_readlink_help)