CA2114: a segurança de método deve ser um superconjunto de tipo

 

Para obter a documentação mais recente do Visual Studio 2017 RC, consulte a documentação do Visual Studio 2017 RC.

TypeNameMethodSecurityShouldBeASupersetOfType
CheckIdCA2114
CategoriaMicrosoft.Security
Alteração SignificativaQuebra

Um tipo tem a segurança declarativa e um de seus métodos tem a segurança declarativa para a mesma ação de segurança, e a ação de segurança não é Demandas de link ou Inheritance Demands, e as permissões verificadas pelo tipo não são um subconjunto das permissões verificadas pelo método.

Um método não deve ter um nível de método e segurança declarativa de classificação nível para a mesma ação. As duas verificações não são combinadas; somente a procura de nível de método é aplicada. Por exemplo, se um tipo requer a permissão X, e um de seus métodos requer a permissão Y, código não tem que ter a permissão X executar o método.

Revisar seu código para assegurar que ambas as ações são necessárias. Se ambas as ações são necessárias, verifique se a ação de nível de método inclui a segurança especificada no nível do tipo. Por exemplo, se seu tipo requer a permissão X, e seu método também deve exigir a permissão Y, o método deve solicitar explicitamente X e Y.

É seguro suprimir um aviso dessa regra se o método não requer segurança especificada pelo tipo. No entanto, isso não é um cenário comum e pode indicar a necessidade para revisão de projeto cuidadosa.

O exemplo a seguir usa permissões de ambiente demonstrar os perigos de violar esta regra. Neste exemplo, o código do aplicativo cria uma instância do tipo protegido antes de negar a permissão necessária do tipo. Em um cenário de ameaça do mundo real, o aplicativo pode exigir outra maneira de fazer uma instância do objeto.

No exemplo a seguir, as demandas da biblioteca permissão de gravação para um tipo e uma permissão de leitura para um método.

using System;
using System.Security;
using System.Security.Permissions;
using System.Runtime.InteropServices;

namespace SecurityRulesLibrary
{
   [EnvironmentPermissionAttribute(SecurityAction.Demand, Write="PersonalInfo")]
   public class MyClassWithTypeSecurity
   {
      [DllImport("kernel32.dll", CharSet=CharSet.Unicode, SetLastError=true)]
      [return:MarshalAs(UnmanagedType.Bool)]
      public static extern bool SetEnvironmentVariable(
         string lpName,
         string lpValue);

      // Constructor.
      public MyClassWithTypeSecurity(int year, int month, int day)
      {
         DateTime birthday = new DateTime(year, month, day);

         // Write out PersonalInfo environment variable.
         SetEnvironmentVariable("PersonalInfo",birthday.ToString());
      }

      [EnvironmentPermissionAttribute(SecurityAction.Demand, Read="PersonalInfo")]
      public string PersonalInformation ()
      { 
         // Read the variable.
         return Environment.GetEnvironmentVariable("PersonalInfo"); 
      }
   }
}

O seguinte código do aplicativo demonstra a vulnerabilidades da biblioteca chamando o método mesmo que não satisfaz o requisito de segurança de classificação nível.

using System;
using System.Security;
using System.Security.Permissions;
using SecurityRulesLibrary;

namespace TestSecRulesLibrary
{
   public class TestMethodLevelSecurity
   {
      MyClassWithTypeSecurity dataHolder;

      void RetrievePersonalInformation(string description) 
      {
         try 
         { 
            Console.WriteLine(
               "{0} Personal information: {1}", 
               description, dataHolder.PersonalInformation());
         }
         catch (SecurityException e) 
         {
            Console.WriteLine(
               "{0} Could not access personal information: {1}", 
               description, e.Message);
         }
      }

      [STAThread]
      public static void Main() 
      {
         TestMethodLevelSecurity me = new TestMethodLevelSecurity();

         me.dataHolder = new MyClassWithTypeSecurity(1964,06,16);

         // Local computer zone starts with all environment permissions.
         me.RetrievePersonalInformation("[All permissions]");

         // Deny the write permission required by the type.
         EnvironmentPermission epw = new EnvironmentPermission(
            EnvironmentPermissionAccess.Write,"PersonalInfo");
         epw.Deny();
         
         // Even though the type requires write permission, 
         // and you do not have it; you can get the data.
         me.RetrievePersonalInformation(
            "[No write permission (demanded by type)]");

         // Reset the permissions and try to get 
         // data without read permission.
         CodeAccessPermission.RevertAll();  

         // Deny the read permission required by the method.
         EnvironmentPermission epr = new EnvironmentPermission(
            EnvironmentPermissionAccess.Read,"PersonalInfo");
         epr.Deny();

         // The method requires read permission, and you
         // do not have it; you cannot get the data.
         me.RetrievePersonalInformation(
            "[No read permission (demanded by method)]");
      }
   }
}

O exemplo produz a seguinte saída.

[Informações pessoais de todas as permissões]: 6/16/1964 de 12:00: 00 AM
[Nenhuma permissão de gravação (necessária do tipo)] Informações pessoais: 6/16/1964 de 12:00: 00 AM
[Nenhuma permissão de leitura (exigida pelo método)] Não foi possível acessar informações pessoais: A solicitação falha.

Diretrizes de codificação segura
Inheritance Demands
Demandas de link
Dados e modelagem

Mostrar: