From: Ulrich Drepper <drepper@redhat.com>
Date: Sun, 25 Nov 2007 21:29:30 +0000 (+0000)
Subject: Clear array before use.
X-Git-Tag: cvs/fedora-glibc-20071212T1051~20
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e8667ddc01786cfe79861a0031d6244212fd7a65;p=platform%2Fupstream%2Fglibc.git

Clear array before use.

	* nscd/hstcache.c (cache_addhst): Likewise.
	* nscd/grpcache.c (cache_addgr): Likewise.
	* nscd/servicescache.c (cache_addserv): Likewise.
	* nscd/pwdcache.c (cache_addpw): Likewise.
	* nscd/initgrcache.c (addinitgroupsX): Likewise.
---

diff --git a/ChangeLog b/ChangeLog
index cc33cef..4a0fd89 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,10 +4,15 @@
 	* nscd/mem.c (MAX_STACK_USE): Remove definition here.
 	(gc): Initialize stack_used based on allocation in prune_cache.
 	* nscd/cache.c (prune_cache): Use heap for mark array if necessary.
-	Clear array bfore use.
+	Clear array before use.
 
 	* nscd/aicache.c (addhstaiX): Update statistics counter in case
 	memory allocate failed.
+	* nscd/hstcache.c (cache_addhst): Likewise.
+	* nscd/grpcache.c (cache_addgr): Likewise.
+	* nscd/servicescache.c (cache_addserv): Likewise.
+	* nscd/pwdcache.c (cache_addpw): Likewise.
+	* nscd/initgrcache.c (addinitgroupsX): Likewise.
 
 2007-11-23  Ulrich Drepper  <drepper@redhat.com>
 
diff --git a/nscd/grpcache.c b/nscd/grpcache.c
index 9cb6105..002f04f 100644
--- a/nscd/grpcache.c
+++ b/nscd/grpcache.c
@@ -285,6 +285,8 @@ cache_addgr (struct database_dyn *db, int fd, request_header *req,
 		  dataset = memcpy (newp, dataset, total + n);
 		  alloca_used = false;
 		}
+	      else
+		++db->head->addfailed;
 
 	      /* Mark the old record as obsolete.  */
 	      dh->usable = false;
diff --git a/nscd/hstcache.c b/nscd/hstcache.c
index c93691d..cc04158 100644
--- a/nscd/hstcache.c
+++ b/nscd/hstcache.c
@@ -325,6 +325,8 @@ cache_addhst (struct database_dyn *db, int fd, request_header *req,
 		      dataset = memcpy (newp, dataset, total + req->key_len);
 		      alloca_used = false;
 		    }
+		  else
+		    ++db->head->addfailed;
 		}
 
 	      /* Mark the old record as obsolete.  */
diff --git a/nscd/initgrcache.c b/nscd/initgrcache.c
index bb4ac37..157cd78 100644
--- a/nscd/initgrcache.c
+++ b/nscd/initgrcache.c
@@ -110,7 +110,7 @@ addinitgroupsX (struct database_dyn *db, int fd, request_header *req,
   bool all_tryagain = true;
   bool any_success = false;
 
-  /* This is temporary memory, we need not (ad must not) call
+  /* This is temporary memory, we need not (and must not) call
      mempool_alloc.  */
   // XXX This really should use alloca.  need to change the backends.
   gid_t *groups = (gid_t *) malloc (size * sizeof (gid_t));
@@ -338,6 +338,8 @@ addinitgroupsX (struct database_dyn *db, int fd, request_header *req,
 		  dataset = memcpy (newp, dataset, total + req->key_len);
 		  alloca_used = false;
 		}
+	      else
+		++db->head->addfailed;
 
 	      /* Mark the old record as obsolete.  */
 	      dh->usable = false;
diff --git a/nscd/pwdcache.c b/nscd/pwdcache.c
index 51ea113..bc1b6ba 100644
--- a/nscd/pwdcache.c
+++ b/nscd/pwdcache.c
@@ -280,6 +280,8 @@ cache_addpw (struct database_dyn *db, int fd, request_header *req,
 		  dataset = memcpy (newp, dataset, total + n);
 		  alloca_used = false;
 		}
+	      else
+		++db->head->addfailed;
 
 	      /* Mark the old record as obsolete.  */
 	      dh->usable = false;
diff --git a/nscd/servicescache.c b/nscd/servicescache.c
index d6bf51d..e122cb3 100644
--- a/nscd/servicescache.c
+++ b/nscd/servicescache.c
@@ -272,6 +272,8 @@ cache_addserv (struct database_dyn *db, int fd, request_header *req,
 		  dataset = memcpy (newp, dataset, total + req->key_len);
 		  alloca_used = false;
 		}
+	      else
+		++db->head->addfailed;
 
 	      /* Mark the old record as obsolete.  */
 	      dh->usable = false;