archive-gh-me/jerryscript
1
0
Fork 0
mirror of https://github.com/jerryscript-project/jerryscript.git synced 2025-12-15 16:29:21 +00:00
Code Issues Projects Releases Wiki Activity

bugfix: jerry_get_object_native_pointer: accept NULL for out_native_info_p (#1770)

Browse Source 
With the jerry_set_object_native_pointer() the native_info_p is optional, so I think it makes sense that it is optional for the getter as well.

JerryScript-DCO-1.0-Signed-off-by: Martijn The martijn.the@intel.com
This commit is contained in:
Martijn Thé 2017-04-24 13:52:00 +02:00 committed by László Langó
parent 07c86ece54
commit 1f6b396ebb
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
jerry-core/jerry.c
View File

@ -2032,7 +2032,12 @@ jerry_get_object_native_pointer (const jerry_value_t obj_val, /**< object to get
}
*out_native_pointer_p = native_pointer_p->data_p;
*out_native_info_p = (const jerry_object_native_info_t *) native_pointer_p->info_p;
if (out_native_info_p)
{
*out_native_info_p = (const jerry_object_native_info_t *) native_pointer_p->info_p;
}
return true;
} /* jerry_get_object_native_pointer */

5
tests/unit-core/test-api.c
View File

@ -761,6 +761,11 @@ main (void)
&& (uintptr_t) ptr == (uintptr_t) 0x0012345678abcdefull
&& out_info_p == &JERRY_NATIVE_HANDLE_INFO_FOR_CTYPE (bind2));
/* Passing NULL for out_info_p is allowed. */
is_ok = jerry_get_object_native_pointer (res, &ptr, NULL);
TEST_ASSERT (is_ok
&& (uintptr_t) ptr == (uintptr_t) 0x0012345678abcdefull);
jerry_release_value (res);
/* Test: Throwing exception from native handler. */