代码检查工具检查出来的线程安全问题

Question

有如下代码

Model sessionModel = request.getSession().getAttribute("model");

Model userModel = new Model();

BeanUtils.copyProperties(userModel, sessionModel);

if (sessionModel != null) {
  userModel.setName("tony");

  request.getSession().setArrtibute("model", userModel);
}

以上代码有什么问题吗？
用安全检查工具，出现警告：线程共享对象 被reuqest.getSession 用setAttribute设置后，更新失败可能

怎么修改好呢？

Answer 1

这里有 get 和 set 操作同一个对象，而 web 应用本身就是多线程环境，把这端代码用同步锁保护起来。内置锁方式：

synchronized(xxx.class){
Model sessionModel = request.getSession().getAttribute("model");

Model userModel = new Model();

BeanUtils.copyProperties(userModel, sessionModel);

if (sessionModel != null) {
  userModel.setName("tony");

  request.getSession().setArrtibute("model", userModel);

}