diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-12-11 02:28:34 +0100 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-12-11 02:28:34 +0100 |
commit | 266ee7626c4c602c6f690c7c663526e38141986b (patch) | |
tree | b28166b4cf938d0a75fc5ad82f84c0cda89cd350 /app/src/main/java/com/wireguard/config/Config.java | |
parent | c1ba1f409cd1566cb11d0c26d610ffdff4d2e24c (diff) |
Throw illegalargumentexception instead of nullpointerexception for builder errors
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'app/src/main/java/com/wireguard/config/Config.java')
-rw-r--r-- | app/src/main/java/com/wireguard/config/Config.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/app/src/main/java/com/wireguard/config/Config.java b/app/src/main/java/com/wireguard/config/Config.java index 6c9d7ba3..7b9bcab3 100644 --- a/app/src/main/java/com/wireguard/config/Config.java +++ b/app/src/main/java/com/wireguard/config/Config.java @@ -30,7 +30,9 @@ public final class Config { private final List<Peer> peers; private Config(final Builder builder) { - interfaze = Objects.requireNonNull(builder.interfaze, "An [Interface] section is required"); + if (builder.interfaze == null) + throw new IllegalArgumentException("An [Interface] section is required"); + interfaze = builder.interfaze; // Defensively copy to ensure immutability even if the Builder is reused. peers = Collections.unmodifiableList(new ArrayList<>(builder.peers)); } |