/** * @id cpp/check-type-before-use * @name Check type before use * @kind path-problem */ import cpp import semmle.code.cpp.dataflow.new.DataFlow import semmle.code.cpp.controlflow.Guards int dyn_input_type_mem() { result = 1 } int dyn_input_type_val() { result = 2 } predicate typeValidationCallMatchesUse(TypeValidationCall call, DynamicInputAccess use) { exists(FieldAccess f, int expected | expected = call.getExpectedInputType(use.getArrayOffset().getValue().toInt()) and f.getQualifier() = use and ( expected = dyn_input_type_mem() and f.getTarget().getName() = "ptr" or expected = dyn_input_type_val() and f.getTarget().getName() = "val" ) ) } /** * An access of a dynamic input array (of type `dyn_input_t`) */ class DynamicInputAccess extends ArrayExpr { DynamicInputAccess() { this.getArrayBase().getType().(DerivedType).getBaseType().getName() = "dyn_input_t" } predicate isTypeNotValidated() { not typeValidationGuardOrIndirect(_, _, _, this.getBasicBlock()) or exists(TypeValidationCall call | typeValidationGuardOrIndirect(_, call, _, this.getBasicBlock()) and not typeValidationCallMatchesUse(call, this) ) } } /** * A call to `DYN_INPUT_TYPE` */ class TypeValidationCall extends FunctionCall { TypeValidationCall() { this.getTarget().hasName("DYN_INPUT_TYPE") } int getExpectedInputType(int input_index) { result = this.getArgument(input_index).getValue().toInt() } } /** * Relates a `call` to a `guard`, which uses the result of the call to validate * equality of the result of `call` against `other` to guard `block`. */ predicate typeValidationGuard( GuardCondition guard, TypeValidationCall call, Expr other, BasicBlock block ) { exists(Expr dest | DataFlow::localExprFlow(call, dest) and guard.ensuresEq(dest, other, 0, block, true) and InputTypesToTypeValidation::hasFlowToExpr(other) ) } predicate typeValidationGuardOrIndirect( GuardCondition guard, TypeValidationCall call, Expr other, BasicBlock block ) { typeValidationGuard(guard, call, other, block) or typeValidationGuardOrIndirect(guard, call, other, block.getEnclosingFunction().getACallToThisFunction().getBasicBlock()) } /** * A global data-flow configuration tracking flow from an entrypoint's * `input_types` parameter to a subsequent use in a GuardCondition. */ module InputTypesToTypeValidation = DataFlow::Make; module InputTypesToTypeValidationConfig implements DataFlow::ConfigSig { predicate isSource(DataFlow::Node source) { exists(EntrypointFunction f | f.getInputTypesParameter() = source.asParameter()) } predicate isSink(DataFlow::Node sink) { // avoid non-monotonic recursion, as `typeValidationGuard` depends on this config sink.asExpr() instanceof VariableAccess } } /** * An entrypoint that has a `dyn_array_t[2]` `input` parameter and * `input_types` parameter. */ class EntrypointFunction extends Function { EntrypointFunction() { this.hasName(["EP_copy_mem", "EP_print_val", "EP_write_val_to_mem"]) } Parameter getInputParameter() { result = this.getParameter(0) } Parameter getInputTypesParameter() { result = this.getParameter(1) } } /** * A global data-flow configuration tracking flow from an entrypoint's * `input` parameter to a subsequent `ArrayExpr` access. */ module InputToAccess = DataFlow::Make; int explorationLimit() { result = 2 } module InputToAccessFlowExploration = InputToAccess::FlowExploration; import InputToAccessFlowExploration::PartialPathGraph module InputToAccessConfig implements DataFlow::ConfigSig { predicate isSource(DataFlow::Node source) { exists(EntrypointFunction ep | ep.getInputParameter() = source.asParameter()) } predicate isSink(DataFlow::Node sink) { exists(DynamicInputAccess access | sink.asExpr() = access.getArrayBase() and access.isTypeNotValidated() ) } } from InputToAccessFlowExploration::PartialPathNode source, InputToAccessFlowExploration::PartialPathNode mid, int dist where InputToAccessFlowExploration::hasPartialFlow(source, mid, dist) and // not a sink not InputToAccessConfig::isSink(mid.getNode()) and // no flow from mid not DataFlow::localFlowStep(mid.getNode(), _) and // more than 1 interprocedural step from source dist >= 1 select mid, source, mid, "Partial flow"