From ff1a5dc75152fb72cd4597036f9c6c712f1c1436 Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Thu, 3 Dec 2015 19:32:04 +0100 Subject: server: Deal with jumbo frames on the wire The 2000 as a number is too small. Modern networks can have a higher MTU (up to 9000). Take this number and assume there is a big header in front of it. --- include/osmo-pcap/osmo_pcap_server.h | 6 +++++- src/osmo_server_network.c | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/include/osmo-pcap/osmo_pcap_server.h b/include/osmo-pcap/osmo_pcap_server.h index 1f2f6fc..edb29d3 100644 --- a/include/osmo-pcap/osmo_pcap_server.h +++ b/include/osmo-pcap/osmo_pcap_server.h @@ -23,6 +23,8 @@ #ifndef OSMO_PCAP_SERVER_H #define OSMO_PCAP_SERVER_H +#include "wireformat.h" + #include #include @@ -40,6 +42,8 @@ struct osmo_pcap_server; #define STATE_INITIAL 0 #define STATE_DATA 1 +#define SERVER_MAX_DATA_SIZE 10000 + struct osmo_pcap_conn { /* list of connections */ struct llist_head entry; @@ -64,7 +68,7 @@ struct osmo_pcap_conn { int state; int pend; int reopen; - char buf[4096]; + char buf[SERVER_MAX_DATA_SIZE + sizeof(struct osmo_pcap_data)]; struct osmo_pcap_data *data; }; diff --git a/src/osmo_server_network.c b/src/osmo_server_network.c index 78cda94..7127401 100644 --- a/src/osmo_server_network.c +++ b/src/osmo_server_network.c @@ -199,7 +199,7 @@ static int read_cb_initial(struct osmo_fd *fd, struct osmo_pcap_conn *conn) } else if (conn->pend == 0) { conn->data->len = ntohs(conn->data->len); - if (conn->data->len > 2000) { + if (conn->data->len > SERVER_MAX_DATA_SIZE) { LOGP(DSERVER, LOGL_ERROR, "Implausible data length: %u\n", conn->data->len); close_connection(conn); -- cgit v1.2.3