界: Code Quality
コードの質が低いと、予測できない動作につながります。ユーザーの視点には、それがしばしば使い勝手の悪さとなって現れます。攻撃者にとっては、予期せぬ方法でシステムにストレスを与える機会となります。
Code Correctness: clone() Invokes Overridable Function
Abstract
クラス内の
clone()
メソッドは、オーバーライド可能な関数を呼び出します。Explanation
clone()
関数がオーバーライド関数をコールする場合、クローンが部分的に初期化された状態で、あるいは壊れた状態で残ることがあります。例 1: 次の
clone()
関数は、オーバーライドされる可能性のあるメソッドをコールしています。
...
class User implements Cloneable {
private String username;
private boolean valid;
public Object clone() throws CloneNotSupportedException {
final User clone = (User) super.clone();
clone.doSomething();
return clone;
}
public void doSomething(){
...
}
}
関数
doSomething()
と括弧で囲まれたクラスは final
ではないため、関数がオーバーライドされる可能性があります。その結果、クローン オブジェクト clone
が部分的に初期化された状態で残され、ロジックを対処しなかった場合、予期しない方法でエラーが起こる可能性があります。References
desc.structural.java.code_correctness_clone_invokes_overridable_function