[seed] initialize and free jchar in binary string, Im still not sure this is the correct way to do string c
- From: Alan Knowles <alank src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [seed] initialize and free jchar in binary string, Im still not sure this is the correct way to do string c
- Date: Wed, 16 May 2012 16:44:24 +0000 (UTC)
commit 0cd15d095eac8b62bbafa3f8379a954b52d52cbc
Author: Alan Knowles <alan akbkhome com>
Date: Thu May 17 00:44:07 2012 +0800
initialize and free jchar in binary string, Im still not sure this is the correct way to do string creation, need to run some tests to ensure it really works
libseed/seed-types.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
---
diff --git a/libseed/seed-types.c b/libseed/seed-types.c
index f4eaaaa..ce0f4f8 100644
--- a/libseed/seed-types.c
+++ b/libseed/seed-types.c
@@ -962,7 +962,7 @@ seed_value_from_gi_argument_full (JSContextRef ctx,
g_base_info_unref ((GIBaseInfo *) array_type_info);
ret = seed_value_from_binary_string (ctx, arg->v_pointer, array_len, exception);
- // always free arg...
+ // always free arg... what about when we do not own it...?
g_free(arg->v_pointer);
return ret;
@@ -2356,7 +2356,7 @@ seed_value_from_binary_string (JSContextRef ctx,
return JSValueMakeNull (ctx);
}
- jchar = g_alloca (sizeof (JSChar) * n_bytes);
+ jchar = g_new0(JSChar, n_bytes);
for(i =0;i < n_bytes; i++)
{
jchar[i] = bytes[i];
@@ -2365,6 +2365,7 @@ seed_value_from_binary_string (JSContextRef ctx,
jsstr = JSStringCreateWithCharacters((const JSChar*)jchar, n_bytes);
valstr = JSValueMakeString (ctx, jsstr);
+ g_free(jchar);
JSStringRelease (jsstr);
return valstr;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]