Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable-v2.12] rimage: fix resource leak #9743

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

abonislawski
Copy link
Member

No description provided.

Ensure null termination in man_module_create_reloc

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
Fix resource leak in man_module_create_reloc

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
@kv2019i
Copy link
Collaborator

kv2019i commented Dec 19, 2024

@abonislawski Can we merge to main first and then backport with a "cherry-picked xxx) entry? We still have time until January, so I'd prefer to have all changes go through main.

@lgirdwood
Copy link
Member

@abonislawski Can we merge to main first and then backport with a "cherry-picked xxx) entry? We still have time until January, so I'd prefer to have all changes go through main.

This needs to be today.

@kv2019i kv2019i requested a review from lyakh December 19, 2024 12:25
@kv2019i
Copy link
Collaborator

kv2019i commented Dec 19, 2024

Let's go with direct PR here then. @abonislawski please submit to main still on the side.

@@ -512,6 +512,8 @@ static int man_module_create_reloc(struct image *image, struct manifest_module *
unsigned int j;

strncpy(name, (char *)sof_mod->module.name, SOF_MAN_MOD_NAME_LEN);
Copy link
Contributor

Choose a reason for hiding this comment

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

s/strncpy/strlcpy ?

Copy link
Member

Choose a reason for hiding this comment

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

This would be a further update if needed, I think we are good to go today with @abonislawski fix.

@@ -512,6 +512,8 @@ static int man_module_create_reloc(struct image *image, struct manifest_module *
unsigned int j;

strncpy(name, (char *)sof_mod->module.name, SOF_MAN_MOD_NAME_LEN);
Copy link
Member

Choose a reason for hiding this comment

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

This would be a further update if needed, I think we are good to go today with @abonislawski fix.

@abonislawski
Copy link
Member Author

@abonislawski please submit to main still on the side.

Main branch PR: #9747

@lgirdwood
Copy link
Member

MTL failure unrelated. Looks like IGK DUT config.

@lgirdwood lgirdwood merged commit 7ac89f8 into thesofproject:stable-v2.12 Dec 19, 2024
87 of 94 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants