Skip to content

Commit f18291d

Browse files
committed
Refactor analysis pipeline completion logic and add support for re-enabling analysis after an abort.
1 parent b35c577 commit f18291d

File tree

3 files changed

+2
-15
lines changed

3 files changed

+2
-15
lines changed

plugins/msvc_rtti/plugin.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ extern "C" {
8080
// Run rtti before debug info is applied.
8181
msvcMetaWorkflow->Insert("core.module.loadDebugInfo", "plugin.msvc.rttiAnalysis");
8282
// Run vft after functions have analyzed (so that the virtual functions have analyzed)
83-
msvcMetaWorkflow->Insert("core.module.notifyCompletion", "plugin.msvc.vftAnalysis");
83+
msvcMetaWorkflow->Insert("core.module.deleteUnusedAutoFunctions", "plugin.msvc.vftAnalysis");
8484
Workflow::RegisterWorkflow(msvcMetaWorkflow);
8585

8686
return true;

plugins/warp/src/plugin/workflow.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,6 @@ pub fn insert_workflow() {
9494
module_meta_workflow
9595
.register_activity(&matcher_activity)
9696
.unwrap();
97-
module_meta_workflow.insert("core.module.notifyCompletion", [MATCHER_ACTIVITY_NAME]);
97+
module_meta_workflow.insert("core.module.deleteUnusedAutoFunctions", [MATCHER_ACTIVITY_NAME]);
9898
module_meta_workflow.register().unwrap();
9999
}

python/workflow.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -650,13 +650,6 @@ def run(self):
650650
else:
651651
return json.loads(core.BNPostWorkflowRequestForBinaryView(self.handle, request))
652652

653-
def abort(self):
654-
request = json.dumps({"command": "abort"})
655-
if self.is_function_machine:
656-
return json.loads(core.BNPostWorkflowRequestForFunction(self.handle, request))
657-
else:
658-
return json.loads(core.BNPostWorkflowRequestForBinaryView(self.handle, request))
659-
660653
def halt(self):
661654
request = json.dumps({"command": "halt"})
662655
if self.is_function_machine:
@@ -768,7 +761,6 @@ class WorkflowMachineCLI(cmd.Cmd):
768761
"d": "dump",
769762
"c": "resume",
770763
"r": "run",
771-
"a": "abort",
772764
"h": "halt",
773765
"s": "step",
774766
"b": "breakpoint",
@@ -854,11 +846,6 @@ def do_run(self, line):
854846
status = self.machine.run()
855847
print(json.dumps(status, indent=4))
856848

857-
def do_abort(self, line):
858-
"""Abort the workflow machine."""
859-
status = self.machine.abort()
860-
print(json.dumps(status, indent=4))
861-
862849
def do_halt(self, line):
863850
"""Halt the workflow machine."""
864851
status = self.machine.halt()

0 commit comments

Comments
 (0)