CodeQL documentation

Potentially incorrect Equals(…) signature

ID: cs/wrong-equals-signature
Kind: problem
Security severity: 
Severity: warning
Precision: medium
   - reliability
   - maintainability
Query suites:
   - csharp-security-and-quality.qls

Click to see the query in the CodeQL repository

The intent of defining an Equals(object) method on a class is generally to ensure that instances of that class are compared using it, both in client code and in library code. The standard Equals(object) method has an object parameter, so Equals(T) methods where T is not object, will often not be used in favor of Equals(object).


Define an Equals(object) method that returns false if the object parameter is not of the type containing the method.


In this example, the Equals method only takes an instance of Bad as a parameter.

using System;

class Bad
    private int id;

    public Bad(int Id)
    { = Id;

    public bool Equals(Bad b) => ==;

In the revised example, Equals(object) is overridden and defined in terms of the Equals(Good) method.

using System;

class Good
    private int id;

    public Good(int Id)
    { = Id;

    public bool Equals(Good g) => ==;

    public override bool Equals(object o)
        if (o is Good g && g.GetType() == typeof(Good))
            return this.Equals(g);
        return false;


  • © GitHub, Inc.
  • Terms
  • Privacy