Fix util_abort called with too few arguments

[eivindjahren]

Issue
Resolves https://github.com/equinor/resdata/security/code-scanning/9 and two similar security reports

[Copilot]

Pull request overview

Fixes format-string argument mismatches in a couple of util_abort() call sites to address CodeQL/security scanning findings (and prevent undefined behavior when aborting).

Changes:

• Add missing __func__ argument to util_abort() in well_segment_link_strict().
• Add missing __func__ argument to util_abort() in two grid keyword copy abort paths.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
lib/resdata/well_segment.cpp	Fixes %s format specifier mismatch by passing __func__ to util_abort() in strict segment linking failure path.
lib/resdata/rd_grid.cpp	Fixes %s format specifier mismatch by passing __func__ to util_abort() in two size-mismatch abort paths.

---

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

[ajaust]

Looks good.

Should we -Wformat to the default compile options? This might be able to detect such errors at compile time.

[eivindjahren]

@ajaust we have -Wall (which gives -Wformat=1), but not -WError and there are a number of warnings. We should probably get that fixed also.