From 5917347c6ffc0f8a7d7bc2cb640600024a2552d6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicolas=20Fran=C3=A7ois?= Date: Thu, 15 Aug 2013 17:03:18 +0200 Subject: [PATCH] Fix boundary conditions. * lib/subordinateio.c (find_free_range): max is allowed for new ranges. --- ChangeLog | 5 +++++ lib/subordinateio.c | 7 +++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 7ae37b1a..910d02f1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2013-08-15 Nicolas François + + * lib/subordinateio.c (find_free_range): max is allowed for new + ranges. + 2013-08-15 Nicolas François * libmisc/find_new_sub_gids.c: Remove dead code. diff --git a/lib/subordinateio.c b/lib/subordinateio.c index e79d9e57..9eff5fbf 100644 --- a/lib/subordinateio.c +++ b/lib/subordinateio.c @@ -225,8 +225,11 @@ static unsigned long find_free_range(struct commonio_db *db, /* Find the top end of the hole before this range */ high = first; - if (high > max) - high = max; + + /* Don't allocate IDs after max (included) */ + if (high > max + 1) { + high = max + 1; + } /* Is the hole before this range large enough? */ if ((high > low) && ((high - low) >= count))