CodeQL documentation

Deserialized delegate

ID: cs/deserialized-delegate
Kind: problem
Severity: warning
Precision: high
   - security
   - external/cwe/cwe-502
Query suites:
   - csharp-code-scanning.qls
   - csharp-security-extended.qls
   - csharp-security-and-quality.qls

Click to see the query in the CodeQL repository

Deserializing a delegate object may result in remote code execution, when an attacker can control the serialized data.


Avoid deserializing delegate objects, if possible, or make sure that the serialized data cannot be controlled by an attacker.


In this example, a file stream is deserialized to a Func<int> object, using a BinaryFormatter. The file stream is a parameter of a public method, so depending on the calls to InvokeSerialized, this may or may not pose a security problem.

using System;
using System.IO;
using System.Runtime.Serialization.Formatters.Binary;

class Bad
    public static int InvokeSerialized(FileStream fs)
        var formatter = new BinaryFormatter();
        // BAD
        var f = (Func<int>)formatter.Deserialize(fs);
        return f();