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

请先前往《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规范[翻译] 》。

 

评论

3 楼 jiasanshou 2014-03-04  

第9行执行的参数 wrapper 有可能是其他线程执行第8行后的值。
chain.doFilter(new ParamFilterWrapper((HttpServletRequest) request), response); 

2 楼 jinnianshilongnian 2013-12-30  

ss135ss 写道

顶，非常感谢开涛兄帮忙解决问题，我们遇到的就是这个问题。 :    D

可以看看翻译的servlet规范

1 楼 ss135ss 2013-12-30  

顶，非常感谢开涛兄帮忙解决问题，我们遇到的就是这个问题。 :    D