Commit 3474cd8
test: verify napi_remove_wrap with napi_delete_reference
Verify that napi_remove_wrap and napi_delete_reference should be safe
to be called consecutively on the in-out params of napi_wrap.
PR-URL: #44754
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>1 parent ce085d7 commit 3474cd8Copy full SHA for 3474cd8
File tree
Expand file treeCollapse file tree
2 files changed
+36
-0
lines changedOpen diff view settings
Filter options
- test/js-native-api/test_reference_double_free
Expand file treeCollapse file tree
2 files changed
+36
-0
lines changedOpen diff view settings
Collapse file
test/js-native-api/test_reference_double_free/test_reference_double_free.c
Copy file name to clipboardExpand all lines: test/js-native-api/test_reference_double_free/test_reference_double_free.c+26Lines changed: 26 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
42 | 42 | |
43 | 43 | |
44 | 44 | |
| 45 | + |
| 46 | + |
| 47 | + |
| 48 | + |
| 49 | + |
| 50 | + |
| 51 | + |
| 52 | + |
| 53 | + |
| 54 | + |
| 55 | + |
| 56 | + |
| 57 | + |
| 58 | + |
| 59 | + |
| 60 | + |
| 61 | + |
| 62 | + |
| 63 | + |
45 | 64 | |
46 | 65 | |
47 | 66 | |
| ||
50 | 69 | |
51 | 70 | |
52 | 71 | |
| 72 | + |
| 73 | + |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | + |
| 78 | + |
53 | 79 | |
54 | 80 | |
55 | 81 | |
Collapse file
test/js-native-api/test_reference_double_free/test_wrap.js
Copy file name to clipboard+10Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
0 commit comments