Attention is currently required from: dexter, fixeria, neels, pespin.

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

Change subject: Change msgb ownership in processing of received msgb ......................................................................

Patch Set 2:

(4 comments)

File src/libosmo-mgcp/mgcp_network.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/36361/comment/ad301aa3_943cce0b PS2, Line 1191: return rc

missing `msgb_free()`?

Done

https://gerrit.osmocom.org/c/osmo-mgw/+/36361/comment/19047f56_d2d419d4 PS2, Line 1205: return rc

missing `msgb_free()`?

Done

https://gerrit.osmocom.org/c/osmo-mgw/+/36361/comment/5da3c860_c084573e PS2, Line 1212: return rc

missing `msgb_free()`?

Done

File src/libosmo-mgcp/mgcp_osmux.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/36361/comment/e161bcb9_20b45499 PS2, Line 217: return -1;

missing `msgb_free()` here and below? AFAIU, if a function claims to take ownership, then it's also […]

Done