From 28355a36da6ad4d1b55ba9bb146fb0b6586e7edb Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Mon, 7 May 2001 17:48:28 +0000 Subject: Per some comments from Lars Kellogg-Stedman , make xreadlink() return NULL on failure, and make sure everyone uses the interface correctly. -Erik --- libbb/xreadlink.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'libbb/xreadlink.c') diff --git a/libbb/xreadlink.c b/libbb/xreadlink.c index 66f63b883..932e487a5 100644 --- a/libbb/xreadlink.c +++ b/libbb/xreadlink.c @@ -1,5 +1,6 @@ /* - * xreadlink.c - safe implementation of readlink + * xreadlink.c - safe implementation of readlink. + * Returns a NULL on failure... */ #include @@ -22,8 +23,10 @@ extern char *xreadlink(const char *path) do { buf = xrealloc(buf, bufsize += GROWBY); readsize = readlink(path, buf, bufsize); /* 1st try */ - if (readsize == -1) - perror_msg("%s:%s", applet_name, path); + if (readsize == -1) { + perror_msg("%s:%s", applet_name, path); + return NULL; + } } while (bufsize < readsize + 1); -- cgit v1.2.3