Attention is currently required from: pespin.

View Change

2 comments:

• File tests/osmo_io/osmo_io_test.c:

  • Patch Set #12, Line 99: for (int i = 0; i < 128; i++)

    why is this needed?

    In io_uring there is still a queue submitted for the read/receive call. After osmo_iofd_free() this read completes (with an error) / is aborted and we return the queue entry and actually free the osmo_io_fd.

    (Without this leak sanitizer reports a mem leak.)

  • Patch Set #12, Line 155: osmo_select_main(1);

    why is this needed?

    see above

To view, visit change 32536. To unsubscribe, or for help writing mail filters, visit settings.