Add pool callback

[JonathanBerrew]

This is a Marc Stern modification, I don't have much more insight on the code he made. To be reviewed with caution and check if this is still relevant

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull request overview

This PR updates request-body pool initialization to propagate the parent pool’s APR abort callback to the newly created msc_reqbody_mp pool.

Changes:

• Set msc_reqbody_mp’s abort function to match msr->mp via apr_pool_abort_set(apr_pool_abort_get(msr->mp), ...).

---

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

[Copilot]

apr_pool_create(&msr->msc_reqbody_mp, NULL); returns an apr_status_t, but the result isn’t checked before calling apr_pool_abort_set(...) (and later using msr->msc_reqbody_mp). If pool creation fails, this will immediately dereference a NULL pool. Capture the return code, and on failure set *error_msg and return an error before using msr->msc_reqbody_mp.