From moxuansheng at gmail.com Sat May 30 03:37:52 2015 From: moxuansheng at gmail.com (xuansheng mo) Date: Sat, 30 May 2015 10:37:52 +0800 Subject: [Iftop-users] about hash_create function Message-ID: hello: The iftop is very userful while inspecting the flows between hosts. And I want to add some log features in our environment. The functions serv_hash_create and ns_hash_create are allocating and returning a hash: hash_type* ns_hash_create() { hash_type* hash_table; hash_table = xcalloc(hash_table_size, sizeof *hash_table); hash_table->size = hash_table_size; hash_table->compare = &ns_hash_compare; hash_table->hash = &ns_hash_hash; hash_table->delete_key = &ns_hash_delete_key; hash_table->copy_key = &ns_hash_copy_key; hash_initialise(hash_table); return hash_table; } hash_type* serv_hash_create() { hash_type* hash_table; hash_table = xcalloc(hash_table_size, sizeof *hash_table); hash_table->size = hash_table_size; hash_table->compare = &serv_hash_compare; hash_table->hash = &serv_hash_hash; hash_table->delete_key = &serv_hash_delete_key; hash_table->copy_key = &serv_hash_copy_key; hash_initialise(hash_table); return hash_table; } Is the clause " hash_table = xcalloc(hash_table_size, sizeof *hash_table);" a bug? It allocate a hash_type array with hash_table_size items. Should be " hash_table = xcalloc(1, sizeof *hash_table);"? And allocate the hash buckets with " hash_initialise(hash_table);" Thanks. -------------- next part -------------- An HTML attachment was scrubbed... URL: