laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31954 ) Change subject: isdn: fix identical operands in v110_adapt_IR8000_to_2400() ...................................................................... isdn: fix identical operands in v110_adapt_IR8000_to_2400() Change-Id: I7df8b214e813cbd14726d5043f666547353e2947 Fixes: CID#310966 --- M src/isdn/v110.c 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/src/isdn/v110.c b/src/isdn/v110.c index 4d13547..46962bf 100644 --- a/src/isdn/v110.c +++ b/src/isdn/v110.c @@ -242,7 +242,8 @@ if (out_len < 24) return -ENOSPC; - if (fr->e_bits[1] != 1 || fr->e_bits[1] != 1 || fr->e_bits[2] != 0) + /* Table 6c / V.110 */ + if (fr->e_bits[0] != 1 || fr->e_bits[1] != 1 || fr->e_bits[2] != 0) return -EINVAL; for (int i = 0; i < 24; i++) { -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31954 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I7df8b214e813cbd14726d5043f666547353e2947 Gerrit-Change-Number: 31954 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged