From ec8bf5c8c12e1784e619b090b87ca567b199ec71 Mon Sep 17 00:00:00 2001 From: Masayuki Ishikawa Date: Fri, 29 Jan 2021 13:34:52 +0900 Subject: [PATCH] net: tcp: Add DEBUGASSERT() in psock_tcp_send() Summary: - This commit adds DEBUGASSERT() to check the IOB size Impact: - None Testing: - Tested with sabre-6quad:netnsh with QEMU Signed-off-by: Masayuki Ishikawa --- net/tcp/tcp_send_buffered.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/net/tcp/tcp_send_buffered.c b/net/tcp/tcp_send_buffered.c index 51c36a6ef67..f99c7487e18 100644 --- a/net/tcp/tcp_send_buffered.c +++ b/net/tcp/tcp_send_buffered.c @@ -1144,6 +1144,11 @@ ssize_t psock_tcp_send(FAR struct socket *psock, FAR const void *buf, */ blresult = net_breaklock(&count); + + /* NOTE: At least IOB needs to hold the packet */ + + DEBUGASSERT((CONFIG_IOB_NBUFFERS * CONFIG_IOB_BUFSIZE) > len); + result = TCP_WBCOPYIN(wrb, (FAR uint8_t *)buf, len); if (blresult >= 0) {