Fix for fileContents = nil

denyskon · denyskon · commit 648a2a414a5b · 2025-04-18T20:11:26.000+02:00
diff --git a/services/repository/files/file.go b/services/repository/files/file.go
@@ -23,7 +23,7 @@ func GetContentsListFromTrees(ctx context.Context, repo *repo_model.Repository,
 	var size int64
 	for _, file := range treeNames {
 		fileContents, _ := GetContents(ctx, repo, file, branch, false) // ok if fails, then will be nil
-		if *fileContents.Content != "" {
+		if fileContents != nil && *fileContents.Content != "" {
 			size += fileContents.Size // if content isn't empty (e. g. due to the single blob being too large), add file size to response size
 		}
 		if size > setting.API.DefaultMaxResponseSize {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ func GetContentsListFromTrees(ctx context.Context, repo *repo_model.Repository,`
`23`	`23`	`var size int64`
`24`	`24`	`for _, file := range treeNames {`
`25`	`25`	`fileContents, _ := GetContents(ctx, repo, file, branch, false) // ok if fails, then will be nil`
`26`		`- if *fileContents.Content != "" {`
	`26`	`+ if fileContents != nil && *fileContents.Content != "" {`
`27`	`27`	`size += fileContents.Size // if content isn't empty (e. g. due to the single blob being too large), add file size to response size`
`28`	`28`	`}`
`29`	`29`	`if size > setting.API.DefaultMaxResponseSize {`