From 6c2ba9a97e244017908b02daf547d149dd13b1a2 Mon Sep 17 00:00:00 2001 From: felmer <felmer> Date: Tue, 11 Sep 2012 09:11:15 +0000 Subject: [PATCH] BIS-142 bug in TransferAuthorizationAnnotations fixed SVN: 26580 --- .../source/java/TransferAuthorizationAnnotations.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/openbis_all/source/java/TransferAuthorizationAnnotations.java b/openbis_all/source/java/TransferAuthorizationAnnotations.java index 002c948f8cd..91a47538039 100644 --- a/openbis_all/source/java/TransferAuthorizationAnnotations.java +++ b/openbis_all/source/java/TransferAuthorizationAnnotations.java @@ -59,6 +59,7 @@ public class TransferAuthorizationAnnotations if (indexOfOpenParanthesis < 0) { indexOfOpenParanthesis = firstLine.length(); + finished = true; } annotationName = firstLine.substring(indexOfStart + 1, indexOfOpenParanthesis); } @@ -169,6 +170,10 @@ public class TransferAuthorizationAnnotations private static String getCanonicalSignature(String joinedSignatureLines) { int parametersStartIndex = joinedSignatureLines.indexOf('('); + if (parametersStartIndex < 0) + { + throw new IllegalArgumentException("Invalid signature: " + joinedSignatureLines); + } int parametersEndIndex = joinedSignatureLines.lastIndexOf(')'); String[] splittedBeginning = joinedSignatureLines.substring(0, parametersStartIndex).split(" "); -- GitLab