OOM分析——错误使用Servlet API导致内存溢出

http://sishuok.com/forum/blogPost/list/7933.html

转载于上面文章

我也遇到过类似的Servlet以非线程安全的方式进行操作的情况，情况是有个servlet上，用了实例变量，之后在并发量上来的时候，发现了用户偶尔会取道了其他用户的结果，后来把这个实例变量消除掉，给他挪到方法内部，问题解决了，这也就说明了在servlet里不建议用实例变量的情况，如下所述那样。

请先前往《Spring内存溢出问题》查看问题，大体问题就是突然间内存飙升，且CPU使用率非常高。

问题分析

通过内存dump分析发现内存中某个key会有几百万个，而且观察这些key会发现有时候是org.springframework.web.servlet.DispatcherServlet.LOCALE_RESOLVER，有时候又变成org.springframework.web.servlet.DispatcherServlet.THEME_RESOLVER，每次可能不一样。

这个key是springmvc放到request中的属性key，而且一个key不可能放到request多次啊，因为request attributes本身通过HashMap存储，且线程不安全；造成这个问题的能想到的就是线程绑定变量泄露/并发操作request。

但是问题通过观察应用，没有使用ThreadLocal模式，那么唯一可能就是并发操作request属性造成的。

另外，通过观察日志，发现如下异常：

查看复制到剪贴板打印

[com.trs.core.frame.exceptions.handlers.ExceptionHandler] [ERROR] 2013-12-24 01:03:42 : No modifications are allowed to a locked ParameterMap

java.lang.IllegalStateException: No modifications are allowed to a locked ParameterMap

at org.apache.catalina.util.ParameterMap.put(ParameterMap.java:166)

at org.apache.catalina.connector.Request.getParameterMap(Request.java:1112)

at org.apache.catalina.connector.RequestFacade.getParameterMap(RequestFacade.java:414)

at javax.servlet.ServletRequestWrapper.getParameterMap(ServletRequestWrapper.java:166)

抛出这个异常的唯一原因就是并发操作request造成的。更加肯定了是并发使用request造成的。

分析源代码

查看复制到剪贴板打印

public class ParamFilter implements Filter {

//省略部分代码

private ParamFilterWrapper wrapper;

public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException {

wrapper = new ParamFilterWrapper((HttpServletRequest) request);

chain.doFilter(wrapper, response);

}

}

发现没有，wrapper竟然是实例变量。

我们知道Servlet规范规定Servlet、Filter默认是单实例，即线程不安全的；假设我们有两个线程P1和P2，如果以如下方式访问：

P1 P2

01 wrapper=request1

02 wrapper=request2

03 doFilter(wrapper)

发现没有，P1线程使用了P2线程的request，因此就是多线程访问request了；很可能就会遇到如上问题。

大家还应该知道HashMap多线程操作时可能会遇到死循环问题，具体可以参考《疫苗：Java
HashMap的死循环》。

对于Servlet、Filter等请务必以线程安全的方式进行操作。对于开发人员来说，建议读读Servlet规范，中文版请参考之前翻译的《Servlet 3.1规范[翻译] 》