Attention is currently required from: pespin.
arehbein has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/34799?usp=email )
Change subject: stream_test: Improve mem mgmt, fix connection mgmt ......................................................................
Patch Set 3:
(3 comments)
File tests/stream/stream_test.c:
https://gerrit.osmocom.org/c/libosmo-netif/+/34799/comment/83301c5d_16fff92c PS3, Line 549: osmo_stream_cli_set_data(osc, ctx); @pespin@sysmocom.de removal of instruction to set data
https://gerrit.osmocom.org/c/libosmo-netif/+/34799/comment/eccf3c5e_ac5540a5 PS3, Line 798: osmo_stream_cli_set_data(osc, ctx); @pespin@sysmocom.de removal of instruction to set data
File tests/stream/stream_test.ok:
https://gerrit.osmocom.org/c/libosmo-netif/+/34799/comment/01da628c_a1cd3d7f PS3, Line 52: {20.000020} [NA] Client's test_segm_ipa_stream_srv_cli_connect_cb(): [1-cli] Appending msg of type IPAC_MSGT_PING into buffer
why this this output change?
It's due to the change of memory management in the test. Before the patch, the client side would set newly created talloc contexts as data for the client connection, this has been removed.
I have tagged you in the respective lines.