From: Hallvard Furuseth <hallvard@openldap.org>
Date: Wed, 21 Oct 2009 11:31:59 +0000 (+0000)
Subject: Cleanup - macroize magic constant (ITS#5909)
X-Git-Tag: ACLCHECK_0~180
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=47617b033c82d9fc07b1302aef101a5ea545bd5e;p=openldap

Cleanup - macroize magic constant (ITS#5909)
---

diff --git a/libraries/liblutil/tavl.c b/libraries/liblutil/tavl.c
index 519b169911..a6a2bf4a5f 100644
--- a/libraries/liblutil/tavl.c
+++ b/libraries/liblutil/tavl.c
@@ -21,6 +21,7 @@
 
 #include "portable.h"
 
+#include <limits.h>
 #include <stdio.h>
 #include <ac/stdlib.h>
 
@@ -35,6 +36,9 @@
 #define AVL_INTERNAL
 #include "avl.h"
 
+/* Maximum tree depth this host's address space could support */
+#define MAX_TREE_DEPTH	(sizeof(void *) * CHAR_BIT)
+
 static const int avl_bfs[] = {LH, RH};
 
 /*
@@ -189,8 +193,8 @@ tavl_delete( Avlnode **root, void* data, AVL_CMP fcmp )
 	int side, side_bf, shorter, nside = -1;
 
 	/* parent stack */
-	Avlnode *pptr[sizeof(void *)*8];
-	unsigned char pdir[sizeof(void *)*8];
+	Avlnode *pptr[MAX_TREE_DEPTH];
+	unsigned char pdir[MAX_TREE_DEPTH];
 	int depth = 0;
 
 	if ( *root == NULL )