Fix undefined behavior in memory allocation

[sebholt]

Pull Request Type

• GitHub Workflow changes
• Documentation or Wiki changes
• Build and Dependency changes
• Runtime changes
  • Render changes
  • Audio changes
  • Input changes
  • Network changes
  • Other changes

Description

Currently mem_rmalloc uses std::malloc for memory allocation and the deallocation is performed in mem_free
which calls ::free. This does seem to work but is undefined behavior because memory allocated with std::malloc
should be freed with std::free not ::free.

We should use either

• the C interface: ::malloc and ::free, or
• C++ interface: std::malloc and std::free.

The mem_malloc/mem_free functions already use the C interface so it makes sense to let mem_rmalloc simply call
mem_malloc instead of std::malloc directly. There's also no speed difference because everything is inline or a macro.

This changes the mem_rmalloc function to call mem_malloc instead of std::malloc to match the mem_free call.

Related Issues

Screenshots (if applicable)

Checklist

• I have tested my changes locally and verified that they work as intended.
• I have documented any new or modified functionality.
• I have reviewed the changes to ensure they do not introduce any unnecessary complexity or duplicate code.
• I understand that by submitting this pull request, I am agreeing to license my contributions under the project's license.

Additional Comments

[pzychotic]

LGTM
But since this code was just recently introduced by @jengelh, I would like to get his opinion before merging.

[sebholt]

Ok. I've updated the PR description to make the issue more clear.

[jengelh]

It's a sound argument.

[pzychotic]

Thanks both of you!