Addrssing the comments from github#3951 (comment) that I missed previously

Author: raulgarciamsft

csharp/ql/src/experimental/Security Features/Serialization/DataSetSerialization.qll

@@ -42,17 +42,17 @@ class SerializableClass extends Class {
   SerializableClass() {
     (
       this.getABaseType*().getQualifiedName() = "System.Xml.Serialization.XmlSerializer" or
-      this.getABaseInterface*().getQualifiedName() = "System.Runtime.Serialization.ISerializable" or
+      this.getABaseType*().getQualifiedName() = "System.Runtime.Serialization.ISerializable" or
       this.getABaseType*().getQualifiedName() = "System.Runtime.Serialization.XmlObjectSerializer" or
-      this.getABaseInterface*().getQualifiedName() =
+      this.getABaseType*().getQualifiedName() =
         "System.Runtime.Serialization.ISerializationSurrogateProvider" or
       this.getABaseType*().getQualifiedName() =
         "System.Runtime.Serialization.XmlSerializableServices" or
-      this.getABaseInterface*().getQualifiedName() = "System.Xml.Serialization.IXmlSerializable"
+      this.getABaseType*().getQualifiedName() = "System.Xml.Serialization.IXmlSerializable"
     )
     or
     exists(Attribute a | a = this.getAnAttribute() |
-      a.getType().getQualifiedName().toString() = "System.SerializableAttribute"
+      a.getType().getQualifiedName() = "System.SerializableAttribute"
     )
   }
 }
@@ -77,13 +77,13 @@ class UnsafeXmlSerializerImplementation extends SerializableClass {
  */
 class UnsafeXmlReadMethod extends Method {
   UnsafeXmlReadMethod() {
-    this.getQualifiedName().toString() = "System.Data.DataTable.ReadXml"
+    this.getQualifiedName() = "System.Data.DataTable.ReadXml"
     or
-    this.getQualifiedName().toString() = "System.Data.DataTable.ReadXmlSchema"
+    this.getQualifiedName() = "System.Data.DataTable.ReadXmlSchema"
     or
-    this.getQualifiedName().toString() = "System.Data.DataSet.ReadXml"
+    this.getQualifiedName() = "System.Data.DataSet.ReadXml"
     or
-    this.getQualifiedName().toString() = "System.Data.DataSet.ReadXmlSchema"
+    this.getQualifiedName() = "System.Data.DataSet.ReadXmlSchema"
     or
     this.getName().matches("ReadXml%") and
     exists(Class c | c.getAMethod() = this |

csharp/ql/src/experimental/Security Features/Serialization/DefiningPotentiallyUnsafeXmlSerializer.ql

@@ -1,6 +1,6 @@
 /**
  * @name Defining a potentially unsafe XML serializer
- * @description Defining an XML serializable class that includes members that derive from dataSet or DataTable type may lead to a security problem. Please visit https://go.microsoft.com/fwlink/?linkid=2132227 for details.
+ * @description Defining an XML serializable class that includes members that derive from DataSet or DataTable type may lead to a security problem. Please visit https://go.microsoft.com/fwlink/?linkid=2132227 for details.
  * @kind problem
  * @problem.severity error
  * @precision medium