laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-remsim/+/28056 )

Change subject: rspro_client_fsm: Log clientConnectResult != ok ......................................................................

rspro_client_fsm: Log clientConnectResult != ok

This should aid debugging.

Related: SYS#5950 Change-Id: I8f584640758be0cceab61dcb333c81374da3288b --- M src/asn1c_helpers.h M src/rspro_client_fsm.c 2 files changed, 4 insertions(+), 0 deletions(-)

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

diff --git a/src/asn1c_helpers.h b/src/asn1c_helpers.h index 8d4e676..ac34a8f 100644 --- a/src/asn1c_helpers.h +++ b/src/asn1c_helpers.h @@ -1,5 +1,6 @@ #pragma once

+#include <asn_application.h> struct asn_TYPE_descriptor_t;

const char *asn_type_name(const asn_TYPE_descriptor_t *td); diff --git a/src/rspro_client_fsm.c b/src/rspro_client_fsm.c index bd07330..bd267ca 100644 --- a/src/rspro_client_fsm.c +++ b/src/rspro_client_fsm.c @@ -32,6 +32,7 @@ #include <osmocom/gsm/protocol/ipaccess.h>

#include "debug.h" +#include "asn1c_helpers.h" #include "rspro_client_fsm.h"

#define S(x) (1 << (x)) @@ -229,6 +230,8 @@ pdu = data; res = rspro_get_result(pdu); if (res != ResultCode_ok) { + LOGPFSML(fi, LOGL_ERROR, "Rx RSPRO connectClientRes(result=%s), closing\n", + asn_enum_name(&asn_DEF_ResultCode, res)); ipa_client_conn_close(srvc->conn); osmo_fsm_inst_dispatch(fi, SRVC_E_TCP_DOWN, NULL); } else {