#1555 В r18202 заметил странное изменение, возможно следы давно поломанной функциональности.

[Anonymous]

Originally created by: a.rain...@gmail.com

Подозреваю, что что-то сломалось в [r18202]:

1937        -    void QueueManager::putDownload(Download* aDownload, bool finished, bool reportFinish) noexcept
1937    +    void QueueManager::putDownload(const string& p_path, Download* aDownload, bool p_is_finished, bool p_is_report_finish) noexcept

2109        -                                        moveFile(aDownload->getTempTarget(), aDownload->getPath());
2106    +                                        moveFile(aDownload->getTempTarget(), p_path);

а если код отключен это вообще странно ибо тогда, как минимум, временные расширения на период закачки .dctmp и поддержка папки для временных файлов отвалятся.

Помимо этого в коммите указаны проблемы в этом месте, связанные с падениями, так что с этой частью явно что-то не так.

Сознавайтесь кто и зачем там что либо правил ранее :)

[Anonymous]

Originally posted by: a.rain...@gmail.com

Просмотрел код внимательнее, занятно там всё
В начале функции уже куски этого Download вытаскивают:

void QueueManager::putDownload(const string& p_path, Download* aDownload, bool p_is_finished, bool p_is_report_finish) noexcept
{
    const HintedUser l_hintedUser = aDownload->getHintedUser(); // crash https://crash-server.com/DumpGroup.aspx?ClientID=ppa&DumpGroupID=155631
    UserPtr l_user = aDownload->getUser(); // Ññûëêó íåëüçÿ - íóæíî äåðæàòü þçåðà.

[Anonymous]

Originally posted by: a.rain...@gmail.com

Судя по второму списку дампов проблема одна и та же - вызов идёт из потока сокета, а дальше Download портится чем то.

Labels: Stability

[Anonymous]

Originally posted by: Pavel.Pimenov@gmail.com

Бага в другом где-то буду искать

Status: Unconfirmed