glance requires md5 implementation be available

Bug #1875439 reported by Brian Rosmaita
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
High
Unassigned
OpenStack Security Advisory
Won't Fix
Undecided
Unassigned

Bug Description

Glance populates a legacy 'checksum' image property which is an md5 hash of image data content. It's a "legacy" property because it has not been required for the validation of downloaded image data since glance version 17.0.0 (Rocky) when the operator-configurable secure "multihash" was implemented. However, the 'checksum' property has continued to be populated for backward compatibility. In order to populate the field, even as a courtesy, an implementation of the md5 algorithm must be available to glance; but this cannot be guaranteed in environments that comply with various security standards (for example, FIPS). As a result, there are environments in which glance cannot be run, and of course, these are most likely exactly the environments in which people want to run glance.

To remove the dependency on the insecure MD5 algorithm, glance should stop populating the legacy 'checksum' field. It has already been made redundant by the secure "multihash" and is unnecessary. In order to preserve backward compatibility, the field will not be removed.

As a timeframe for fixing this: an announcement can be made to operators as part of the Ussuri release, and code using md5 will be removed during the Victoria development cycle. Thus the Victoria release will not require Glance to be executed in a non-compliant security environment.

Tags: security
Revision history for this message
Jeremy Stanley (fungi) wrote :

Given this is only being fixed in master, and is also not in itself a vulnerability, I don't think we'll need a formal security advisory and CVE assignment. This is probably most accurately classified as a security hardening opportunity (report class D in the VMT's taxonomy): https://security.openstack.org/vmt-process.html#incident-report-taxonomy

Changed in ossa:
status: New → Won't Fix
information type: Public Security → Public
tags: added: security
Revision history for this message
Jeremy Stanley (fungi) wrote :

The Ussuri announcement to operators is probably best covered in the release notes and highlights (as well as deprecation wording in Glance's documentation). The Victoria announcement might make sense to be accompanied by an OSSN (OpenStack Security Note).

Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

Patch to the ussuri release notes: https://review.opendev.org/723638

Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

I forgot to mention that I agree with Jeremy's assessment of this as class D. An OSSN during Victoria is a good idea; people for whom this is a problem may wish to plan ahead to upgrade their clouds.

Changed in glance:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

Filed https://bugs.launchpad.net/glance/+bug/1875629 so we don't forget about the documentation update that Jeremy suggested.

Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

Filed https://bugs.launchpad.net/glance/+bug/1875630 so we don't forget about the OSSN.

Revision history for this message
Nick Tait (nickthetait) wrote :

Class D and some documentation notes make sense to me.

Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

This was fixed by Change-Id: I0603ba217d6dc19f5c9f73c60c7b365efd28d30b to glance_store, which was merged in Wallaby.

Changed in glance:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.