2

g_strdup を解放しようとしていますが、何が間違っているのかわかりません。

を使用しvalgrind --tool=memcheck --leak-check=yes ./a.outて取得し続けます:

==4506== 40 bytes in 10 blocks are definitely lost in loss record 2 of 9
==4506==    at 0x4024C1C: malloc (vg_replace_malloc.c:195)
==4506==    by 0x40782E3: g_malloc (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x4090CA8: g_strdup (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x8048722: add_inv (dup.c:26)
==4506==    by 0x80487E6: main (dup.c:47)

==4506== 504 bytes in 1 blocks are possibly lost in loss record 4 of 9
==4506==    at 0x4023E2E: memalign (vg_replace_malloc.c:532)
==4506==    by 0x4023E8B: posix_memalign (vg_replace_malloc.c:660)
==4506==    by 0x408D61D: ??? (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x408E5AC: g_slice_alloc (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x4061628: g_hash_table_new_full (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x40616C7: g_hash_table_new (in /lib/libglib-2.0.so.0.2200.3)
==4506==    by 0x8048795: main (dup.c:42)

解放するためにさまざまな方法を試しましたが、これまでのところ成功していません。どんな助けにも感謝します。ありがとう

ところで:コンパイルして正常に実行されます。


#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <glib.h>
#include <stdint.h>

struct s { 
    char *data;
};

static GHashTable *hashtable1;
static GHashTable *hashtable2;

static void add_inv(GHashTable *table, const char *key)
{
    gpointer old_value, old_key;
    gint value;

    if(g_hash_table_lookup_extended(table,key, &old_key, &old_value)){
        value = GPOINTER_TO_INT(old_value);
        value = value + 2;
        /*g_free (old_key);*/
    } else {
        value = 5;
    }   
    g_hash_table_replace(table, g_strdup(key), GINT_TO_POINTER(value));
}

static void print_hash_kv (gpointer key, gpointer value, gpointer user_data){
    gchar *k = (gchar *) key;
    gchar *h = (gchar *) value;
    printf("%s: %d \n",k, h); 
}

int main(int argc, char *argv[]){

    struct s t;

    t.data = "bar";

    int i,j;
    hashtable1 = g_hash_table_new(g_str_hash, g_str_equal);
    hashtable2 = g_hash_table_new(g_str_hash, g_str_equal);

    for(i=0;i<10;i++){
        add_inv(hashtable1, t.data);
        add_inv(hashtable2, t.data);
    }

    /*free(t.data);*/
    /*free(t.data);*/

    g_hash_table_foreach (hashtable1, print_hash_kv, NULL); 
    g_hash_table_foreach (hashtable2, print_hash_kv, NULL);

    g_hash_table_destroy(hashtable1);   
    g_hash_table_destroy(hashtable2);

    return 0;
}
4

2 に答える 2

1

g_strdup(key) はメモリを割り当てますが、誰もそのメモリを解放しません。

g_hash_table_new を使用する代わりに、独自の key_destroy_func を g_hash_table_new_full に提供する必要があります。

于 2010-03-13T18:01:15.197 に答える
-1

g_strdup()ハッシュテーブルに入れるすべてのキーはなぜですか?そうする必要がありますか?GTK がchar*ハッシュテーブル内のすべてのキーを複製する必要がある場合、実行時にキーが解放されるに違いありませんg_hash_table_destroy()

GTK ドキュメントを確認してください。

于 2010-03-13T18:04:53.393 に答える