From d616f6d0dfd7aee9af76aad2b46a749dc3eee87d 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;