lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/42823?usp=email )

Change subject: m3ua: fix missing decoding of message priority & network indicator flag ......................................................................

m3ua: fix missing decoding of message priority & network indicator flag

The missing parenthesis always or'ed with a 0x0 into the sio field because of operator precendence. Left shift is stronger than a binary and.

Related: SYS#8061 Change-Id: I7a07363646f66c5c672a2c0d261ca33356b43031 --- M src/m3ua.c 1 file changed, 2 insertions(+), 2 deletions(-)

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve daniel: Looks good to me, approved

diff --git a/src/m3ua.c b/src/m3ua.c index 243f5b9..583800b 100644 --- a/src/m3ua.c +++ b/src/m3ua.c @@ -324,8 +324,8 @@ param->sls = mdh->sls; /* re-construct SIO */ param->sio = (mdh->si & 0xF) | - (mdh->mp & 0x3 << 4) | - (mdh->ni & 0x3 << 6); + ((mdh->mp & 0x3) << 4) | + ((mdh->ni & 0x3) << 6); }

struct msgb *m3ua_msgb_alloc(const char *name)