diff options
author | Endi S. Dewata <edewata@redhat.com> | 2013-11-11 15:39:25 -0500 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2013-11-14 15:14:20 -0500 |
commit | 17a52b686bd81cda1bce76b454b3127d6575de62 (patch) | |
tree | ca32ab491f231af733e6fb9016e97b6dcd787298 /base/server/cmscore/src/com/netscape/cmscore/selftests | |
parent | 2442de787890cc6a7ea703f26fd1527718b2ac1b (diff) | |
download | pki-17a52b686bd81cda1bce76b454b3127d6575de62.tar.gz pki-17a52b686bd81cda1bce76b454b3127d6575de62.tar.xz pki-17a52b686bd81cda1bce76b454b3127d6575de62.zip |
Added more null parameter checking.
Some REST methods have been modified to check for null parameters
and return the proper error code.
Ticket #749
Diffstat (limited to 'base/server/cmscore/src/com/netscape/cmscore/selftests')
-rw-r--r-- | base/server/cmscore/src/com/netscape/cmscore/selftests/SelfTestService.java | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/base/server/cmscore/src/com/netscape/cmscore/selftests/SelfTestService.java b/base/server/cmscore/src/com/netscape/cmscore/selftests/SelfTestService.java index 4fcddb7e4..bd02b6752 100644 --- a/base/server/cmscore/src/com/netscape/cmscore/selftests/SelfTestService.java +++ b/base/server/cmscore/src/com/netscape/cmscore/selftests/SelfTestService.java @@ -141,6 +141,8 @@ public class SelfTestService extends PKIService implements SelfTestResource { @Override public SelfTestData getSelfTest(String selfTestID) { + if (selfTestID == null) throw new BadRequestException("Self test ID is null."); + CMS.debug("SelfTestService.getSelfTest(\"" + selfTestID + "\")"); try { @@ -156,6 +158,8 @@ public class SelfTestService extends PKIService implements SelfTestResource { @Override public void executeSelfTests(String action) { + if (action == null) throw new BadRequestException("Action is null."); + CMS.debug("SelfTestService.executeSelfTests(\"" + action + "\")"); if (!"run".equals(action)) { |