Attention is currently required from: neels.

laforge has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/osmo-upf/+/37824?usp=email )

Change subject: fix msgb memleak on GTP echo response ......................................................................

Patch Set 1: Code-Review+1

(1 comment)

File src/osmo-upf/upf_gtpu_echo.c:

https://gerrit.osmocom.org/c/osmo-upf/+/37824/comment/81f63d2f_8da0a9c0?usp=... : PS1, Line 91: sendto this is of course a valid fix. Seeing all those hand-rolled socket operations, I'm wondering if it wouldn't make sense to migrate this over to osmo_io. For sure it's not a performance critical code path - but it would avoid having to manually have a read_cb, reading from the socket, etc.