You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
gitea/services/repository
zeripath 39a0db6ecf
Prevent dangling archiver goroutine (#19516) (#19526)
Backport #19516

Within doArchive there is a service goroutine that performs the
archiving function.  This goroutine reports its error using a `chan
error` called `done`. Prior to this PR this channel had 0 capacity
meaning that the goroutine would block until the `done` channel was
cleared - however there are a couple of ways in which this channel might
not be read.

The simplest solution is to add a single space of capacity to the
goroutine which will mean that the goroutine will always complete and
even if the `done` channel is not read it will be simply garbage
collected away.

(The PR also contains two other places when setting up the indexers
which do not leak but where the blocking of the sending goroutine is
also unnecessary and so we should just add a small amount of capacity
and let the sending goroutine complete as soon as it can.)

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: 6543 <6543@obermui.de>

Co-authored-by: 6543 <6543@obermui.de>
3 years ago
..
archiver Prevent dangling archiver goroutine (#19516) (#19526) 3 years ago
files Stop trimming preceding and suffixing spaces from editor filenames (#18334) 3 years ago
adopt.go Use IterateBufferSize whilst querying repositories during adoption check (#19140) (#19160) 3 years ago
adopt_test.go services/repository: fix ListUnadoptedRepositories incorrect total count (#17865) 3 years ago
avatar.go Some repository refactors (#17950) 3 years ago
avatar_test.go Some repository refactors (#17950) 3 years ago
branch.go Use full output of git show-ref --tags to get tags for PushUpdateAddTag (#19235) (#19236) 3 years ago
cache.go Move repository model into models/repo (#17933) 3 years ago
check.go Move repository model into models/repo (#17933) 3 years ago
fork.go Fix forked repositories missed tags (#18719) (#18735) 3 years ago
fork_test.go Some repository refactors (#17950) 3 years ago
hooks.go Some repository refactors (#17950) 3 years ago
main_test.go Decouple unit test code from business code (#17623) 3 years ago
push.go Fix compare link in active feeds for new branch (#19149) (#19185) 3 years ago
repository.go Move keys to models/asymkey (#17917) 3 years ago
template.go Some repository refactors (#17950) 3 years ago
transfer.go Fix rename notification bug (#18008) 3 years ago
transfer_test.go Move repository model into models/repo (#17933) 3 years ago