mirrors/etcd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #5696 from xiang90/fix_panic

Browse Source 
etcdserver: fix panic when getting header of raft request
This commit is contained in:
Xiang Li
2016-06-16 13:58:50 -07:00
committed by GitHub
parent b3558894f2 adff458895
commit 2d08e093c1
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
etcdserver/apply_auth.go
View File

@ -30,7 +30,12 @@ func newAuthApplierV3(as auth.AuthStore, base applierV3) *authApplierV3 {
} }
func (aa *authApplierV3) Apply(r *pb.InternalRaftRequest) *applyResult { func (aa *authApplierV3) Apply(r *pb.InternalRaftRequest) *applyResult {
user := r.Header.Username var user string
if r.Header != nil {
// backward-compatible with pre-3.0 releases when internalRaftRequest
// does not have header field
user = r.Header.Username
}
if needAdminPermission(r) && !aa.as.IsAdminPermitted(user) { if needAdminPermission(r) && !aa.as.IsAdminPermitted(user) {
return &applyResult{err: auth.ErrPermissionDenied} return &applyResult{err: auth.ErrPermissionDenied}
} }