YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix argument boundary check order during number format conversion (#4031)

Browse Source 
Also added some tests related to the issues caused by the wrong order

JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi aszilagy@inf.u-szeged.hu
This commit is contained in:
Szilagyi Adam
2020-07-24 15:55:12 +02:00
committed by GitHub
parent d39a076b2e
commit da5b058dec
2 changed files with 44 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/ecma/builtin-objects/ecma-builtin-number-prototype.c
+16 -7
View File
@@ -493,17 +493,12 @@ ecma_builtin_number_prototype_object_to_number_convert (ecma_number_t this_num,
return to_integer;
}
/* Argument boundary checks */
if (mode != NUMBER_ROUTINE_TO_PRECISION
/* Argument boundary check for toFixed method */
if (mode == NUMBER_ROUTINE_TO_FIXED
&& (arg_num <= -1 || arg_num >= 101))
{
return ecma_raise_range_error (ECMA_ERR_MSG ("Fraction digits must be between 0 and 100."));
}
else if (mode == NUMBER_ROUTINE_TO_PRECISION
&& (arg_num < 1 || arg_num > 100))
{
return ecma_raise_range_error (ECMA_ERR_MSG ("Precision digits must be between 1 and 100."));
}
/* Handle NaN separately */
if (ecma_number_is_nan (this_num))
@@ -566,6 +561,20 @@ ecma_builtin_number_prototype_object_to_number_convert (ecma_number_t this_num,
return ecma_make_string_value (ecma_stringbuilder_finalize (&builder));
}
/* Argument boundary check for toExponential and toPrecision methods */
if (mode == NUMBER_ROUTINE_TO_EXPONENTIAL
&& (arg_num <= -1 || arg_num >= 101))
{
ecma_stringbuilder_destroy (&builder);
return ecma_raise_range_error (ECMA_ERR_MSG ("Fraction digits must be between 0 and 100."));
}
else if (mode == NUMBER_ROUTINE_TO_PRECISION
&& (arg_num < 1 || arg_num > 100))
{
ecma_stringbuilder_destroy (&builder);
return ecma_raise_range_error (ECMA_ERR_MSG ("Precision digits must be between 1 and 100."));
}
num_of_digits = ecma_number_to_decimal (this_num, digits, &exponent);
/* Handle undefined argument */