Single-use wrapper functions that just call a wrapper function

around calloc() and end up producing duplicate error messages are
just confusing.

Eliminate new_hash_bucket(), new_lease(), new_leases(), new_group().

1 files changed, 1 insertions, 31 deletions

diff --git a/usr.sbin/dhcpd/alloc.c b/usr.sbin/dhcpd/alloc.c
index 69309c0deec..efa7d2a74f4 100644
--- a/usr.sbin/dhcpd/alloc.c
+++ b/usr.sbin/dhcpd/alloc.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: alloc.c,v 1.5 2010/01/01 01:47:41 krw Exp $	*/
+/*	$OpenBSD: alloc.c,v 1.6 2010/01/01 06:25:37 krw Exp $	*/
 
 /* Memory allocation... */
 
@@ -104,14 +104,6 @@ new_hash_table(int count, char *name)
 	return (rval);
 }
 
-struct hash_bucket *
-new_hash_bucket(char *name)
-{
-	struct hash_bucket *rval = dmalloc(sizeof(struct hash_bucket), name);
-
-	return (rval);
-}
-
 void
 free_hash_bucket(struct hash_bucket *ptr, char *name)
 {
@@ -169,28 +161,6 @@ free_lease_state(struct lease_state *ptr, char *name)
 	free_lease_states = ptr;
 }
 
-struct lease *
-new_leases(int n, char *name)
-{
-	struct lease *rval = dmalloc(n * sizeof(struct lease), name);
-	return (rval);
-}
-
-struct lease *
-new_lease(char *name)
-{
-	struct lease *rval = dmalloc(sizeof(struct lease), name);
-	return (rval);
-}
-
-struct group *
-new_group(char *name)
-{
-	struct group *rval =
-		dmalloc(sizeof(struct group), name);
-	return (rval);
-}
-
 void
 free_lease(struct lease *ptr, char *name)
 {