Skip to content

Initialize pointer variables to NULL in ModelicaInternal.c#4783

Open
DagBruck wants to merge 2 commits intomasterfrom
DagBruck-patch-1
Open

Initialize pointer variables to NULL in ModelicaInternal.c#4783
DagBruck wants to merge 2 commits intomasterfrom
DagBruck-patch-1

Conversation

@DagBruck
Copy link
Copy Markdown
Contributor

Pointer variables initialized at definition.

Pointer variables initialized at definition.
@DagBruck DagBruck assigned HansOlsson and unassigned HansOlsson Apr 29, 2026
@DagBruck DagBruck self-assigned this Apr 29, 2026
@DagBruck DagBruck added the L: C-Sources Issue addresses Modelica/Resources/C-Sources label Apr 29, 2026
@StephanZiegler
Copy link
Copy Markdown
Contributor

Code looks fine to me.
I added a changelog entry, please check if this is fine.

Should anyone outside our company review this as well?

@DagBruck DagBruck requested review from GallLeo and beutlich and removed request for StephanZiegler April 29, 2026 13:34
Copy link
Copy Markdown
Member

@beutlich beutlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this just to silence some compiler warnings or is there actually a code path where unitialized variable is accessed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L: C-Sources Issue addresses Modelica/Resources/C-Sources

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants