diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2008-05-18 04:25:44 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2008-05-18 04:25:44 +0000 |
| commit | fa526148d6d78bf97f5628a023d290d071b47592 (patch) | |
| tree | 174a71fb9489261f7ab72cba4cf8073a3a3bfa76 | |
| parent | 77d40f10ecc05ede24ae4f34d75f920ddf94a773 (diff) | |
| download | glibc-fa526148d6d78bf97f5628a023d290d071b47592.tar.gz glibc-fa526148d6d78bf97f5628a023d290d071b47592.tar.xz glibc-fa526148d6d78bf97f5628a023d290d071b47592.zip | |
* nscd/mem.c (gc): Correctly determine highest used array element
in mark.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | nscd/mem.c | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog index 1aa2a36e74..e17090f990 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2008-05-17 Ulrich Drepper <drepper@redhat.com> + * nscd/mem.c (gc): Correctly determine highest used array element + in mark. + * nscd/mem.c (markrange): Add assert to check entries are all aligned. Small cleanup in bitmap use. diff --git a/nscd/mem.c b/nscd/mem.c index 0bcb2aaef5..bf3777d119 100644 --- a/nscd/mem.c +++ b/nscd/mem.c @@ -131,8 +131,8 @@ gc (struct database_dyn *db) size_t stack_used = sizeof (bool) * db->head->module; if (__builtin_expect (stack_used > MAX_STACK_USE, 0)) stack_used = 0; - size_t memory_needed = ((db->head->first_free / BLOCK_ALIGN + BITS - 1) - / BITS) * sizeof (BITMAP_T); + size_t nmark = (db->head->first_free / BLOCK_ALIGN + BITS - 1) / BITS; + size_t memory_needed = nmark * sizeof (BITMAP_T); if (stack_used + memory_needed <= MAX_STACK_USE) { mark = (BITMAP_T *) alloca (memory_needed); @@ -234,7 +234,7 @@ gc (struct database_dyn *db) qsort (he, cnt, sizeof (struct hashentry *), sort_he); /* Determine the highest used address. */ - size_t high = sizeof (mark); + size_t high = nmark; while (high > 0 && mark[high - 1] == 0) --high; |