status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-11 06:16:08 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #2912 from hashicorp/issue-2624

Browse Source 
Adds guard in session ACL check in case session is not found.
This commit is contained in:
James Phillips 2017-04-13 20:10:19 -07:00 committed by GitHub
commit da304f94ae
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
consul/session_endpoint.go
View File

@ -43,6 +43,9 @@ func (s *Session) Apply(args *structs.SessionRequest, reply *string) error {
state := s.srv.fsm.State() state := s.srv.fsm.State()
_, existing, err := state.SessionGet(nil, args.Session.ID) _, existing, err := state.SessionGet(nil, args.Session.ID)
if err != nil { if err != nil {
return fmt.Errorf("Session lookup failed: %v", err)
}
if existing == nil {
return fmt.Errorf("Unknown session %q", args.Session.ID) return fmt.Errorf("Unknown session %q", args.Session.ID)
} }
if !acl.SessionWrite(existing.Node) { if !acl.SessionWrite(existing.Node) {