commit b683dcfe6bda723413b19c3bf5789d1a3bc3f7c7
author Vadim Yanitskiy <axilirator@gmail.com> Sat May 11 02:17:16 2019 +0700
committer Harald Welte <laforge@gnumonks.org> Sun May 26 09:27:22 2019 +0000
tree 1639f6ec722cf2f9e9054837b03715792fc001c6
parent a3cdab448121c76a56ef547ffdc61e3f58a93dbc

transaction: accept trans_type enum in trans_log_subsys()

Change-Id: I3c373d20ebd6e96ebd57f84b74dc15a6b69c03ac

src/libmsc/transaction.c

diff --git a/src/libmsc/transaction.c b/src/libmsc/transaction.c
index d6f8c3b..ebdaced 100644
--- a/src/libmsc/transaction.c
+++ b/src/libmsc/transaction.c
@@ -127,11 +127,12 @@
 			      enum trans_type type, uint8_t trans_id,
 			      uint32_t callref)
 {
-	struct gsm_trans *trans = NULL; /* (NULL for LOG_TRANS() before allocation) */
+	int subsys = trans_log_subsys(type);
+	struct gsm_trans *trans;
 
 	/* a valid subscriber is indispensable */
 	if (vsub == NULL) {
-		LOG_TRANS(trans, LOGL_ERROR, "unable to alloc transaction, invalid subscriber (NULL)\n");
+		LOGP(subsys, LOGL_ERROR, "unable to alloc transaction, invalid subscriber (NULL)\n");
 		return NULL;
 	}
 
@@ -142,11 +143,11 @@
 	*trans = (struct gsm_trans){
 		.vsub = vsub,
 		.type = type,
+		.log_subsys = subsys,
 		.transaction_id = trans_id,
 		.callref = callref,
 		.net = net,
 	};
-	trans->log_subsys = trans_log_subsys(trans);
 	vlr_subscr_get(vsub, trans_vsub_use(type));
 	llist_add_tail(&trans->entry, &net->trans_list);