Skip to content

fix: pass report_id directly to get_data_set_extra#2900

Merged
greenbonebot merged 1 commit intomainfrom
fix/pass-report_id-extra
May 5, 2026
Merged

fix: pass report_id directly to get_data_set_extra#2900
greenbonebot merged 1 commit intomainfrom
fix/pass-report_id-extra

Conversation

@ozgen
Copy link
Copy Markdown
Member

@ozgen ozgen commented May 5, 2026

What

  • Fixed unnecessary g_strdup() calls in several get_report_* parsers.
  • The original attribute value is now passed directly to get_data_set_extra().
  • Updated the same issue in reports like ports, applications, operating systems, vulnerabilities, and hosts.

Why

  • get_data_set_extra() already duplicates the value internally.
  • Calling g_strdup() before passing the value caused memory leaks.
  • This fix keeps the same behavior and avoids the leaks.

References

GEA-1710

@ozgen ozgen requested review from a team as code owners May 5, 2026 12:47
@greenbonebot greenbonebot enabled auto-merge (rebase) May 5, 2026 12:47
@greenbonebot greenbonebot merged commit be1a0e0 into main May 5, 2026
27 checks passed
@greenbonebot greenbonebot deleted the fix/pass-report_id-extra branch May 5, 2026 12:53
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.

3 participants