mirror of
https://gitlab.gnome.org/GNOME/glib.git
synced 2024-12-28 00:16:15 +01:00
79225a62c0
If not done, it would leak the memory as address sanitizer reports: ==1294==ERROR: LeakSanitizer: detected memory leaks Direct leak of 40 byte(s) in 1 object(s) allocated from: #0 0x7fa7a94b7602 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x98602) #1 0x44c7a7 in __config_new girepository/cmph/cmph_structs.c:11 #2 0x44aaa7 in cmph_config_new girepository/cmph/cmph.c:291 #3 0x446fb5 in _gi_typelib_hash_builder_prepare girepository/gthash.c:114 #4 0x406cf7 in add_directory_index_section girepository/girmodule.c:270 #5 0x409ee6 in _g_ir_module_build_typelib girepository/girmodule.c:546 #6 0x404ada in main tools/compiler.c:217 #7 0x7fa7a70d482f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f) ==4091==ERROR: LeakSanitizer: detected memory leaks Direct leak of 40 byte(s) in 1 object(s) allocated from: #0 0x7fc20c854602 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x98602) #1 0x44a3f3 in cmph_io_vector_new girepository/cmph/cmph.c:228 #2 0x44a965 in cmph_io_vector_adapter girepository/cmph/cmph.c:276 #3 0x446f9f in _gi_typelib_hash_builder_prepare girepository/gthash.c:113 #4 0x406cf7 in add_directory_index_section girepository/girmodule.c:270 #5 0x409ee6 in _g_ir_module_build_typelib girepository/girmodule.c:546 #6 0x404ada in main tools/compiler.c:217 #7 0x7fc20a47182f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)
224 lines
6.3 KiB
C
224 lines
6.3 KiB
C
/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*-
|
|
* GObject introspection: Typelib hashing
|
|
*
|
|
* Copyright (C) 2010 Red Hat, Inc.
|
|
*
|
|
* This library is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 2 of the License, or (at your option) any later version.
|
|
*
|
|
* This library is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
* Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with this library; if not, write to the
|
|
* Free Software Foundation, Inc., 59 Temple Place - Suite 330,
|
|
* Boston, MA 02111-1307, USA.
|
|
*/
|
|
|
|
#include <glib.h>
|
|
#include <glib-object.h>
|
|
#include <string.h>
|
|
|
|
#include "cmph/cmph.h"
|
|
#include "gitypelib-internal.h"
|
|
|
|
#define ALIGN_VALUE(this, boundary) \
|
|
(( ((unsigned long)(this)) + (((unsigned long)(boundary)) -1)) & (~(((unsigned long)(boundary))-1)))
|
|
|
|
/*
|
|
* String hashing in the typelib. We have a set of static (fixed) strings,
|
|
* and given one, we need to find its index number. This problem is perfect
|
|
* hashing: http://en.wikipedia.org/wiki/Perfect_hashing
|
|
*
|
|
* I chose CMPH (http://cmph.sourceforge.net/) as it seemed high
|
|
* quality, well documented, and easy to embed.
|
|
*
|
|
* CMPH provides a number of algorithms; I chose BDZ, because while CHD
|
|
* appears to be the "best", the simplicitly of BDZ appealed, and really,
|
|
* we're only talking about thousands of strings here, not millions, so
|
|
* a few microseconds is no big deal.
|
|
*
|
|
* In memory, the format is:
|
|
* INT32 mph_size
|
|
* MPH (mph_size bytes)
|
|
* (padding for alignment to uint32 if necessary)
|
|
* INDEX (array of guint16)
|
|
*
|
|
* Because BDZ is not order preserving, we need a lookaside table which
|
|
* maps the hash value into the directory index.
|
|
*/
|
|
|
|
struct _GITypelibHashBuilder {
|
|
gboolean prepared;
|
|
gboolean buildable;
|
|
cmph_t *c;
|
|
GHashTable *strings;
|
|
guint32 dirmap_offset;
|
|
guint32 packed_size;
|
|
};
|
|
|
|
GITypelibHashBuilder *
|
|
_gi_typelib_hash_builder_new (void)
|
|
{
|
|
GITypelibHashBuilder *builder = g_slice_new0 (GITypelibHashBuilder);
|
|
builder->c = NULL;
|
|
builder->strings = g_hash_table_new_full (g_str_hash, g_str_equal, g_free, NULL);
|
|
return builder;
|
|
}
|
|
|
|
void
|
|
_gi_typelib_hash_builder_add_string (GITypelibHashBuilder *builder,
|
|
const char *str,
|
|
guint16 value)
|
|
{
|
|
g_return_if_fail (builder->c == NULL);
|
|
g_hash_table_insert (builder->strings, g_strdup (str), GUINT_TO_POINTER ((guint) value));
|
|
}
|
|
|
|
gboolean
|
|
_gi_typelib_hash_builder_prepare (GITypelibHashBuilder *builder)
|
|
{
|
|
char **strs;
|
|
GHashTableIter hashiter;
|
|
gpointer key, value;
|
|
cmph_io_adapter_t *io;
|
|
cmph_config_t *config;
|
|
guint32 num_elts;
|
|
guint32 offset;
|
|
guint i;
|
|
|
|
if (builder->prepared)
|
|
return builder->buildable;
|
|
g_assert (builder->c == NULL);
|
|
|
|
num_elts = g_hash_table_size (builder->strings);
|
|
g_assert (num_elts <= 65536);
|
|
|
|
strs = (char**) g_new (char *, num_elts + 1);
|
|
|
|
i = 0;
|
|
g_hash_table_iter_init (&hashiter, builder->strings);
|
|
while (g_hash_table_iter_next (&hashiter, &key, &value))
|
|
{
|
|
const char *str = key;
|
|
|
|
strs[i++] = g_strdup (str);
|
|
}
|
|
strs[i++] = NULL;
|
|
|
|
io = cmph_io_vector_adapter (strs, num_elts);
|
|
config = cmph_config_new (io);
|
|
cmph_config_set_algo (config, CMPH_BDZ);
|
|
|
|
builder->c = cmph_new (config);
|
|
builder->prepared = TRUE;
|
|
if (!builder->c)
|
|
{
|
|
builder->buildable = FALSE;
|
|
goto out;
|
|
}
|
|
builder->buildable = TRUE;
|
|
g_assert (cmph_size (builder->c) == num_elts);
|
|
|
|
/* Pack a size counter at front */
|
|
offset = sizeof(guint32) + cmph_packed_size (builder->c);
|
|
builder->dirmap_offset = ALIGN_VALUE (offset, 4);
|
|
builder->packed_size = builder->dirmap_offset + (num_elts * sizeof(guint16));
|
|
out:
|
|
cmph_config_destroy (config);
|
|
cmph_io_vector_adapter_destroy (io);
|
|
return builder->buildable;
|
|
}
|
|
|
|
guint32
|
|
_gi_typelib_hash_builder_get_buffer_size (GITypelibHashBuilder *builder)
|
|
{
|
|
g_return_val_if_fail (builder != NULL, 0);
|
|
g_return_val_if_fail (builder->prepared, 0);
|
|
g_return_val_if_fail (builder->buildable, 0 );
|
|
|
|
return builder->packed_size;
|
|
}
|
|
|
|
void
|
|
_gi_typelib_hash_builder_pack (GITypelibHashBuilder *builder, guint8* mem, guint32 len)
|
|
{
|
|
guint16 *table;
|
|
GHashTableIter hashiter;
|
|
gpointer key, value;
|
|
guint32 num_elts;
|
|
guint8 *packed_mem;
|
|
|
|
g_return_if_fail (builder != NULL);
|
|
g_return_if_fail (builder->prepared);
|
|
g_return_if_fail (builder->buildable);
|
|
|
|
g_assert (len >= builder->packed_size);
|
|
g_assert ((((unsigned long)mem) & 0x3) == 0);
|
|
|
|
memset (mem, 0, len);
|
|
|
|
*((guint32*) mem) = builder->dirmap_offset;
|
|
packed_mem = (guint8*)(mem + sizeof(guint32));
|
|
cmph_pack (builder->c, packed_mem);
|
|
|
|
table = (guint16*) (mem + builder->dirmap_offset);
|
|
|
|
num_elts = g_hash_table_size (builder->strings);
|
|
g_hash_table_iter_init (&hashiter, builder->strings);
|
|
while (g_hash_table_iter_next (&hashiter, &key, &value))
|
|
{
|
|
const char *str = key;
|
|
guint16 strval = (guint16)GPOINTER_TO_UINT(value);
|
|
guint32 hashv;
|
|
|
|
hashv = cmph_search_packed (packed_mem, str, strlen (str));
|
|
g_assert (hashv >= 0 && hashv < num_elts);
|
|
table[hashv] = strval;
|
|
}
|
|
}
|
|
|
|
void
|
|
_gi_typelib_hash_builder_destroy (GITypelibHashBuilder *builder)
|
|
{
|
|
if (builder->c)
|
|
{
|
|
cmph_destroy (builder->c);
|
|
builder->c = NULL;
|
|
}
|
|
g_hash_table_destroy (builder->strings);
|
|
g_slice_free (GITypelibHashBuilder, builder);
|
|
}
|
|
|
|
guint16
|
|
_gi_typelib_hash_search (guint8* memory, const char *str, guint n_entries)
|
|
{
|
|
guint32 *mph;
|
|
guint16 *table;
|
|
guint32 dirmap_offset;
|
|
guint32 offset;
|
|
|
|
g_assert ((((unsigned long)memory) & 0x3) == 0);
|
|
mph = ((guint32*)memory)+1;
|
|
|
|
offset = cmph_search_packed (mph, str, strlen (str));
|
|
|
|
/* Make sure that offset always lies in the entries array. cmph
|
|
cometimes generates offset larger than number of entries (for
|
|
'str' argument which is not in the hashed list). In this case,
|
|
fake the correct result and depend on caller's final check that
|
|
the entry is really the one that the caller wanted. */
|
|
if (offset >= n_entries)
|
|
offset = 0;
|
|
|
|
dirmap_offset = *((guint32*)memory);
|
|
table = (guint16*) (memory + dirmap_offset);
|
|
|
|
return table[offset];
|
|
}
|
|
|