Check webhooks slice length before calling xorm (#20642) (#20768)

Fixes: #20641 Co-authored-by: Lauris BH <[email protected]> Co-authored-by: John Olheiser <[email protected]> Co-authored-by: Lunny Xiao <[email protected]> Co-authored-by: Gabriel Vasile <[email protected]>
go-gitea · Aug 12, 2022 · ac9792c · ac9792c
1 parent f7c874c
commit ac9792c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/models/webhook/webhook.go b/models/webhook/webhook.go
@@ -399,6 +399,10 @@ func CreateWebhook(ctx context.Context, w *Webhook) error {
 
 // CreateWebhooks creates multiple web hooks
 func CreateWebhooks(ctx context.Context, ws []*Webhook) error {
+	// xorm returns err "no element on slice when insert" for empty slices.
+	if len(ws) == 0 {
+		return nil
+	}
 	for i := 0; i < len(ws); i++ {
 		ws[i].Type = strings.TrimSpace(ws[i].Type)
 	}