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