Re: [PATCH] Fix several missing close() calls.

2013-01-24 Thread Chet Ramey
On 1/24/13 5:53 PM, Roman Rakus wrote: > There are missing calls of close() leading to resource leak (fd leak). Thanks for the report and fix. Chet -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa, vita brevis'' - Hippocrates Chet Ramey, ITS, CWRUc

[PATCH] Fix several missing close() calls.

2013-01-24 Thread Roman Rakus
There are missing calls of close() leading to resource leak (fd leak). Simple reproducer: . / and /proc/$$/fd contain one open fd for each above call Signed-off-by: Roman Rakus --- builtins/evalfile.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/builtins/evalfile.c b/builtins/evalfile.