diff options
author | Martin Polden <mpolden@mpolden.no> | 2019-09-24 12:47:20 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2019-09-24 12:47:20 +0200 |
commit | ac418673c6fb129f8e75b81e9c534b05daba54b2 (patch) | |
tree | 130fd24b5aa6493579cf9beaf4591cb89aca7ffb /node-repository/src | |
parent | a975345f861f8560def95fa3e92364ecaa0bd225 (diff) |
Require OS version to be non-empty
Diffstat (limited to 'node-repository/src')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/os/OsVersion.java | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/os/OsVersion.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/os/OsVersion.java index 99945ce46e8..8719a80e578 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/os/OsVersion.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/os/OsVersion.java @@ -16,7 +16,7 @@ public class OsVersion { private final boolean active; public OsVersion(Version version, boolean active) { - this.version = version; + this.version = requireNonEmpty(version); this.active = active; } @@ -49,4 +49,10 @@ public class OsVersion { return "OS version " + version + " [active: " + active + "]"; } + private static Version requireNonEmpty(Version version) { + Objects.requireNonNull(version, "version must be non-null"); + if (version.isEmpty()) throw new IllegalArgumentException("version must be non-empty"); + return version; + } + } |