Changeset 152631 in webkit

Timestamp:

Jul 15, 2013 6:24:25 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[ATK] Leak: Leak in WebKitAccessibleTextGetText
​https://bugs.webkit.org/show_bug.cgi?id=118596

Patch by Brian Holt <​brian.holt@samsung.com> on 2013-07-15
Reviewed by Carlos Garcia Campos.

Fixed memory leak.

• accessibility/atk/WebKitAccessibleInterfaceText.cpp:

(getGailTextUtilForAtk):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• accessibility/atk/WebKitAccessibleInterfaceText.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-07-15  Brian Holt  <brian.holt@samsung.com>
+
+        [ATK] Leak: Leak in WebKitAccessibleTextGetText
+        https://bugs.webkit.org/show_bug.cgi?id=118596
+
+        Reviewed by Carlos Garcia Campos.
+
+        Fixed memory leak.
+
+        * accessibility/atk/WebKitAccessibleInterfaceText.cpp:
+        (getGailTextUtilForAtk):
+
 2013-07-14  Jon Lee  <jonlee@apple.com>
 

trunk/Source/WebCore/accessibility/atk/WebKitAccessibleInterfaceText.cpp

@@ -169 +169 @@
 {
     GailTextUtil* gailTextUtil = gail_text_util_new();
-    gail_text_util_text_setup(gailTextUtil, webkitAccessibleTextGetText(textObject, 0, -1));
+    GOwnPtr<char> text(webkitAccessibleTextGetText(textObject, 0, -1));
+    gail_text_util_text_setup(gailTextUtil, text.get());
     return gailTextUtil;
 }