Red Hat Bugzilla – Bug 1335919
NSS: Disable TLS connections with less than 1023-bit DH parameters
Last modified: 2017-09-25 08:56:25 EDT
RHEL includes several cryptographic components who's security doesn't remain constant over time. Algorithms such as (cryptographic) hashing and encryption typically have a lifetime after which they are considered either too risky to use or plain insecure. That would mean we need to phase out such algorithms from the default settings, or completely disable if they could cause irreparable issue.
This bug is about disabling any TLS connections which utilize less than 1024-bit Diffie-Hellman parameters, while at the same hand providing a configuration method for these parameters to be allowed when needed.
For future extensibility in RHEL-7 it is recommended for any introduced configuration method to be re-usable for future algorithm or parameter deprecation (e.g., SHA1 or less than 1024-bit RSA/DH parameters), and ideally part of upstream.
Nikos, we must clarify, what should be done for DH parameters of EXACTLY 1024 bit?
Disable, or keep enabled?
Let's assume, that Nikos want us to disable the DH params with exactly 1024 bit.
Hubert, you said in the past, we require 1024 bit for Java compatibility.
And we had implemented API SSL_EnableWeakDHEPrimeGroup to provide support for 1024 bit compatibility.
Does that mean, we should patch SSL_EnableWeakDHEPrimeGroup to NOT enable the 1024 bit group?
Open question, should it return a failure, or should it be a no-operation?
Daiki pointed me to a patch in RHEL 6.x, which shows, that upstream requires a minimum DH size of 1023 bits.
We had decreased that limit in previous RHEL 6.x to 768 bits.
Should we use the same as upstream, and increase the limit to 1023 ?
(In reply to Kai Engert (:kaie) from comment #4)
> Let's assume, that Nikos want us to disable the DH params with exactly 1024
> Hubert, you said in the past, we require 1024 bit for Java compatibility.
> And we had implemented API SSL_EnableWeakDHEPrimeGroup to provide support
> for 1024 bit compatibility.
> Does that mean, we should patch SSL_EnableWeakDHEPrimeGroup to NOT enable
> the 1024 bit group?
> Open question, should it return a failure, or should it be a no-operation?
If the system policy is to disable 1024 bit DH and application tries to enable 1024 bit DH, in my opinion the call should fail.
Similarly to how trying to generate 256 bit RSA key now will fail.
1. the application should not use this API by default, it should require explicit setting by user (so user should be able to disable use of that api in application)
2. even in the case the application is broken and forces that setting on user, the user can change the system policy to legacy where the call should succeed
I believe the summary of this bug should be changed to:
"Disable ... with less than 1023-bit DH"
If this is what we do, then we'll continue to support the 1024-bit DH, and the current APIs don't need to be changed, because we'd still allow the application to allow the 1024 bit groups.
(I hope I'm not confusing anything between DH and DHE?)
Possibly, the only change necessary here is to remove this patch?
(In reply to Kai Engert (:kaie) from comment #10)
> I believe the summary of this bug should be changed to:
> "Disable ... with less than 1023-bit DH"
> If this is what we do, then we'll continue to support the 1024-bit DH, and
> the current APIs don't need to be changed, because we'd still allow the
> application to allow the 1024 bit groups.
yes, we want to continue to support 1024 bit
while dropping support for 1023 bit at the same time is an arbitrary threshold, I haven't see any servers that advertise 1023 bit DH parameters (unlike RSA keys)
> (I hope I'm not confusing anything between DH and DHE?)
while there is a difference between (static) DH and DHE, NSS only supports the latter, and in that case it's all unambitious
> Possibly, the only change necessary here is to remove this patch?
no, it's not sufficient. The change in question must be reversible by changing system configuration, yes the default should match the upstream behaviour, but user needs to be able to set the NSS behaviour to the way NSS behaved in the past
Issue with NSS requiring an empty line at the end of policy file before it is recognised is tracked in bug 1397979.
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory, and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.
cannot find this in release docs, could someone please point me to where this is stated in the official docs?
Other wise I am inclined to reopen this bug until docs text is included in release notes
It is on the deprecated functionality section: