Re: [Mesa-dev] [PATCH 6/6] mesa: Fix hash table leak

2012-05-25 Thread Ian Romanick
On 05/24/2012 06:29 PM, Stéphane Marchesin wrote: From: Antoine Labour When a value was replaced, the new key was strdup'd and leaked. To fix this, we modify the hash table implementation to return whether the value was replaced and free() the (now useless) duplicate string. This looks good, b

[Mesa-dev] [PATCH 6/6] mesa: Fix hash table leak

2012-05-24 Thread Stéphane Marchesin
From: Antoine Labour When a value was replaced, the new key was strdup'd and leaked. To fix this, we modify the hash table implementation to return whether the value was replaced and free() the (now useless) duplicate string. --- src/mesa/program/hash_table.c |5 +++-- src/mesa/program/hash_