From 9915ebec8a20b9fccd8459c6a5a63c08d1fae635 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Wed, 4 Dec 2013 17:55:17 +0000 Subject: [PATCH] * dtable.cc (dtable::find_unused_handle): When extending, always make sure that there is a NOFILE_INCR chunk following the free fd. --- winsup/cygwin/ChangeLog | 5 +++++ winsup/cygwin/dtable.cc | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 89dce26f6..490a7d675 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2013-12-04 Christopher Faylor + + * dtable.cc (dtable::find_unused_handle): When extending, always make + sure that there is a NOFILE_INCR chunk following the free fd. + 2013-12-03 Christopher Faylor * select.cc (select): Add workaround for, as yet undebugged, diff --git a/winsup/cygwin/dtable.cc b/winsup/cygwin/dtable.cc index e97966b31..454798427 100644 --- a/winsup/cygwin/dtable.cc +++ b/winsup/cygwin/dtable.cc @@ -223,7 +223,9 @@ dtable::delete_archetype (fhandler_base *fh) int dtable::find_unused_handle (size_t start) { - size_t extendby = (start >= size) ? 1 + start - size : NOFILE_INCR; + /* When extending, always make sure that there is a NOFILE_INCR chunk + following the empty fd. */ + size_t extendby = NOFILE_INCR + ((start >= size) ? 1 + start - size : 0); /* This do loop should only ever execute twice. */ int res = -1;