jbremer / cuckoo

61%

master: 62%

LAST BUILD ON BRANCH scheduler

branch: scheduler

CHANGE BRANCH

x

Reset

• scheduler
• 2.0.0
• 2.0.1a1
• 2.0.5.2
• 206
• 2061
• 2062
• 64bit0
• actual-tesseract-fix
• analyzer-unittest
• anonymous-reprocess
• api-unic
• cfg1
• cleanup-webif
• compat1
• config-caching
• core-aborted
• core-submit
• critical-timeout
• cwd-migr
• cwddel
• dashb1
• dashboard
• data-init-file
• data-init-file2
• dde1
• dde2
• dist0
• dist1
• dist2
• dumpmem0
• dumpmem1
• eggh021
• elastic-mongo-abstract
• empty-reprocess
• ensure-archives
• esmerge
• expose0
• extend-task
• extr-wheel
• extracted
• extracted3
• feedback
• finalize-database-migration
• finalize-import
• fix-screenshots
• fixup-fileupload
• fixup-machine-app
• fixup-rooter
• global-db-instance
• high0
• ifconfig
• inetsim2
• init-binaries
• json-speedup
• kernel
• kernel2
• large-file-support
• lib-issue
• linux-analysis
• linux2
• lnk-support
• load-signatures
• logging-tweak
• macosx0
• macosx1
• macosx2
• macosx3
• macro
• magic
• magic0
• magic1
• magic64
• mainexc
• master
• merge-http
• merge-pcap
• misc-changes
• mongo-abstract
• mongo-connect
• mongo-testing
• monitor-fixes
• newdocs2
• no-tesseract-warning
• none-filetype
• obsolete-dirs
• package
• pcap1
• pcapwebif
• pdf-report
• pdf-update
• pdf0
• performance
• pidfile
• pipinst
• postsubmit-page
• ppid0
• pr-merge1
• preinstall
• procmem
• ps1trail
• rarfile
• rdp
• rdp-merge0
• recentfiles
• recentfiles2
• recentfiles3
• reinstall
• relative-webif
• releases
• rename
• replay
• replay1
• requests2.13
• reworkscheduler
• rooter1
• sdist
• setup.py-exceptions
• sflock-api
• sflock-upd
• simplify-packages
• startup-tweaks
• stricter-packages
• stuff1
• submit-urls
• test0
• things
• travis-macosx
• tweaks1
• uni0
• uni1
• unicode-filenames
• unicode-support
• unicwd
• unilog
• upd-deps
• upd-setuptools
• ux-fixes
• ux-merges
• view-tasks
• virtualbox-tweakings
• virtualbox-tweaks
• vm1
• vmrouting-tweak1
• volatility
• volatility-coverage
• vpn-routing
• wip-jbr
• yara-offsets
• yara360
• yara361
• yara362
• yaraload
• yaramatchstrings

Build # 760

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

8756 of 14393 relevant lines covered (60.84%)

1.2 hits per line