Don't call stat() just to determine symbolic lengths

The new bftw_cached_stat() helper gets us stat info if we already have
it, but doesn't call stat() if we don't.  In that case we just take a
guess for the initial length to readlinkat().  This lets us avoid stat()
entirely in many cases for -lname and -printf %l.

1 files changed, 3 insertions, 5 deletions

diff --git a/eval.c b/eval.c
index f8b9a6d..b50efd7 100644
--- a/eval.c
+++ b/eval.c
@@ -484,12 +484,10 @@ bool eval_lname(const struct expr *expr, struct eval_state *state) {
 		goto done;
 	}
 
-	const struct bfs_stat *statbuf = eval_stat(state);
-	if (!statbuf) {
-		goto done;
-	}
+	const struct bfs_stat *statbuf = bftw_cached_stat(ftwbuf, BFS_STAT_NOFOLLOW);
+	size_t len = statbuf ? statbuf->size : 0;
 
-	name = xreadlinkat(ftwbuf->at_fd, ftwbuf->at_path, statbuf->size);
+	name = xreadlinkat(ftwbuf->at_fd, ftwbuf->at_path, len);
 	if (!name) {
 		eval_report_error(state);
 		goto done;