fixeria submitted this change.

View Change

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
pfcp_peer: do not route incoming PDUs without SEID

Change-Id: I7d33ec4e883b217b2b4f65e4866d9365c32c753d
---
M src/pfcp_peer.erl
1 file changed, 3 insertions(+), 1 deletion(-)

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 change 37964. To unsubscribe, or for help writing mail filters, visit settings.

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 <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>