C#


CSHARP.CLASS.ACCESS.BYPASS : Reflection Bypasses Member Accessibility (C#)

Summary

Accessibility of a program member is bypassed through reflection.

It is considered to be a bad practice to bypass and not consider the accessibility of a method or a field. So one should never change or ignore accessibility of classes, methods or fields since it could lead to security issues. This checker warns if this is achieved through reflection.

Properties

Class Name Reflection Bypasses Member Accessibility (C#)
Significance reliability
Mnemonic CSHARP.CLASS.ACCESS.BYPASS
Categories
CWE CWE:915 Improperly Controlled Modification of Dynamically-Determined Object Attributes
Availability Available for C# only.
Enabling Checks for this warning class are enabled by default. To disable them, add the following WARNING_FILTER rule to the project configuration file.
WARNING_FILTER += discard class="Reflection Bypasses Member Accessibility (C#)"

Example

using System.Reflection;

namespace ReflectionExample
{
    public class ReflectionCode
    {
        public static void ByPassingAccessibility()
        { 
            Type classToModify = Type.GetType("ClassToModify");
            BindingFlags newAttribute = BindingFlags.NonPublic;
            MethodInfo methodToSearch = dynClass.GetMethod("mymethod", newAttribute); // "Reflection Bypasses Member Accessibility (C#)" warning issued here 
                                                                                      // - newAttribute is set to BindingFlags.NonPublic, so this call will also retrieve non-visible progam members
            object result = methodToSearch.Invoke(classToModify, null);
        } 
    }
}

Resolution

Don't change or bypass the accessibility level of program members. Only use reflective method calls that take into account the visibility of the target member.

Relevant Configuration File Parameters

The following configuration file parameters affect checks for this warning class.