Attention is currently required from: osmith.

View Change

1 comment:

• File src/osmo-bsc-nat/bssap_conn.c:

  • Patch Set #2, Line 60: len = IP_V4_ADDR_LEN;

    Thanks for the suggestion! […]

    AFAIR and from waht you mention, msgb_put checks length by calling osmo_panic(), which is not desirable. That osmo_panic is just to signal that something is really wrong in the caller code. So we want proper checking here, not just osmo_panic.

    I still think the proper way is to handle one check inside gsm0808_enc_aoip_trasp_addr() and another one directly in this function for the path calling tlv_encode_one(). Because for the first path, you always want to have that checked, and I'd expect that function to check that. For the other path, you are really doing different stuff and it's fine doing the check in the same function.

To view, visit change 28582. To unsubscribe, or for help writing mail filters, visit settings.