GitHub Advisoryadvisory · npm

GHSA-4jg2-84c2-pj95 Improper Control of Generation of Code ('Code Injection') in @asyncapi/modelina

CRITICAL Published 2021-09-21 Modified 2023-01-26 Official page ↗
Aliases
CVE-2023-23619
Ecosystem
npm
Packages
@asyncapi/modelina
Severity
Critical
CVSS vector
CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:C/C:H/I:H/A:H
Weaknesses
CWE-94
Reviewed
2021-09-21

Affected packages

• npm @asyncapi/modelina: < 1.0.0 → fixed in 1.0.0

Description

Impact Anyone who is using the default presets and/or does not handle the functionality themself. Patches It is impossible to fully guard against this, because users have access to the original raw information. However, as of version 1, if you only access the constrained models, you will not encounter this issue. Further similar situations are NOT seen as a security issue, but intended behavior. Workarounds Fully custom presets that change the entire rendering process which can then escape the user input. For more information Even though that I changed all the presets here, the vulnerability is still present throughout. I am using a JSON Schema here for simplicity. ```ts const jsonSchemaDoc = { $id: 'CustomClass', type: 'object', properties: { 'property: any; \n constructor(){console.log("injected")} \n private _temp': { type: 'string' }, } }; generator = new TypeScriptGenerator( { presets: [ { class: { property({ propertyName, content }) { return `private ${propertyName}: any;`; }, ctor() { return ''; }, getter() { return ''; }, setter() { return ''; } } } ] } ); const inputModel = await generator.process(jsonSchemaDoc); ``` This would render ```ts export class CustomClass { private property: any; constructor(){console.log("injected")} private _temp: any; private additionalProperties: any; } ```

Related

Alias · CVE (1)

  • CVE-2023-23619 Modelina is a library for generating data models based on inputs such as AsyncAPI, OpenAPI, or JSON Schema documents. vulnerability

Weakness · CWE (1)

  • CWE-94 Improper Control of Generation of Code weakness

References