Skip to content

Conversation

@WWeiOne
Copy link
Contributor

@WWeiOne WWeiOne commented May 27, 2025

This issue #117 will trigger double free #89, which usually occurs in large topologies and workloads
double free or corruption (!prev)
double free or corruption (out)

previous commit based on #127

@WWeiOne WWeiOne changed the title Fix double free [Fix] double free May 27, 2025
@WWeiOne WWeiOne changed the title [Fix] double free [Fix] Double free May 27, 2025
@HeRaNO
Copy link
Contributor

HeRaNO commented May 29, 2025

7210777 can also close #6. However, in #14 @Huoyuan100861 argues that the fix might affect the correctness of the generated tree node results.

@WWeiOne
Copy link
Contributor Author

WWeiOne commented May 30, 2025

@Huoyuan100861 Could you provide some information about the validation? #14

@zyksir zyksir requested review from Huoyuan100861 and zyksir May 30, 2025 03:43
@zyksir zyksir self-assigned this May 30, 2025
@zyksir
Copy link
Collaborator

zyksir commented May 30, 2025

Well Done! @WWeiOne
Since this commit is based on #127, maybe we should check and merge that PR first? @Huoyuan100861

@Huoyuan100861 Huoyuan100861 merged commit a920cdc into aliyun:master Jul 9, 2025
1 check passed
AndreaRaffaelli pushed a commit to AndreaRaffaelli/SimAI that referenced this pull request Oct 21, 2025
* [Fix] network_frontend ns3 integer overflow

* [Fix] double free
gallomassimo pushed a commit to NetMeasurements-Team/SimAI that referenced this pull request Dec 8, 2025
* [Fix] network_frontend ns3 integer overflow

* [Fix] double free
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