A web server generally only creates one instance of servlet or JSP class (i.e., treats the class as a Singleton), and will have multiple threads invoke methods on that instance to service multiple simultaneous requests.

Bad Practice

class MyServlet extends HttpServlet {

    private HashMap<String, User> users; // This field may be left open to concurrent modification.

    // ...

    @Override
    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
        resp.setStatus(200);
        resp.setHeader("Content-Type", "application/json");

        String name = req.getParameter("name");

        users.put(name, ...); // This access is not synchronized and could result in concurrent modification of users.
    }
}

Accessing such variables without synchronizing on them could allow ConcurrentModificationExceptions. This could also result in race conditions occurring between threads that modify the concerned field.

Recommended

Consider using some form of synchronization to ensure that such variables can be accessed safely in a concurrent context.

private synchronized doOperationOnUsers(String name) {
        // users is only modified within this method.
 

        users.put(name, ...);
    }

References

SpotBugs - MSF_MUTABLE_SERVLET_FIELD
CWE-362 - Concurrent Execution using Shared Resource with Improper Synchronization ('Race Condition')

4k4xs4pH1r3 / artemisa

No ignore rules found