[PATCH] vpn: Do not access invalid hash table pointer

Praveen C pravinc123 at gmail.com
Sun Nov 25 20:13:46 PST 2012


Hi Daniel,
The g_hash_table_lookup routine already has a NULL check for the hash
handle.
So your patch doesn't makes sense to me.

gpointer
g_hash_table_lookup (GHashTable   *hash_table,
                     gconstpointer key)
{
  GHashNode *node;
  guint      node_index;

  g_return_val_if_fail (hash_table != NULL, NULL);

................



Thanks & Regards,
Praveen C



On Mon, Nov 26, 2012 at 3:23 AM, Daniel Wagner <wagi at monom.org> wrote:

> This patch is not enough since a few lines later the hash table is
> accessed again... damit
>
> ______________________________**_________________
> connman mailing list
> connman at connman.net
> http://lists.connman.net/**listinfo/connman<http://lists.connman.net/listinfo/connman>
>



More information about the connman mailing list