loadRepoOwner must not attempt to use a nil action.Repo

singuliere · singuliere · commit 3fc19d85b103 · 2022-05-04T01:40:13.000+01:00
diff --git a/models/action_list.go b/models/action_list.go
@@ -80,6 +80,9 @@ func (actions ActionList) loadRepoOwner(e db.Engine, userMap map[int64]*user_mod
 	}
 
 	for _, action := range actions {
+    if action.Repo == nil {
+      continue
+    }
 		repoOwner, ok := userMap[action.Repo.OwnerID]
 		if !ok {
 			repoOwner, err = user_model.GetUserByID(action.Repo.OwnerID)

Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,9 @@ func (actions ActionList) loadRepoOwner(e db.Engine, userMap map[int64]*user_mod`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`for _, action := range actions {`
	`83`	`+ if action.Repo == nil {`
	`84`	`+ continue`
	`85`	`+ }`
`83`	`86`	`repoOwner, ok := userMap[action.Repo.OwnerID]`
`84`	`87`	`if !ok {`
`85`	`88`	`repoOwner, err = user_model.GetUserByID(action.Repo.OwnerID)`