[BusyBox] [PATCH] readlink -f

[Colin Watson]

On Tue, Nov 16, 2004 at 11:29:44AM +0300, Vladimir N. Oleynik wrote:
> Colin,
> >-	if (argc != 2)
> >+	if (optind + 1 != argc)
> > 		bb_show_usage();
> 
> But, if CONFIG_FEATURE_READLINK_FOLLOW unset, then "optind" may be 
> undefined?

Fair point. Another iteration of readlink-follow.patch attached.

Thanks,

-- 
Colin Watson                                  [cjwatson at flatline.org.uk]
-------------- next part --------------
Index: debianutils/Config.in
===================================================================
RCS file: /var/cvs/busybox/debianutils/Config.in,v
retrieving revision 1.7
diff -p -u -r1.7 Config.in
--- debianutils/Config.in	15 Mar 2004 08:28:24 -0000	1.7
+++ debianutils/Config.in	16 Nov 2004 11:17:50 -0000
@@ -24,6 +24,12 @@ config CONFIG_READLINK
 	  This program reads a symbolic link and returns the name
 	  of the file it points to
 
+config CONFIG_FEATURE_READLINK_FOLLOW
+	bool "  Enable canonicalization by following all symlinks (-f)"
+	default n
+	help
+	  Enable the readlink option (-f).
+
 config CONFIG_RUN_PARTS
 	bool "run-parts"
 	default n
Index: debianutils/readlink.c
===================================================================
RCS file: /var/cvs/busybox/debianutils/readlink.c,v
retrieving revision 1.2
diff -p -u -r1.2 readlink.c
--- debianutils/readlink.c	19 Mar 2003 09:11:41 -0000	1.2
+++ debianutils/readlink.c	16 Nov 2004 11:17:50 -0000
@@ -23,18 +23,38 @@
 #include <errno.h>
 #include <unistd.h>
 #include <stdlib.h>
+#include <getopt.h>
 #include "busybox.h"
 
+#ifdef CONFIG_FEATURE_READLINK_FOLLOW
+# define READLINK_FOLLOW	"f"
+# define READLINK_FLAG_f	(1 << 0)
+#else
+# define READLINK_FOLLOW	""
+#endif
+
+static const char readlink_options[] = READLINK_FOLLOW;
+
 int readlink_main(int argc, char **argv)
 {
 	char *buf = NULL;
+	unsigned long opt = bb_getopt_ulflags(argc, argv, readlink_options);
+#ifdef CONFIG_FEATURE_READLINK_FOLLOW
+	RESERVE_CONFIG_BUFFER(resolved_path, PATH_MAX);
+#endif
 
 	/* no options, no getopt */
 
-	if (argc != 2)
+	if (optind + 1 != argc)
 		bb_show_usage();
 
-	buf = xreadlink(argv[1]);
+#ifdef CONFIG_FEATURE_READLINK_FOLLOW
+	if (opt & READLINK_FLAG_f) {
+		buf = realpath(argv[optind], resolved_path);
+	} else
+#endif
+		buf = xreadlink(argv[optind]);
+
 	if (!buf)
 		return EXIT_FAILURE;
 	puts(buf);
Index: include/usage.h
===================================================================
RCS file: /var/cvs/busybox/include/usage.h,v
retrieving revision 1.222
diff -p -u -r1.222 usage.h
--- include/usage.h	14 Sep 2004 16:23:56 -0000	1.222
+++ include/usage.h	16 Nov 2004 11:17:51 -0000
@@ -1985,10 +1985,18 @@
 	"\t-s\tSet the system date and time (default).\n" \
 	"\t-p\tPrint the date and time."
 
+#ifdef CONFIG_FEATURE_READLINK_FOLLOW
+#define USAGE_READLINK_FOLLOW(a) a
+#else
+#define USAGE_READLINK_FOLLOW(a)
+#endif
+
 #define readlink_trivial_usage \
-	""
+	USAGE_READLINK_FOLLOW("[-f] ") "FILE"
 #define readlink_full_usage \
-	"Displays the value of a symbolic link."
+	"Displays the value of a symbolic link." \
+	USAGE_READLINK_FOLLOW("\n\nOptions:\n" \
+	"\t-f\tcanonicalize by following all symlinks")
 
 #define realpath_trivial_usage \
 	"pathname  ..."