| commit | 075e0741aa752994f9706c54273f29dbb9ab7f06 | [log] [tgz] |
|---|---|---|
| author | Damien Neil <dneil@google.com> | Thu Feb 27 12:33:08 2020 -0800 |
| committer | Damien Neil <dneil@google.com> | Thu Feb 27 20:50:32 2020 +0000 |
| tree | d18741b6bf3079f93e46ec985ffa782ebb92c627 | |
| parent | a786db3a49ac0ed6fdb304ce3b0045b26cbd8c67 [diff] [blame] |
internal/impl: fix non-atomic access of MessageState.mi MessageState's mi field must only be accessed via atomic operations. Rename the field to 'atomicMessageInfo' to make it incorrect access obvious. Fix one incorrect non-atomic access. Change-Id: If80343fb1b82186416f007ca5b340a4926b1cd3c Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/221419 Reviewed-by: Joe Tsai <joetsai@google.com>
diff --git a/internal/impl/message_reflect.go b/internal/impl/message_reflect.go index f5f7f2b..aac55ee 100644 --- a/internal/impl/message_reflect.go +++ b/internal/impl/message_reflect.go
@@ -256,7 +256,7 @@ pragma.DoNotCompare pragma.DoNotCopy - mi *MessageInfo + atomicMessageInfo *MessageInfo } type messageState MessageState