Shouldn't this be volatile ?