Bug 52064

Summary:	OutOfMemory Risk in CacheManager
Product:	JMeter - Now in Github	Reporter:	Philippe Mouawad <p.mouawad>
Component:	HTTP	Assignee:	JMeter issues mailing list <issues>
Status:	RESOLVED FIXED
Severity:	major	CC:	p.mouawad
Priority:	P2
Version:	2.5.1
Target Milestone:	---
Hardware:	All
OS:	All
Attachments:	Test plan to reproduce issue Memory behaviour before fix Memory behaviour after fix

Description Philippe Mouawad 2011-10-20 11:12:05 UTC

Hello,
I am doing some tests on an application that uses GET requests with a parameter nocache generated for each request.
This application generates this request every second.

So I simulated this and ran an endurance test.

I noticed JMeter memory increased during the test.
I made a Heap Dump and saw that CacheManager was overwhelmed by this request.

There is of course an issue with WebServer config that should add Expires Header but I think we should protect JMeter from this by limiting size on Cachemanager HashMap,
as it is the case for a real browser that has a limited memory size for cache.

Comment 1 Philippe Mouawad 2011-10-20 11:12:29 UTC

Concerns nightly at 20 october 2011

Comment 2 Philippe Mouawad 2011-10-20 19:08:06 UTC

Created attachment 27829 [details]
Test plan to reproduce issue

Add a static index.html file in Apache Tomcat webapps/ROOT folder.

Comment 3 Philippe Mouawad 2011-10-20 19:08:54 UTC

Created attachment 27830 [details]
Memory behaviour before fix

Comment 4 Philippe Mouawad 2011-10-20 19:09:19 UTC

Created attachment 27831 [details]
Memory behaviour after fix

Comment 5 Philippe Mouawad 2011-10-20 19:09:38 UTC

Date: Thu Oct 20 19:07:13 2011
New Revision: 1186994

URL: http://svn.apache.org/viewvc?rev=1186994&view=rev
Log:
Bug 52064 - OutOfMemory Risk in CacheManager

Modified:
   jakarta/jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties
   jakarta/jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties
   jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java
   jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/gui/CacheManagerGui.java
   jakarta/jmeter/trunk/xdocs/changes.xml

Comment 6 The ASF infrastructure team 2022-09-24 20:37:47 UTC

This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2615