Skip to content

Commit

Permalink
Fix lexing of illegal exponential.
Browse files Browse the repository at this point in the history
Related issue: jerryscript-project#339

JerryScript-DCO-1.0-Signed-off-by: Ilyong Cho ily.cho@samsung.com
  • Loading branch information
ILyoan committed Oct 23, 2015
1 parent 448984d commit 31ae097
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 0 deletions.
7 changes: 7 additions & 0 deletions jerry-core/parser/js/lexer.cpp
Expand Up @@ -969,6 +969,13 @@ lexer_parse_number (void)
consume_char ();
}

if (!lit_char_is_decimal_digit (LA (0)))
{
PARSE_ERROR (JSP_EARLY_ERROR_SYNTAX,
"Exponential mark in a numeric literal should followed by a singed interger",
lit_utf8_iterator_get_pos (&src_iter));
}

continue;
}
}
Expand Down
24 changes: 24 additions & 0 deletions tests/jerry/regression-test-issue-339.js
@@ -0,0 +1,24 @@
// Copyright 2015 Samsung Electronics Co., Ltd.
// Copyright 2015 University of Szeged.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

try
{
eval ('var y = 4e;');
assert (false);
}
catch (e)
{
assert (e instanceof SyntaxError);
}

0 comments on commit 31ae097

Please sign in to comment.