Hi,

defect found by coverity, the attached patch is an attempt
to fix it

thanks & regards

Jaroslav
From 460f4f10ba70319f80e475f531034c4f263ce137 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jaroslav=20=C5=A0karvada?= <jskar...@redhat.com>
Date: Fri, 30 Jan 2015 11:10:46 +0100
Subject: [PATCH] fts: fix double call of close(p->fts_symfd)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Jaroslav Å karvada <jskar...@redhat.com>
---
 lib/fts.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/fts.c b/lib/fts.c
index 5cb4cb0..3eadee7 100644
--- a/lib/fts.c
+++ b/lib/fts.c
@@ -1084,8 +1084,8 @@ cd_dot_dot:
                         __set_errno (saved_errno);
                         p->fts_errno = errno;
                         SET(FTS_STOP);
-                }
-                (void)close(p->fts_symfd);
+                } else
+                        (void)close(p->fts_symfd);
         } else if (!(p->fts_flags & FTS_DONTCHDIR) &&
                    fts_safe_changedir(sp, p->fts_parent, -1, "..")) {
                 p->fts_errno = errno;
-- 
2.1.0

Reply via email to