java - 如何使用AOP拦截File、FileReader、FileWriter、FileInputStream、FileOutputStream的构造函数？

Question

我想拦截 File、FileReader、FileWriter、FileInputStream 和 FileOutputStream 的构造函数，并防止任何文件名包含“..”(以防止路径遍历攻击)或“\0”(以防止文件名空字符攻击)。

关于如何使用 SecurityManager 做同样的事情，我还有另一个悬而未决的问题，但还没有人回答它，所以我希望这种替代方法能奏效。

这是用于 tomcat 上的 spring webapp。

我知道我可以通过创建自己的 SafeFile、SafeFileReader 等类并修改代码以使用它们来手动执行此操作。但是，我们的代码中有 960 个地方使用了这些对象的构造函数，所以我宁愿避免这种情况，除非这是唯一的方法。

Answer 1

即使 Sotirios Delimanolis 链接到这里的答案是正确的(我自己写的)，只要 AspectJ 而不是基于代理的 Spring AOP 是必要的，请注意你不能使用 execution(*.new(. .)) 用于 JDK 类，因为这些类在默认情况下被排除在切面编织之外。为了编织到 JDK 类中(执行连接点在逻辑上位于被调用方代码中)，您需要修改 JDK 的 rt.jar 或至少将修改后的 JDK 类放在 JDK 本身之前的引导类路径中。这是可能的，但不是那么微不足道。

但是有一个更简单的选择:通过 call(*.new(..)) 编织到调用者(你自己的代码)，而不是被调用者 - 注意 call 之间的区别() 和 execution()。这反过来意味着您无法拦截对不是由您自己的编织代码或 JDK 本身发出的 JDK 类的调用。因此，如果您需要一个 100% 的解决方案，即使是对于不在您自己控制之下的代码，您最终也会编织 JDK。如果您只想保护自己的类(class)，则可能没有必要。 :-)

尽管我是 AspectJ 的 super 粉丝，但我想强调的是，我也是干净代码和重构的坚定支持者。像 IntelliJ IDEA 或 Eclipse 这样的任何体面的 IDE 都应该使重构 960 调用以使用您建议的安全包装类变得非常简单。如果它如此重要，你为什么不这样做呢？ IDEA 的结构搜索和替换可以在几分钟内为您完成，如果不是几秒钟的话。 AOP 可以，但不应该用来修补你自己代码的缺陷。所以请去重构，之后你会更快乐。