From: Ulrich Drepper <drepper@redhat.com>
Date: Sun, 18 May 2008 04:17:14 +0000 (+0000)
Subject: * nscd/mem.c (markrange): Add assert to check entries are all
X-Git-Tag: cvs/fedora-glibc-20080518T1017~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=77d40f10ecc05ede24ae4f34d75f920ddf94a773;p=platform%2Fupstream%2Fglibc.git

* nscd/mem.c (markrange): Add assert to check entries are all

	aligned.  Small cleanup in bitmap use.
---

diff --git a/ChangeLog b/ChangeLog
index e0ef19b..1aa2a36 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2008-05-17  Ulrich Drepper  <drepper@redhat.com>
 
+	* nscd/mem.c (markrange): Add assert to check entries are all
+	aligned.  Small cleanup in bitmap use.
+
 	* nscd/nscd.h (mem_in_flight): Replace blockaddr field with
 	blockoff of type nscd_ssize_t.
 	* nscd/mem.c (gc): Simplify markrange call for on-flight blocks.
diff --git a/nscd/mem.c b/nscd/mem.c
index 903f91f..0bcb2aa 100644
--- a/nscd/mem.c
+++ b/nscd/mem.c
@@ -79,6 +79,7 @@ static void
 markrange (BITMAP_T *mark, ref_t start, size_t len)
 {
   /* Adjust parameters for block alignment.  */
+  assert ((start & BLOCK_ALIGN_M1) == 0);
   start /= BLOCK_ALIGN;
   len = (len + BLOCK_ALIGN_M1) / BLOCK_ALIGN;
 
@@ -93,7 +94,7 @@ markrange (BITMAP_T *mark, ref_t start, size_t len)
 	  return;
 	}
 
-      mark[elem++] |= 0xff << (start % BITS);
+      mark[elem++] |= ALLBITS << (start % BITS);
       len -= BITS - (start % BITS);
     }