Kaydet (Commit) 7d7c2ab1 authored tarafından Tor Lillqvist's avatar Tor Lillqvist

tdf#84881: Call NSS_CMSSignerInfo_AddSigningTime() only if not using a TSA

Something is still wrong, Adobe Reader still says the PDF is signed with the
local machine's timestamp, though.

Change-Id: Ic9ed3190901025be48e1de191df976e1aa454822
üst d1132ff3
...@@ -6987,16 +6987,15 @@ bool PDFWriterImpl::finalizeSignature() ...@@ -6987,16 +6987,15 @@ bool PDFWriterImpl::finalizeSignature()
return false; return false;
} }
} }
else if (NSS_CMSSignerInfo_AddSigningTime(cms_signer, PR_Now()) != SECSuccess)
if (NSS_CMSSignerInfo_IncludeCerts(cms_signer, NSSCMSCM_CertChain, certUsageEmailSigner) != SECSuccess)
{ {
SAL_WARN("vcl.pdfwriter", "PDF signing: can't include cert chain."); SAL_WARN("vcl.pdfwriter", "PDF signing: can't add signing time.");
return false; return false;
} }
if (NSS_CMSSignerInfo_AddSigningTime(cms_signer, PR_Now()) != SECSuccess) if (NSS_CMSSignerInfo_IncludeCerts(cms_signer, NSSCMSCM_CertChain, certUsageEmailSigner) != SECSuccess)
{ {
SAL_WARN("vcl.pdfwriter", "PDF signing: can't add signing time."); SAL_WARN("vcl.pdfwriter", "PDF signing: can't include cert chain.");
return false; return false;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment