nextgenhealthcare / connect

The swiss army knife of healthcare integration.
Other
868 stars 265 forks source link

[SECURITY] Switch XStream to Use an allowlist Instead of a denylist #5952

Closed jdonextgen closed 8 months ago

jdonextgen commented 8 months ago

Describe the security issue Mirth Connect uses the XStream library to serialize and deserialize objects. It is a potential security risk to allow unexpected object types to be processed through XStream. Therefore we are now being more restrictive about what types are allowed by using an allowlist instead of a denylist. The only types allowed are those that are strictly necessary for Mirth Connect to operate.

Vulnerability Location This vulnerability exists in the XStream library or how we use the XStream library.

Environment (please complete the following information if it is applicable to the issue)

Suggested remediation Switch to using an allowlist which would make the accepted types more restrictive.

Additional context n/a

jdonextgen commented 8 months ago

Fix has been committed and will be available in Mirth Connect 4.4.1

todb-cisa commented 1 month ago

This version appears to fix CVE-2023-43208, implied rather strongly by this blog post Is this accurate?