Commit 2d5fc3ad authored by Chris Allegretta's avatar Chris Allegretta
Browse files

- nano.c:do_int_speller() - Fix FD leaks which occur outside of errors (David Benbennick)

git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@1363 35c25a1d-7b9e-4130-9fde-d3aeb78583b8
parent d757e258
Showing with 3 additions and 0 deletions
+3 -0
...@@ -157,6 +157,7 @@ Changes ...@@ -157,6 +157,7 @@ Changes
either be improved or removed. I chose improved]. either be improved or removed. I chose improved].
- Change all child error checks to use one goto (gasp!) called - Change all child error checks to use one goto (gasp!) called
close_pipes_and_exit, so we don't leak FDs. close_pipes_and_exit, so we don't leak FDs.
- Fix FD leaks which occur outside of errors (David Benbennick).
do_int_speller(), do_alt_speller() do_int_speller(), do_alt_speller()
- Programs now return char *, NULL for successful completion, - Programs now return char *, NULL for successful completion,
otherwise the error string to display. This allows us to give otherwise the error string to display. This allows us to give
......
...@@ -1766,6 +1766,7 @@ char *do_int_speller(char *tempfile_name) ...@@ -1766,6 +1766,7 @@ char *do_int_speller(char *tempfile_name)
exit(1); exit(1);
} }
close(spell_fd[0]);
close(sort_fd[1]); close(sort_fd[1]);
/* A new process to run uniq in */ /* A new process to run uniq in */
...@@ -1791,6 +1792,7 @@ char *do_int_speller(char *tempfile_name) ...@@ -1791,6 +1792,7 @@ char *do_int_speller(char *tempfile_name)
exit(1); exit(1);
} }
close(sort_fd[0]);
close(uniq_fd[1]); close(uniq_fd[1]);
/* Child process was not forked successfully */ /* Child process was not forked successfully */
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment