fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/38030?usp=email )


Change subject: pfcp_peer: fix incorrect order of logging arguments
......................................................................

pfcp_peer: fix incorrect order of logging arguments

Change-Id: I4c2326d7143e8d4035d75cc60b53fc8cf33611cb
---
M src/pfcp_peer.erl
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-s1gw 
refs/changes/30/38030/1

diff --git a/src/pfcp_peer.erl b/src/pfcp_peer.erl
index 8c29a9f..c97304b 100644
--- a/src/pfcp_peer.erl
+++ b/src/pfcp_peer.erl
@@ -303,10 +303,10 @@
 -spec decode_pdu(binary(), tuple(), peer_state()) -> pfcp_pdu().
 decode_pdu(Data, {FromIp, FromPort}, _S) ->
     ?LOG_DEBUG("Rx encoded PFCP PDU from ~p:~p: ~p",
-               [Data, FromIp, FromPort]),
+               [FromIp, FromPort, Data]),
     PDU = pfcp_packet:decode(Data),
     ?LOG_DEBUG("Rx PFCP PDU from ~p:~p: ~p",
-               [PDU, FromIp, FromPort]),
+               [FromIp, FromPort, PDU]),
     %% TODO: check PDU#pfcp.seq_no
     %% TODO: there can be batched PDUs
     PDU.

--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/38030?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I4c2326d7143e8d4035d75cc60b53fc8cf33611cb
Gerrit-Change-Number: 38030
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>

Reply via email to