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.