执行 .NET “安全检查”会导致安全错误吗？

Question

我有一段代码：

public void MyMethod()
{
   DirectoryEntry de; 
   ...
   de.AuthenticationType = AuthenticationTypes.Secure;
   ...
}

那个FxCop chokes on:

CA2122：不要间接暴露具有链接需求的方法

解决方案：MyMethod() 调用具有 LinkDemand 的 DirectoryEntry.AuthenticationType.set(AuthenticationTypes)。
通过进行此调用，DirectoryEntry.AuthenticationType.set(AuthenticationTypes) 间接暴露给用户代码。

信息：不要用不执行安全检查的方法包装受 LinkDemand 保护的方法。 LinkDemand 检查直接调用者的权限，而不是检查调用堆栈中所有调用者的权限。在这种情况下，将检查包装方法的权限。如果包装方法本身不检查调用者的权限 在调用堆栈的较高位置，恶意代码可能能够执行包装函数，即使它没有执行此操作的权限。”

我完全赞成添加 something, somewhere 来“修复”这个问题。但如果它会导致当前为客户工作的代码自发不为客户工作，则无法添加。

注意：我不知道要添加什么，或者在哪里（FxCop 不包含该信息），我不知道如果它是死胡同，不想深入研究代码安全的秘密世界。

如果我向MyMethod 添加“安全检查*”，当前有效的代码是否有可能停止工作？

---

我假设如果有人没有权限，现在编写的代码将不会工作。换句话说：

directoryEntry.AuthenticationType = AuthenticationTypes.Secure

如果某人没有正确的“权限”，将会已经失败。在调用堆栈的更高位置添加“安全检查”不会改变这一事实 - 只会更快地触发失败。在这种情况下，添加安全检查就可以了。

另一方面，如果：

public void MyMethod() {...}

MyMethod();

目前有效，但是

[SecurityCheck(...)]
public void MyMethod() {...}

AD.MyMethod()

将开始失败，然后我不能真正添加​​它。

尤其是在每个人都使用的库代码中。

---

我无法自己测试的原因是没有人知道如何复制出现问题的情况。

就像大多数人通过尝试connect to AD with a username and password and read a property 来检查活动目录的凭据一样。即使用户名/密码正确，您也可能被允许读取属性。我敢打赌，除了我自己之外，没有人知道配置 ActiveDirectory 来复制该失败案例。

就我而言，我认为没有人知道如何配置东西以使代码安全性失败。

Answer 1

我假设现在编写的代码如果有人 没有权限

LinkDemand 不是这样，它只要求直接调用者拥有目标权限。这就是 FxCop 警告您潜在问题的原因：恶意调用者可能会使用您的方法间接调用 DirectoryEntry 方法，尽管它本身没有足够的 CAS 权限。

解决此问题的最佳方法取决于几件事，包括您使用的 .NET Framework 版本。对于早期的 .NET 版本，将 LinkDemand 或对不受限制的 DirectoryServicesPermission 的完整需求添加到您的方法中。如果 FxCop 很高兴，那么您就已经解决了这个特殊问题。但是，某些调用者可能无法再调用您的方法。如果这是一个问题，您需要评估您对 DirectoryEntry 的使用，并确定它是否可能被恶意调用者滥用，然后再取消保护。

对于更新的 .NET 版本，事情变得更加复杂。如果您需要这方面的帮助，请说明您所针对的 .NET 版本以及您是否认为有必要支持部分受信任的调用方。