Attention is currently required from: pespin.

falconia has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/32988 )

Change subject: refactor: replace rtppayload_is_valid() with preening before enqueue ......................................................................

Patch Set 3:

(2 comments)

File src/common/l1sap.c:

https://gerrit.osmocom.org/c/osmo-bts/+/32988/comment/6fb30cbf_dd458549 PS2, Line 1891: case PL_DECISION_ACCEPT_ASIS:

I'd go for PL_DECISION_ACCEPT. Sounds shorter and more familiar to other systems (eg. iptables).

Done

File src/common/rtp_input_preen.c:

https://gerrit.osmocom.org/c/osmo-bts/+/32988/comment/e0e1fda7_1bc49a8e PS2, Line 37: static bool rtppayload_is_octet_aligned(const uint8_t *rtp_pl, unsigned payload_len)

Adding it to my mental notes for the next version.

Done