From 11c676a4c30fda70f314529384d7094d30a3476f Mon Sep 17 00:00:00 2001
From: Spencer Sevilla <spencer.builds.networks@gmail.com>
Date: Thu, 4 Jan 2024 12:22:34 -0800
Subject: [PATCH] [pfcp] sendto should always return OGS_OK (#81)

sendto() was crashing the entire SMF when it received an EDESTADDRREQ error. True source of the error is unknown (maybe wireguard hiccupping? looks like the packet was sent out on the wrong wg tunnel) but regardless of the error we should actually just handle gracefully, log error, and move on. PFCP already has its own retrans/recovery mechanisms so loss here is irrelevant.
---
 lib/pfcp/path.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/pfcp/path.c b/lib/pfcp/path.c
index 58aa108589..5ad122befd 100644
--- a/lib/pfcp/path.c
+++ b/lib/pfcp/path.c
@@ -122,7 +122,6 @@ int ogs_pfcp_sendto(ogs_pfcp_node_t *node, ogs_pkbuf_t *pkbuf)
                     sock->fd, pkbuf->data, pkbuf->len,
                     OGS_ADDR(addr, buf), OGS_PORT(addr));
         }
-        return OGS_ERROR;
     }
 
     return OGS_OK;