Per some comments from Lars Kellogg-Stedman <lars@larsshack.org>,

make xreadlink() return NULL on failure, and make sure everyone
uses the interface correctly.
 -Erik


git-svn-id: svn://busybox.net/trunk/busybox@2551 69ca8d6d-28ef-0310-b511-8ec308f3f277

1 files changed, 2 insertions, 0 deletions

diff --git a/miscutils/readlink.c b/miscutils/readlink.c
index 226649544..c46ebd108 100644
--- a/miscutils/readlink.c
+++ b/miscutils/readlink.c
@@ -37,6 +37,8 @@ int readlink_main(int argc, char **argv)
                 show_usage();
 
         buf = xreadlink(argv[1]);
+        if (!buf)
+                return EXIT_FAILURE;
         puts(buf);
 #ifdef BB_FEATURE_CLEAN_UP
         free(buf);