luxolus
/
open-nomad
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

fingerprinters accessing node information should be thread safe

This commit is contained in:
Chelsea Holland Komlo 2018-02-26 14:55:53 -05:00
parent 64badc3562
commit b7bcd0b59f
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
client/fingerprint_manager.go
View File

@ -100,15 +100,20 @@ func (fm *FingerprintManager) setupDrivers(drivers []string) error {
func (fm *FingerprintManager) fingerprint(name string, f fingerprint.Fingerprint) (bool, error) {
request := &cstructs.FingerprintRequest{Config: fm.getConfig(), Node: fm.node}
var response cstructs.FingerprintResponse
if err := f.Fingerprint(request, &response); err != nil {
fm.nodeLock.Lock()
err := f.Fingerprint(request, &response)
fm.nodeLock.Unlock()
if err != nil {
return false, err
}
fm.nodeLock.Lock()
if node := fm.updateNode(&response); node != nil {
fm.nodeLock.Lock()
fm.node = node
fm.nodeLock.Unlock()
}
fm.nodeLock.Unlock()
return response.Detected, nil
}