Блок synchronized

[Sumy-ua]

Есть статья http://www.ibm.com/developerworks/ru...ry/j-jtp09238/
Листинг 5 в англоязычной версии статьи

public void updateHighScore(PlayerScore newScore) {
     ServletContext ctx = getServletConfig().getServletContext();
     synchronized (lock) {
         PlayerScore hs = (PlayerScore) ctx.getAttribute("highScore");
         if (newScore.score > hs.score)
             ctx.setAttribute("highScore", newScore);
     }
}

Листинг 5 в русскоязычной версии статьи

public void updateHighScore(PlayerScore newScore) {
    ServletContext ctx = getServletConfig().getServletContext();
    PlayerScore hs = (PlayerScore) ctx.getAttribute("highScore");
    synchronized (lock) {
        if (newScore.score > hs.score) 
            ctx.setAttribute("highScore", newScore);
    }
}

Есть ли здесь ошибка? Или оба листинга правильные?

[Stilet]

Не вижу тут ошибок, хотя все зависит от тела getAttribute()