Attention is currently required from: dexter.

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31577 )

Change subject: abis_rsl: assert maximum length ......................................................................

Patch Set 3:

(1 comment)

File src/osmo-bsc/abis_rsl.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/31577/comment/33323c21_2a72466b PS3, Line 948: val we have to be a bit carefult with assert's to avoid ending up in the sukchan-style of software development, where any (even unexpeted) protocol message can create a denial of service against the program. I didn't check who calls this function, but it looks like it's triggered from the RSL input path..

In general, ASSERT should be used to protect us against programming mistakes in this program (BSC). But not against unexpected messages received from other network elements. In that case we need to do runtime length checking + error handling