mTraits is used without been locked

mTraits could be read/written by more than one thread,
And it will cause some timing issue if there is no synchronization
Solution:
use getId() instead of mTraits->id when use its id.

Bug: 142629712
Test: This is seldom timing issue, and may be reproduced by monkey test (enable uasan)
Change-Id: I96870027fd171fea15957b292c4898043262d9cf