mbox series

[0/2] plugins: Freeing allocated values in hash tables.

Message ID 20210421140934.7561-1-ma.mandourr@gmail.com (mailing list archive)
Headers show
Series plugins: Freeing allocated values in hash tables. | expand

Message

Mahmoud Abumandour April 21, 2021, 2:09 p.m. UTC 
A hash table made using ``g_hash_table_new`` requires manually
freeing any dynamically allocated keys/values. The two patches
in this series fixes this issue in hotblocks and hotpages plugins.

Mahmoud Mandour (2):
  plugins/hotblocks: Properly freed the hash table values
  plugins/hotpages: Properly freed the hash table values

 contrib/plugins/hotblocks.c | 3 ++-
 contrib/plugins/hotpages.c  | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

Comments

Alex Bennée April 26, 2021, 4:53 p.m. UTC | #1 
Mahmoud Mandour <ma.mandourr@gmail.com> writes:

> A hash table made using ``g_hash_table_new`` requires manually
> freeing any dynamically allocated keys/values. The two patches
> in this series fixes this issue in hotblocks and hotpages plugins.

Queued to plugins/next, thanks.
Alex Bennée April 26, 2021, 4:54 p.m. UTC | #2 
Alex Bennée <alex.bennee@linaro.org> writes:

> Mahmoud Mandour <ma.mandourr@gmail.com> writes:
>
>> A hash table made using ``g_hash_table_new`` requires manually
>> freeing any dynamically allocated keys/values. The two patches
>> in this series fixes this issue in hotblocks and hotpages plugins.
>
> Queued to plugins/next, thanks.

Oops, dequeuing and applying v2 ;-)