フィールドは同期化しているので、おそらく null ではないと思われます。 null のフィールドを同期化すると NullPointerException がスローされるので、null チェックは無意味になります。 別のフィールドで同期化した方がよいです。