2
61%
master: 62%

Ran 10 Oct 2017 10:08AM UTC

Files 168

Run time 7s

Badge

Embed ▾

Committed 10 Oct 2017 09:40AM UTC coverage: 59.168%. First build

Job # 760.2

Build Type

push

travis-ci

Committed by

jbremer

Commit Message

resolve race condition with report processing (thanks Bas van Sisseren)

When process_results was enabled the Scheduler would still log one
message after the task status had been set to TASK_REPORTED, potentially
logging to $CWD/storage/analyses/task_id/cuckoo.log while this file not
longer existed (as external processes may have already scooped the task).

When process_results was disabled the Scheduler would still log two
messages after setting TASK_COMPLETED, potentially writing to a closed
and removed file.

The above two situations would allow the final log.exception() in
AnalysisManager()'s run() method to throw an exception which, due to the
treading nature of this class, wasn't caught properly. Finally, the
exception would ignore the execution of task_log_stop() and decreasing
the active_analysis_count variable, which would hang Cuckoo forever in
the case of a "cuckoo -m 10000"-like situation.

This is now resolved thanks to Bas van Sisseren for identifying the root
cause and providing an initial patch to mitigate the issue.

Run Details

8516 of 14393 relevant lines covered (59.17%)

0.59 hits per line

jbremer / cuckoo / 760 / 2
61%
master: 62%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 760.2

jbremer / cuckoo / 760 / 2 61% master: 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 760.2

jbremer / cuckoo / 760 / 2
61%
master: 62%

README BADGES
x