Kingfisher: Pointer double free issue with ImagePrefetcher

[kukushi]

Check List

Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

• [x] I have read the wiki page and cheat sheet, but there is no information I need.
• [x] I have searched in existing issues, but did not find a same one.
• [x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description

What

Crashed when prefetching images in a row.

Reproduce

You can test it on the latest Kingfisher codebase in ImagePrefetcherTests.swift.

func testPrefetchMultiTimes() {
        testURLs.forEach { stub($0, data: testImageData) }

        for _ in 0..<10000 {
            let prefetcher = ImagePrefetcher(resources: testURLs)
            prefetcher.start()
        }
    }

You may need to run it serveral times before the crashed happened.

Other comment

[kukushi]

Each prefetcher instance runs on an independent serial queue, while they are using the same image downloader. When downloading images, the downloader concurrently overwrites the same task's onTaskDone delegate's block, which cause a double free issue.

[onevcat]

Thanks for reporting this. I will check it as soon as I can.

[onevcat]

Should be fixed by making all prefetcher use the same queue.

[kukushi]

Will the new version release in coming days?

[onevcat]

5.1.1 was released.