Revert "Simplify MissingCallTarget for calli"

tamasvajk · tamasvajk · commit fa2f3456113d · 2021-03-02T12:58:42.000+01:00
This reverts commit 3b82abd.
diff --git a/csharp/ql/src/semmle/code/cil/ConsistencyChecks.qll b/csharp/ql/src/semmle/code/cil/ConsistencyChecks.qll
@@ -83,7 +83,7 @@ class MissingCallTarget extends InstructionViolation {
     exists(Call c | c = instruction |
       count(c.getTarget()) != 1 and not c instanceof Opcodes::Calli
       or
-      count(c.(Opcodes::Calli).getTargetType()) != 1
+      count(c.(Opcodes::Calli).getTargetType()) != 1 and c instanceof Opcodes::Calli
     )
   }
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ class MissingCallTarget extends InstructionViolation {`
`83`	`83`	`exists(Call c \| c = instruction \|`
`84`	`84`	`count(c.getTarget()) != 1 and not c instanceof Opcodes::Calli`
`85`	`85`	`or`
`86`		`- count(c.(Opcodes::Calli).getTargetType()) != 1`
	`86`	`+ count(c.(Opcodes::Calli).getTargetType()) != 1 and c instanceof Opcodes::Calli`
`87`	`87`	`)`
`88`	`88`	`}`
`89`	`89`