mirror of
https://github.com/open5gs/open5gs.git
synced 2025-10-23 07:41:57 +00:00
3a91d2aa3f
Prior to this change, `session_write_callback()` unconditionally asserted that `sbi_sess->poll.write` was non-NULL when the write queue drained, then removed it from the poll set. In edge cases—particularly when using curl 8.x with external epoll and `SIGPIPE` disabled—a late `EPOLLOUT` or errant write-ready notification could arrive after `poll.write` had already been cleared. This triggered the assertion in `nghttp2-server.c:1765`, aborting the process on shutdown or session teardown. This commit replaces the hard assertion with a runtime guard. If `poll.write` is present, it is removed and reset to NULL as before. If it is already NULL, we emit an warning log (`ogs_warn`) instead of aborting. This ensures any stray write events after cleanup are safely ignored, allowing a clean exit without crashing. - Wrap `ogs_pollset_remove()` and pointer clear in `if (sbi_sess->poll.write)` - Log an warning when `poll.write` is unexpectedly absent - Preserve original behavior when `poll.write` is valid This change resolves the fatal assertion observed on process exit after the EPOLLERR/SIGPIPE fix and improves overall shutdown robustness.