osmo-msc[master]: libmsc: bssap: Fix compilation warning
Pau Espin Pedrol
gerrit-no-reply at lists.osmocom.org
Thu Feb 15 18:35:52 UTC 2018
Patch Set 4:
Line 313: memcpy(msg->l3h, TLVP_VAL(tp, GSM0808_IE_LAYER_3_INFORMATION), msgb_l3len(msg));
> I still don't get why we would want to copy the payload here. The point of
As the FIXME comment indicates, current code seems a bit hackish, like modifying the tail directly and probably creating a gap between lower layers and l3h.
Now that I think about it, it would even make more sense to use memmove instead?
What do you propose otherwise? just dropping the FIXME comment and leave it as it is now?
To view, visit https://gerrit.osmocom.org/5424
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Stefan Sperling <ssperling at sysmocom.de>
More information about the gerrit-log