1
62%
master: 62%

Ran 06 Nov 2017 11:48PM UTC

Files 170

Run time 6s

Badge

Embed ▾

Committed 06 Nov 2017 11:39PM UTC coverage: 61.413% (+1.5%) from 59.874%

Job # 768.1

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

9048 of 14733 relevant lines covered (61.41%)

0.61 hits per line

jbremer / cuckoo / 768 / 1
62%
master: 62%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 768.1

jbremer / cuckoo / 768 / 1 62% master: 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 768.1

jbremer / cuckoo / 768 / 1
62%
master: 62%

README BADGES
x