Attention is currently required from: laforge.

dexter has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/libosmocore/+/36324?usp=email )

Change subject: msgb: Also copy msgb {dst, lchan, cb} during msgb_copy_* ......................................................................

Patch Set 2: Code-Review+1

(1 comment)

File src/core/msgb.c:

https://gerrit.osmocom.org/c/libosmocore/+/36324/comment/63c57054_58af55e5?u... : PS1, Line 320: * and adjusts the pointers (incl l1h-l4h) accordingly. The cb part is not copied.

I think the cb was not copied as a performance reason, since lots of code doesn't use cb at all. […]

I wonder how much the performance impact of copying the cb buffer would be. The additional effort should be neglectable. While the function was specified not to copy the cb, it thereis nothing said about msg->lchan and msg->dst. Maybe we could just drop the memcpy line?