diff options
author | Bernard Normier <bernard@zeroc.com> | 2017-01-12 14:39:56 -0500 |
---|---|---|
committer | Bernard Normier <bernard@zeroc.com> | 2017-01-12 14:39:56 -0500 |
commit | 21d56ef199fe0002c90b57d873878ac9ba4a32e1 (patch) | |
tree | a39441e45379a574beb6563cb5b5c34cdcdf5cb1 /java/src | |
parent | Revert "Replaced Borders by Paddings to eliminate deprecation warnings" (diff) | |
download | ice-21d56ef199fe0002c90b57d873878ac9ba4a32e1.tar.bz2 ice-21d56ef199fe0002c90b57d873878ac9ba4a32e1.tar.xz ice-21d56ef199fe0002c90b57d873878ac9ba4a32e1.zip |
Reject empty endpoint in OA endpoint list
Diffstat (limited to 'java/src')
-rw-r--r-- | java/src/Ice/src/main/java/com/zeroc/Ice/ObjectAdapterI.java | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/java/src/Ice/src/main/java/com/zeroc/Ice/ObjectAdapterI.java b/java/src/Ice/src/main/java/com/zeroc/Ice/ObjectAdapterI.java index 1bbe0aa90e6..de83357a228 100644 --- a/java/src/Ice/src/main/java/com/zeroc/Ice/ObjectAdapterI.java +++ b/java/src/Ice/src/main/java/com/zeroc/Ice/ObjectAdapterI.java @@ -1210,6 +1210,10 @@ public final class ObjectAdapterI implements ObjectAdapter beg = com.zeroc.IceUtilInternal.StringUtil.findFirstNotOf(endpts, delim, end); if(beg == -1) { + if(!endpoints.isEmpty()) + { + throw new EndpointParseException("invalid empty object adapter endpoint"); + } break; } @@ -1258,17 +1262,14 @@ public final class ObjectAdapterI implements ObjectAdapter if(end == beg) { - ++end; - continue; + throw new EndpointParseException("invalid empty object adapter endpoint"); } String s = endpts.substring(beg, end); com.zeroc.IceInternal.EndpointI endp = _instance.endpointFactoryManager().create(s, oaEndpoints); if(endp == null) { - EndpointParseException e = new EndpointParseException(); - e.str = "invalid object adapter endpoint `" + s + "'"; - throw e; + throw new EndpointParseException("invalid object adapter endpoint `" + s + "'"); } endpoints.add(endp); |