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属性造成的。

 

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

查看 复制到剪贴板 打印

1. [com.trs.core.frame.exceptions.handlers.ExceptionHandler] [ERROR] 2013-12-24 01:03:42 : No modifications are allowed to a locked ParameterMap   
2. java.lang.IllegalStateException: No modifications are allowed to a locked ParameterMap  
3.     at org.apache.catalina.util.ParameterMap.put(ParameterMap.java:166)  
4.     at org.apache.catalina.connector.Request.getParameterMap(Request.java:1112)  
5.     at org.apache.catalina.connector.RequestFacade.getParameterMap(RequestFacade.java:414)  
6.     at javax.servlet.ServletRequestWrapper.getParameterMap(ServletRequestWrapper.java:166)  

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

 

分析源代码

查看 复制到剪贴板 打印

1. public class ParamFilter implements Filter {  
2.      
3.     //省略部分代码  
4.           
5.     private ParamFilterWrapper wrapper;  
6.       
7.     public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException {  
8.         wrapper = new ParamFilterWrapper((HttpServletRequest) request);  
9.         chain.doFilter(wrapper, response);  
10.     }   
11. }  

发现没有，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规范[翻译] 》。