models/issue: fix panic when clear labels (#5445)

3 files changed, 8 insertions, 3 deletions

diff --git a/gogs.go b/gogs.go
index a433ef24..9c3171d3 100644
--- a/gogs.go
+++ b/gogs.go
@@ -16,7 +16,7 @@ import (
 	"github.com/gogs/gogs/pkg/setting"
 )
 
-const APP_VER = "0.11.73.1202"
+const APP_VER = "0.11.74.1202"
 
 func init() {
 	setting.AppVer = APP_VER
diff --git a/models/issue.go b/models/issue.go
index de87080a..b81f384b 100644
--- a/models/issue.go
+++ b/models/issue.go
@@ -314,8 +314,13 @@ func (issue *Issue) clearLabels(e *xorm.Session) (err error) {
 		return fmt.Errorf("getLabels: %v", err)
 	}
 
+	// NOTE: issue.removeLabel slices issue.Labels, so we need to create another slice to be unaffected.
+	labels := make([]*Label, len(issue.Labels))
 	for i := range issue.Labels {
-		if err = issue.removeLabel(e, issue.Labels[i]); err != nil {
+		labels[i] = issue.Labels[i]
+	}
+	for i := range labels {
+		if err = issue.removeLabel(e, labels[i]); err != nil {
 			return fmt.Errorf("removeLabel: %v", err)
 		}
 	}
diff --git a/templates/.VERSION b/templates/.VERSION
index 8b3d065d..5aa190e4 100644
--- a/templates/.VERSION
+++ b/templates/.VERSION
@@ -1 +1 @@
-0.11.73.1202
+0.11.74.1202