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规范[翻译] 》。
已有 2 人发表留言,猛击->> 这里<<-参与讨论
ITeye推荐