diff options
author | Abhishek Koneru <akoneru@redhat.com> | 2012-06-22 17:25:32 -0400 |
---|---|---|
committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-06-29 14:15:51 -0500 |
commit | d2b06ee1f9fcc42be83f04f64559cfa6317bda67 (patch) | |
tree | 5bcffe77451fb1ab194339d9aa8e759968fd4d27 /base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java | |
parent | eb4b46f25ca8dec8ac79c26cfb02e8918000c88e (diff) | |
download | pki-d2b06ee1f9fcc42be83f04f64559cfa6317bda67.tar.gz pki-d2b06ee1f9fcc42be83f04f64559cfa6317bda67.tar.xz pki-d2b06ee1f9fcc42be83f04f64559cfa6317bda67.zip |
Fix for handling null object value passed to DBAttrMapper as part of Coverity fix for Forward NULL cases in DogTag 10.
Diffstat (limited to 'base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java')
-rw-r--r-- | base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java b/base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java index 61d823b88..a074b41e6 100644 --- a/base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java +++ b/base/common/src/com/netscape/cmscore/dbs/CertRecordMapper.java @@ -59,17 +59,13 @@ public class CertRecordMapper implements IDBAttrMapper { public void mapObjectToLDAPAttributeSet(IDBObj parent, String name, Object obj, LDAPAttributeSet attrs) throws EBaseException { - try { - CertRecord rec = (CertRecord) obj; - - attrs.add(new LDAPAttribute( - CertDBSchema.LDAP_ATTR_CERT_RECORD_ID, - rec.getSerialNumber().toString())); - } catch (Exception e) { - Debug.trace(e.toString()); - throw new EDBException( - CMS.getUserMessage("CMS_DBS_SERIALIZE_FAILED", name)); + if (obj == null) { + throw new EBaseException(CMS.getUserMessage("CMS_DBS_SERIALIZE_FAILED", name)); } + CertRecord rec = (CertRecord) obj; + attrs.add(new LDAPAttribute( + CertDBSchema.LDAP_ATTR_CERT_RECORD_ID, + rec.getSerialNumber().toString())); } public void mapLDAPAttributeSetToObject(LDAPAttributeSet attrs, |