Skip to content

Commit

Permalink
Merge pull request #1258 from ethantkoenig/fix/access
Browse files Browse the repository at this point in the history
Fix nil-dereference bug
  • Loading branch information
andreynering authored Mar 16, 2017
2 parents efbb895 + 17f403f commit 43c5469
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion modules/context/repo.go
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,11 @@ func RepoAssignment(args ...bool) macaron.Handler {
if ctx.IsSigned && ctx.User.IsAdmin {
ctx.Repo.AccessMode = models.AccessModeOwner
} else {
mode, err := models.AccessLevel(ctx.User.ID, repo)
var userID int64
if ctx.User != nil {
userID = ctx.User.ID
}
mode, err := models.AccessLevel(userID, repo)
if err != nil {
ctx.Handle(500, "AccessLevel", err)
return
Expand Down

0 comments on commit 43c5469

Please sign in to comment.