Bug 1846565

Summary:	Extensions in CertificateExtensions class could get out of order and cause signature discrepency
Product:	Red Hat Enterprise Linux 8	Reporter:	Christina Fu <cfu>
Component:	jss	Assignee:	RHCS Maintainers <rhcs-maint>
Status:	CLOSED NOTABUG	QA Contact:	PKI QE <bugzilla-pkiqe>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	8.3	CC:	dmoluguw
Target Milestone:	rc
Target Release:	8.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-06-22 18:14:20 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Christina Fu 2020-06-11 22:51:06 UTC

Description of problem:
Some application may rely on the order of the extensions to stay unchanged, such as when the object is manipulated by different parties and signed.  To ensure that the order doesn't change. it might be best to substitute "LinkedHashMap" with "Hashtable".

Comment 1 Dinesh Prasanth 2020-06-22 18:14:20 UTC

This bug was originally addressed via JSS PR: https://github.com/dogtagpki/jss/pull/575

Commit Information:
===================
commit 3789795e651d6f4d1e4c404fa0d7402a2d0a740a
Author: Christina Fu <cfu>
Date:   Thu Jun 11 10:54:58 2020 -0700

    bug1846565 - JSS: Extensions in CertificateExtensions class could get out of order and cause signature discrepency
    
    This patch replaces Hashtable with LinkedHashMap to ensure order of
    extensions.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1846565


Justification for CLOSING this bug:
===================================
This bug does not fix any bugs or add any new features. Instead, it modifies the
underlying datastructure used to store the data. This is low-level information
that cannot be tested OR any test cases be created. FWIW, if a CA (like dogtag PKI CA)
can issue a cert with right extension, this bug is assumed VERIFIED. So, I'm closing this
as NOTABUG