Attention is currently required from: jolly.

View Change

2 comments:

• File src/host/layer23/include/osmocom/bb/mobile/gsm48_rr.h:

  • Patch Set #5, Line 65: struct gsm48_chan_desc ch_desc;

    did you check all users of this struct to make sure eveyrthing is updated? Is this field always present in all contexts where this struct is used?

• File src/host/layer23/src/mobile/gsm48_mm.c:

  • Patch Set #5, Line 882: memcpy(&rrh->ch_desc, ch_desc, sizeof(rrh->ch_desc));

    you may want to memzero the rrh_ch_desc here if ch_desc=NULL

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