Change in ...osmocom-bb[master]: trxcon/scheduer: fix: properly check rc of gsm0503_pdtch_encode()

[fixeria]

fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14638


Change subject: trxcon/scheduer: fix: properly check rc of 
gsm0503_pdtch_encode()
......................................................................

trxcon/scheduer: fix: properly check rc of gsm0503_pdtch_encode()

The gsm0503_pdtch_encode() returns negative number on error,
and the amount of encoded bits in case of success.

Change-Id: I7d75141142922909330c5e86be8734bb06cd57a4
---
M src/host/trxcon/sched_lchan_pdtch.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/38/14638/1

diff --git a/src/host/trxcon/sched_lchan_pdtch.c 
b/src/host/trxcon/sched_lchan_pdtch.c
index ad021a9..733e574 100644
--- a/src/host/trxcon/sched_lchan_pdtch.c
+++ b/src/host/trxcon/sched_lchan_pdtch.c
@@ -145,7 +145,7 @@
        /* Encode payload */
        rc = gsm0503_pdtch_encode(buffer, lchan->prim->payload,
                lchan->prim->payload_len);
-       if (rc) {
+       if (rc < 0) {
                LOGP(DSCHD, LOGL_ERROR, "Failed to encode L2 payload (len=%zu): 
%s\n",
                     lchan->prim->payload_len, 
osmo_hexdump(lchan->prim->payload,
                                                            
lchan->prim->payload_len));

--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/14638
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7d75141142922909330c5e86be8734bb06cd57a4
Gerrit-Change-Number: 14638
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilira...@gmail.com>
Gerrit-MessageType: newchange