pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcap/+/39366?usp=email )

Change subject: server: Correct error check calling osmo_pcap_conn_process_data() ......................................................................

server: Correct error check calling osmo_pcap_conn_process_data()

The function is not expected to return >0, so clarify it when calling it.

Change-Id: I8338b886db7bb7996229d78fe19025ebe5d0c5f8 --- M src/osmo_server_network.c 1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-pcap refs/changes/66/39366/1

diff --git a/src/osmo_server_network.c b/src/osmo_server_network.c index c93ce6d..5b31958 100644 --- a/src/osmo_server_network.c +++ b/src/osmo_server_network.c @@ -252,7 +252,7 @@ if ((rc = validate_link_data(conn, data)) < 0) return rc;

- if ((rc = osmo_pcap_conn_process_data(conn, &data->data[0], data->len))) + if ((rc = osmo_pcap_conn_process_data(conn, &data->data[0], data->len)) < 0) return rc; return 1; }