[#297]fix:event params() not being freed in events.c

[Labreo]

Summary

• What: Fixed a memory leak in events.c by freeing eventParams before freeing the main EventNode.
• Motivation: When events are created, memory is allocated for both the event node and its specific parameters. The old cleanup function (freeEventList) only freed the outer node, leaving the parameters in memory. This fix cleans up both, preventing memory leaks during simulations.

How was this change tested?

make smoke
python tools/smoke_check.py run verbose <list tests/smoke subdirectories with changed outputs>

make test

Reproduction steps

Not required.

Related issues

• Fixes Bug:Memory leak in freeEventList(): eventParams not freed #297

Checklist

• Related issues are listed above. PRs without an approved, related issue may not get reviewed.
• PR title has the issue number in it ("[#] <concise description of proposed change>")
• Tests added/updated for new features (if applicable)
• Documentation updated (if applicable)
• docs/CHANGELOG.md updated with noteworthy changes
• Code formatted with clang-format (run git clang-format if needed)

---

Note: See CONTRIBUTING.md for additional guidance. This repository uses automated formatting checks; if the pre-commit hook blocks your commit, run git clang-format to format staged changes.

[Copilot]

Pull request overview

Fixes a memory leak in SIPNET’s agronomic events cleanup by ensuring each EventNode’s type-specific eventParams allocation is freed when tearing down the global event linked list.

Changes:

• Update freeEventList() to free(prev->eventParams) (when non-NULL) before freeing the EventNode.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.