Code Review
/
govpp.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
review
|
tree
raw
|
patch
| inline |
side by side
(parent:
e7cec91
)
Fix missing lock in LookupByID
31/11731/1
author
Ondrej Fabry
<ofabry@cisco.com>
Fri, 13 Apr 2018 03:33:03 +0000
(
05:33
+0200)
committer
Ondrej Fabry
<ofabry@cisco.com>
Fri, 13 Apr 2018 03:33:03 +0000
(
05:33
+0200)
Change-Id: I699052d179c98ad3a29218fa33fafef0affb97a6
Signed-off-by: Ondrej Fabry <ofabry@cisco.com>
core/request_handler.go
patch
|
blob
|
history
diff --git
a/core/request_handler.go
b/core/request_handler.go
index
7c185cd
..
27ff4fc
100644
(file)
--- a/
core/request_handler.go
+++ b/
core/request_handler.go
@@
-227,6
+227,9
@@
func (c *Connection) LookupByID(ID uint16) (string, error) {
return "", errors.New("nil connection passed in")
}
+ c.msgIDsLock.Lock()
+ defer c.msgIDsLock.Unlock()
+
for key, id := range c.msgIDs {
if id == ID {
return key, nil