Skip to content

Commit dfafb1a

Browse files
dbatyairerobika
authored andcommitted
Fix unhandled exceptions with unicode error messages (#2994)
Fixes #2993 JerryScript-DCO-1.0-Signed-off-by: Dániel Bátyai [email protected]
1 parent a2d242e commit dfafb1a

File tree

7 files changed

+28
-13
lines changed

7 files changed

+28
-13
lines changed

jerry-main/main-unix-snapshot.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
172172
return;
173173
}
174174

175-
jerry_size_t err_str_size = jerry_get_string_size (err_str_val);
175+
jerry_size_t err_str_size = jerry_get_utf8_string_size (err_str_val);
176176

177177
if (err_str_size >= 256)
178178
{
@@ -182,7 +182,7 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
182182
}
183183

184184
jerry_char_t err_str_buf[256];
185-
jerry_size_t string_end = jerry_string_to_char_buffer (err_str_val, err_str_buf, err_str_size);
185+
jerry_size_t string_end = jerry_string_to_utf8_char_buffer (err_str_val, err_str_buf, err_str_size);
186186
assert (string_end == err_str_size);
187187
err_str_buf[string_end] = 0;
188188

jerry-main/main-unix.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,15 +118,15 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
118118
if (!jerry_value_is_error (item_val)
119119
&& jerry_value_is_string (item_val))
120120
{
121-
jerry_size_t str_size = jerry_get_string_size (item_val);
121+
jerry_size_t str_size = jerry_get_utf8_string_size (item_val);
122122

123123
if (str_size >= 256)
124124
{
125125
printf ("%3u: [Backtrace string too long]\n", i);
126126
}
127127
else
128128
{
129-
jerry_size_t string_end = jerry_string_to_char_buffer (item_val, err_str_buf, str_size);
129+
jerry_size_t string_end = jerry_string_to_utf8_char_buffer (item_val, err_str_buf, str_size);
130130
assert (string_end == str_size);
131131
err_str_buf[string_end] = 0;
132132

@@ -141,7 +141,7 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
141141
}
142142

143143
jerry_value_t err_str_val = jerry_value_to_string (error_value);
144-
jerry_size_t err_str_size = jerry_get_string_size (err_str_val);
144+
jerry_size_t err_str_size = jerry_get_utf8_string_size (err_str_val);
145145

146146
if (err_str_size >= 256)
147147
{

targets/curie_bsp/jerry_app/quark/main.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ void jerry_resolve_error (jerry_value_t ret_value)
4848
{
4949
ret_value = jerry_get_value_from_error (ret_value, true);
5050
jerry_value_t err_str_val = jerry_value_to_string (ret_value);
51-
jerry_size_t err_str_size = jerry_get_string_size (err_str_val);
51+
jerry_size_t err_str_size = jerry_get_utf8_string_size (err_str_val);
5252
jerry_char_t *err_str_buf = (jerry_char_t *) balloc (err_str_size, NULL);
53-
jerry_size_t sz = jerry_string_to_char_buffer (err_str_val, err_str_buf, err_str_size);
53+
jerry_size_t sz = jerry_string_to_utf8_char_buffer (err_str_val, err_str_buf, err_str_size);
5454
err_str_buf[sz] = 0;
5555
printk ("Script Error: unhandled exception: %s\n", err_str_buf);
5656
bfree(err_str_buf);

targets/esp8266/user/jerry_extapi.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ DELCARE_HANDLER(print) {
5555
{
5656
if (jerry_value_is_string (args_p[cc]))
5757
{
58-
jerry_size_t size = jerry_get_string_size (args_p[0]);
58+
jerry_size_t size = jerry_get_utf8_string_size (args_p[0]);
5959
char *buffer;
6060
buffer = (char *) malloc(size + 1);
6161

@@ -66,9 +66,9 @@ DELCARE_HANDLER(print) {
6666
continue;
6767
}
6868

69-
jerry_string_to_char_buffer (args_p[cc],
70-
(jerry_char_t *) buffer,
71-
size);
69+
jerry_string_to_utf8_char_buffer (args_p[cc],
70+
(jerry_char_t *) buffer,
71+
size);
7272
*(buffer + size) = 0;
7373
printf("%s ", buffer);
7474
free (buffer);

targets/nuttx-stm32f4/jerry_main.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
160160

161161
error_value = jerry_get_value_from_error (error_value, false);
162162
jerry_value_t err_str_val = jerry_value_to_string (error_value);
163-
jerry_size_t err_str_size = jerry_get_string_size (err_str_val);
163+
jerry_size_t err_str_size = jerry_get_utf8_string_size (err_str_val);
164164
jerry_char_t err_str_buf[256];
165165

166166
jerry_release_value (error_value);

targets/tizenrt-artik053/apps/jerryscript/jerry_main.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ print_unhandled_exception (jerry_value_t error_value) /**< error value */
132132

133133
error_value = jerry_get_value_from_error (error_value, false);
134134
jerry_value_t err_str_val = jerry_value_to_string (error_value);
135-
jerry_size_t err_str_size = jerry_get_string_size (err_str_val);
135+
jerry_size_t err_str_size = jerry_get_utf8_string_size (err_str_val);
136136
jerry_char_t err_str_buf[256];
137137

138138
jerry_release_value (error_value);
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
// Copyright JS Foundation and other contributors, http://js.foundation
2+
//
3+
// Licensed under the Apache License, Version 2.0 (the "License");
4+
// you may not use this file except in compliance with the License.
5+
// You may obtain a copy of the License at
6+
//
7+
// http://www.apache.org/licenses/LICENSE-2.0
8+
//
9+
// Unless required by applicable law or agreed to in writing, software
10+
// distributed under the License is distributed on an "AS IS" BASIS
11+
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12+
// See the License for the specific language governing permissions and
13+
// limitations under the License.
14+
15+
throw new SyntaxError("𐐀");

0 commit comments

Comments
 (0)