Store iovec array on the stack instead of heap allocating it for the lifetime of writev() in StreamSocket::DoWrite()

1 files changed, 9 insertions, 7 deletions

diff --git a/src/inspsocket.cpp b/src/inspsocket.cpp
index 645947f56..ee5287e5f 100644
--- a/src/inspsocket.cpp
+++ b/src/inspsocket.cpp
@@ -339,15 +339,17 @@ void StreamSocket::DoWrite()
 			}
 
 			int rv_max = 0;
-			iovec* iovecs = new iovec[bufcount];
-			for(int i=0; i < bufcount; i++)
+			int rv;
 			{
-				iovecs[i].iov_base = const_cast<char*>(sendq[i].data());
-				iovecs[i].iov_len = sendq[i].length();
-				rv_max += sendq[i].length();
+				iovec iovecs[MYIOV_MAX];
+				for (int i = 0; i < bufcount; i++)
+				{
+					iovecs[i].iov_base = const_cast<char*>(sendq[i].data());
+					iovecs[i].iov_len = sendq[i].length();
+					rv_max += sendq[i].length();
+				}
+				rv = writev(fd, iovecs, bufcount);
 			}
-			int rv = writev(fd, iovecs, bufcount);
-			delete[] iovecs;
 
 			if (rv == (int)sendq_len)
 			{