[#CONF-9521] TinyMceServlet has a memory leak - Atlassian JIRA

New and Improved 3.13 Beta. Highlights: Shareable filters and dashboards and lots of other goodies. Any feedback can be raised as JIRA issues in the JIRA project.

Issue Details (XML | Word | Printable)

Key:	CONF-9521
Type:	Bug
Status:	Closed
Resolution:	Fixed
Priority:	Critical
Assignee:	Unassigned
Reporter:	Don Willis [Atlassian]
Votes:	0
Watchers:	1

Operations

If you were logged in you would be able to see more operations.

Confluence

TinyMceServlet has a memory leak

Created: 20/Sep/07 03:00 AM Updated: 11/Oct/07 04:04 AM

Component/s:

Performance, WYSIWYG editing

Affects Version/s:

2.5.5

Fix Version/s:

2.5.8, 2.6.0

Time Tracking:

Not Specified

File Attachments:	1. atlassian-tinymce-plugin-2.2.1.jar (198 kB)
Environment:	Confluence using the Tiny MCE Editor Plugin version 2.2

Participants:	Don Willis [Atlassian] and Matt Ryall [Atlassian]
Since last comment:	46 weeks, 1 day ago
Internal Complexity:	7
Resolution Date:	11/Oct/07 04:04 AM
Internal Value:	9
Labels:

Description

The TinyMce rich text editor plugin bundled with Confluence has a Memory leak. The TinyMceConfiguration object we use as a key in our javascript cache has a broken hashcode method that allows duplicate configurations to be added. This causes a Memory leak of approximately 300k per edit request on average.

This can eventually lead to an OutOfMemoryError.

It was introduced with the fix for ~~CONF-6987~~ in Confluence 2.5.5

All

Comments

Work Log

Change History

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Matt Ryall [Atlassian] added a comment - 20/Sep/07 03:02 AM

When was this bug introduced? Could be important to note.

[ Permlink | « Hide ]

Don Willis [Atlassian] added a comment - 24/Sep/07 11:19 PM

Fixed in 2.6 branch by using 2.2.1 of tinyMCE, see http://svn.atlassian.com/fisheye/changelog/public/contrib/confluence/atlassian-tinymce-plugin?cs=10403 but not yet in trunk.

[ Permlink | « Hide ]

Don Willis [Atlassian] added a comment - 11/Oct/07 04:01 AM

This should have the new plugin version attached. I can't attach the file without reopening.

[ Permlink | « Hide ]

Don Willis [Atlassian] added a comment - 11/Oct/07 04:03 AM

Uploading this version of the tinyMCE plugin to Confluence 2.5.5 -> 2.5.7 should stop the memory leak.

[ Permlink | « Hide ]

Don Willis [Atlassian] added a comment - 11/Oct/07 04:04 AM

The patch is now attached. I can reclose the issue.