C#


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

要旨

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.

プロパティ

クラス名 Reflection Bypasses Member Accessibility (C#)
日本語クラス名 Reflection Bypasses Member Accessibility (C#)
クラス分類 信頼性 (reliability)
ニーモニック CSHARP.CLASS.ACCESS.BYPASS
カテゴリー
CWE CWE:915 Improperly Controlled Modification of Dynamically-Determined Object Attributes
対応言語 C# で利用可能です。
有効/無効設定 このワーニングクラスのチェックはデフォルトで有効になっています。チェックを無効にするにはプロジェクト設定ファイル (configuration file)に以下の WARNING_FILTER ルールを追加してください。
WARNING_FILTER += discard class="Reflection Bypasses Member Accessibility (C#)"

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);
        } 
    }
}

解決法

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.

関連のある設定ファイルパラメータ

設定ファイルの以下のパラメータがこのワーニングクラスのチェックに影響します。