Remove redundant taint-tracking from MarshalingFunction and UnmarshalingFunction classes in EncodingXml module.

Author: gagliardetto

ql/src/semmle/go/frameworks/stdlib/EncodingXml.qll

@@ -7,30 +7,22 @@ import go
 /** Provides models of commonly used functions in the `encoding/xml` package. */
 module EncodingXml {
   /** The `Marshal` or `MarshalIndent` function in the `encoding/xml` package. */
-  class MarshalFunction extends TaintTracking::FunctionModel, MarshalingFunction::Range {
+  private class MarshalFunction extends MarshalingFunction::Range {
     MarshalFunction() {
       this.hasQualifiedName("encoding/xml", "Marshal") or
       this.hasQualifiedName("encoding/xml", "MarshalIndent")
     }
 
-    override predicate hasTaintFlow(FunctionInput inp, FunctionOutput outp) {
-      inp = getAnInput() and outp = getOutput()
-    }
-
     override FunctionInput getAnInput() { result.isParameter(0) }
 
     override FunctionOutput getOutput() { result.isResult(0) }
 
     override string getFormat() { result = "XML" }
   }
 
-  private class UnmarshalFunction extends TaintTracking::FunctionModel, UnmarshalingFunction::Range {
+  private class UnmarshalFunction extends UnmarshalingFunction::Range {
     UnmarshalFunction() { this.hasQualifiedName("encoding/xml", "Unmarshal") }
 
-    override predicate hasTaintFlow(FunctionInput inp, FunctionOutput outp) {
-      inp = getAnInput() and outp = getOutput()
-    }
-
     override FunctionInput getAnInput() { result.isParameter(0) }
 
     override FunctionOutput getOutput() { result.isParameter(1) }