Externalizable but no public no-argument constructor¶
ID: java/missing-no-arg-constructor-on-externalizable
Kind: problem
Security severity: 
Severity: warning
Precision: medium
Tags:
   - reliability
   - maintainability
   - language-features
Query suites:
   - java-security-and-quality.qls
Click to see the query in the CodeQL repository
A class that implements java.io.Externalizable must have a public no-argument constructor. The constructor is used by the Java serialization framework when it creates the object during deserialization. If the class does not define such a constructor, the Java serialization framework throws an InvalidClassException.
The Java Development Kit API documentation for Externalizable states:
When an
Externalizableobject is reconstructed, an instance is created using the public no-arg constructor, then thereadExternalmethod called.
Recommendation¶
Make sure that externalizable classes always have a no-argument constructor.
Example¶
In the following example, WrongMemo does not declare a public no-argument constructor. When the Java serialization framework tries to deserialize the object, an InvalidClassException is thrown. However, Memo does declare a public no-argument constructor, so that the object is deserialized successfully.
class WrongMemo implements Externalizable {
    private String memo;
    // BAD: No public no-argument constructor is defined. Deserializing this object
    // causes an 'InvalidClassException'.
    public WrongMemo(String memo) {
        this.memo = memo;
    }
    public void writeExternal(ObjectOutput arg0) throws IOException {
        //...
    }
    public void readExternal(ObjectInput in) throws IOException, ClassNotFoundException {
        //...
    }
}
class Memo implements Externalizable {
    private String memo;
    // GOOD: Declare a public no-argument constructor, which is used by the
    // serialization framework when the object is deserialized.
    public Memo() {
    }
    public Memo(String memo) {
        this.memo = memo;
    }
    public void writeExternal(ObjectOutput out) throws IOException {
        //...
    }
    public void readExternal(ObjectInput in) throws IOException, ClassNotFoundException {
        //...
    }
}
References¶
- Java API Specification: Externalizable. 



