From 1c4cfe3e200b5a2329191886865d1dacdb79e4e9 Mon Sep 17 00:00:00 2001
From: Robert Fancsik <frobert@inf.u-szeged.hu>
Date: Thu, 5 Sep 2019 01:42:16 +0200
Subject: [PATCH] Property release the callback result in
 Array.prototype.[find/findIndex] (#3052)

This patch fixes #3049.

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu
---
 .../ecma-builtin-array-prototype.c               |  8 ++++++--
 tests/jerry/es2015/regression-test-issue-3049.js | 16 ++++++++++++++++
 2 files changed, 22 insertions(+), 2 deletions(-)
 create mode 100644 tests/jerry/es2015/regression-test-issue-3049.js

diff --git a/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c b/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c
index 267e63c03..970455b29 100644
--- a/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c
+++ b/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c
@@ -1992,11 +1992,16 @@ ecma_builtin_array_prototype_object_find (ecma_value_t predicate, /**< callback
         return call_value;
       }
 
-      if (ecma_op_to_boolean (call_value))
+      bool call_value_to_bool = ecma_op_to_boolean (call_value);
+
+      ecma_free_value (call_value);
+
+      if (call_value_to_bool)
       {
         /* 8.f */
         if (is_find)
         {
+          ecma_free_value (current_index);
           return get_value;
         }
 
@@ -2004,7 +2009,6 @@ ecma_builtin_array_prototype_object_find (ecma_value_t predicate, /**< callback
         return current_index;
       }
 
-      ecma_free_value (call_value);
       ecma_free_value (get_value);
       ecma_free_value (current_index);
     }
diff --git a/tests/jerry/es2015/regression-test-issue-3049.js b/tests/jerry/es2015/regression-test-issue-3049.js
new file mode 100644
index 000000000..25002b790
--- /dev/null
+++ b/tests/jerry/es2015/regression-test-issue-3049.js
@@ -0,0 +1,16 @@
+// Copyright JS Foundation and other contributors, http://js.foundation
+//
+// 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.
+
+var func = function (a, b) { return a + b };
+assert(['A'].findIndex(func) == 0);