fixeria has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/37964?usp=email )
Change subject: pfcp_peer: do not route incoming PDUs without SEID
......................................................................
pfcp_peer: do not route incoming PDUs without SEID
Change-Id: I7d33ec4e883b217b2b4f65e4866d9365c32c753d
---
M src/pfcp_peer.erl
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
pespin: Looks good to me, but someone else must approve
fixeria: Looks good to me, approved
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
diff --git a/src/pfcp_peer.erl b/src/pfcp_peer.erl
index 0479071..cc98c49 100644
--- a/src/pfcp_peer.erl
+++ b/src/pfcp_peer.erl
@@ -238,8 +238,10 @@
%% 3GPP TS 29.244, 6.2.2.2 Heartbeat Request
#pfcp{type = heartbeat_request} ->
recv_heartbeat_request(PDU, {FromIp, FromPort}, S);
+ #pfcp{seid = SEID} when is_integer(SEID) ->
+ route_pdu(PDU, S);
_ ->
- route_pdu(PDU, S)
+ ?LOG_NOTICE("Rx unexpected PFCP PDU: ~p", [PDU])
end,
{keep_state, S};
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/37964?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I7d33ec4e883b217b2b4f65e4866d9365c32c753d
Gerrit-Change-Number: 37964
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
