Merged revisions 221971 via svnmerge from

https://origsvn.digium.com/svn/asterisk/trunk

................
  r221971 | tilghman | 2009-10-02 11:59:57 -0500 (Fri, 02 Oct 2009) | 9 lines

  Merged revisions 221970 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.4

  ........
    r221970 | tilghman | 2009-10-02 11:58:03 -0500 (Fri, 02 Oct 2009) | 2 lines

    Ensure the result of the hash function is positive.  Negative array offsets suck.
  ........
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.1@221973 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/funcs/func_lock.c b/funcs/func_lock.c
index 432292b39405cabee560a82b55eca203d90723f1..5d116e82c8460331a978299bef9d6945bc040978 100644 (file)
--- a/funcs/func_lock.c
+++ b/funcs/func_lock.c
@@ -153,7 +153,7 @@ static void *lock_broker(void *unused)
 
 static int null_hash_cb(const void *obj, const int flags)
 {
-       return abs((int)(long) obj);
+       return (int)(long) obj;
 }
 
 static int null_cmp_cb(void *obj, void *arg, int flags)

diff --git a/main/astobj2.c b/main/astobj2.c
index 378a8311edbcec7dcbd6bff649fe589ca316eb78..c49432fa10f49636bbeb1b5934039a8898949ec4 100644 (file)
--- a/main/astobj2.c
+++ b/main/astobj2.c
@@ -503,7 +503,7 @@ static struct bucket_list *__ao2_link(struct ao2_container *c, void *user_data)
        if (!p)
                return NULL;
 
-       i = c->hash_fn(user_data, OBJ_POINTER);
+       i = abs(c->hash_fn(user_data, OBJ_POINTER));
 
        ao2_lock(c);
        i %= c->n_buckets;