Closed Bug 384470 Opened 16 years ago Closed 2 years ago

javadoc for org.mozilla.jss.pkix.cms.SignedData.getSignerInfos() is incorrect

Categories

(JSS Graveyard :: Documentation, defect)

x86
Windows XP
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: david.konrad.stutzman, Assigned: david.konrad.stutzman)

References

()

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4) Gecko/20070515 Firefox/2.0.0.4
Build Identifier: 4.2

I think the javadoc for the getSignerInfos() method is wrong.  It says it returns "a SET of org.mozilla.jss.pkcs7.SignerInfo" but I think it is a SET of org.mozilla.jss.pkix.cms.SignerInfo.


Reproducible: Always

Steps to Reproduce:
Below isn't really complete but you should get the point...

import org.mozilla.jss.pkix.cms.ContentInfo;
import org.mozilla.jss.pkix.cms.SignedData;
//import org.mozilla.jss.pkix.cms.SignerInfo;
import org.mozilla.jss.pkcs7.SignerInfo;

ContentInfo ci = (ContentInfo) ASN1Util.decode(ContentInfo.getTemplate(), binaryResponse);
SignedData response = (SignedData) ci.getInterpretedContent();
SignerInfo sinfo = (SignerInfo) response.getSignerInfos().elementAt(0);
Actual Results:  
org.mozilla.jss.pkix.cms.SignerInfo cannot be cast to org.mozilla.jss.pkcs7.SignerInfo
QA Contact: jss-qa
Assignee: gbmozilla → nobody
Attached patch Fix for javadocSplinter Review
Fixes the type mentioned in the JavaDoc to be the correct package. 

Also removes 3 unused imports (that part of patch can easily be discarded if you don't feel proper to change in this issue).
Assignee: nobody → david.konrad.stutzman
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: javadoc for org.mozilla.jss.pkix.cms.SignerInfo.getSignerInfos() is incorrect → javadoc for org.mozilla.jss.pkix.cms.SignedData.getSignerInfos() is incorrect
Attachment #8942270 - Flags: review?
JSS development has moved from the Mozilla community to the Dogtag PKI community. Please re-file this bug at https://github.com/dogtagpki/jss if it is still relevant. Thank you!
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.