From fea1c178b47cf7ac95ab27c39a98e0464e19976c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ville=20Skytt=C3=A4?= <ville.skytta@iki.fi>
Date: Fri, 25 Jan 2013 00:06:34 +0200
Subject: [PATCH] Avoid sourcing dirs in completion loader to avoid fd leaks
 (RedHat: #903540).

---
 bash_completion | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/bash_completion b/bash_completion
index 870811b4..952f5662 100644
--- a/bash_completion
+++ b/bash_completion
@@ -1918,11 +1918,12 @@ complete -F _minimal ''
 # set up dynamic completion loading
 _completion_loader()
 {
-    local compdir=./completions
-    [[ $BASH_SOURCE == */* ]] && compdir="${BASH_SOURCE%/*}/completions"
+    local compfile=./completions
+    [[ $BASH_SOURCE == */* ]] && compfile="${BASH_SOURCE%/*}/completions"
+    compfile+="${1##*/}"
 
-    # Try basename.
-    . "$compdir/${1##*/}" &>/dev/null && return 124
+    # Avoid trying to source dirs; https://bugzilla.redhat.com/903540
+    [[ -f "$compfile" ]] && . "$compfile" &>/dev/null && return 124
 
     # Need to define *something*, otherwise there will be no completion at all.
     complete -F _minimal "$1" && return 124