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/logging | |
| 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/logging')
| -rw-r--r-- | base/server/cmscore/src/com/netscape/cmscore/logging/AuditService.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/base/server/cmscore/src/com/netscape/cmscore/logging/AuditService.java b/base/server/cmscore/src/com/netscape/cmscore/logging/AuditService.java index 89ac6263e..1d07114b1 100644 --- a/base/server/cmscore/src/com/netscape/cmscore/logging/AuditService.java +++ b/base/server/cmscore/src/com/netscape/cmscore/logging/AuditService.java @@ -35,6 +35,7 @@ import org.apache.commons.lang.StringUtils; import org.jboss.resteasy.plugins.providers.atom.Link; import com.netscape.certsrv.apps.CMS; +import com.netscape.certsrv.base.BadRequestException; import com.netscape.certsrv.base.EBaseException; import com.netscape.certsrv.base.IConfigStore; import com.netscape.certsrv.base.PKIException; @@ -116,6 +117,8 @@ public class AuditService extends PKIService implements AuditResource { @Override public Response updateAuditConfig(AuditConfig auditConfig) { + if (auditConfig == null) throw new BadRequestException("Audit config is null."); + CMS.debug("AuditService.updateAuditConfig()"); try { |