Changeset 207986 in webkit

Timestamp:

Oct 27, 2016, 3:59:16 AM (10 years ago)

Author:

Carlos Garcia Campos

Message:

Merge r207842 - jsc.cpp is leaking memory allocated by readline in runInteractive
​https://bugs.webkit.org/show_bug.cgi?id=163958

According to ​http://web.mit.edu/gnu/doc/html/rlman_2.html,
"The line readline returns is allocated with malloc ();
you should free () the line when you are done with it."
The memory allocated by readline is not being freed when it should.

Patch by Christopher Reid <​Christopher.Reid@am.sony.com> on 2016-10-25
Reviewed by Mark Lam.

• jsc.cpp:

Location:

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• jsc.cpp (modified) (1 diff)

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2016-10-25  Christopher Reid  <Christopher.Reid@am.sony.com>
+
+        jsc.cpp is leaking memory allocated by readline in runInteractive
+        https://bugs.webkit.org/show_bug.cgi?id=163958
+
+        According to http://web.mit.edu/gnu/doc/html/rlman_2.html,
+        "The line readline returns is allocated with malloc ();
+        you should free () the line when you are done with it."
+        The memory allocated by readline is not being freed when it should.
+
+        Reviewed by Mark Lam.
+
+        * jsc.cpp:
+
 2016-10-20  Zan Dobersek  <zdobersek@igalia.com>
 

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/jsc.cpp

@@ -2248 +2248 @@
             source = source + '\n';
             checkSyntax(globalObject->vm(), makeSource(source, interpreterName), error);
-            if (!line[0])
+            if (!line[0]) {
+                free(line);
                 break;
+            }
             add_history(line);
+            free(line);
         } while (error.syntaxErrorType() == ParserError::SyntaxErrorRecoverable);