From 7ead692bcf0e3aacd3eb9ff0a14811a9122c5c57 Mon Sep 17 00:00:00 2001 From: Kamil Dudka Date: Wed, 11 Feb 2015 18:41:43 +0100 Subject: [PATCH] fts: avoid crash when a cycle is added while traversing This could be triggered by auto-mounting a recursive bind mount. Reported by Michael Chapman in: https://bugzilla.redhat.com/1188498 * lib/fts.c (fts_read): Avoid removing the original hash table item when leaving a directory that caused a cycle, and preserve the FTS_DC flag. Change-Id: I24bfd7a2f46197ec8dff5ddabca3cab3837b28be Signed-off-by: Hyunjee Kim --- gnulib/lib/fts.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/gnulib/lib/fts.c b/gnulib/lib/fts.c index 1b5384b..501b334 100644 --- a/gnulib/lib/fts.c +++ b/gnulib/lib/fts.c @@ -809,9 +809,16 @@ check_for_dir: p->fts_errno = errno; SET(FTS_STOP); } - p->fts_info = p->fts_errno ? FTS_ERR : FTS_DP; - if (p->fts_errno == 0) - LEAVE_DIR (sp, p, "3"); + /* If the directory causes a cycle, preserve the FTS_DC flag and keep + * the corresponding dev/ino pair in the hash table. It is going to be + * removed when leaving the original directory. + */ + + if (p->fts_info != FTS_DC) { + p->fts_info = p->fts_errno ? FTS_ERR : FTS_DP; + if (p->fts_errno == 0) + LEAVE_DIR (sp, p, "3"); + } return ISSET(FTS_STOP) ? NULL : p; } -- 2.7.4