aboutsummaryrefslogtreecommitdiffstats
path: root/CommonLibs/Sockets.h
diff options
context:
space:
mode:
authorTom Tsou <tom.tsou@ettus.com>2016-04-28 21:55:17 -0700
committerTom Tsou <tom.tsou@ettus.com>2016-05-02 17:37:05 -0700
commit2c650a6895f573e4455f55f0d1ed136ba8ae4744 (patch)
treeb0a1317a4cda61ba1d90dab064e1909a1f1057b9 /CommonLibs/Sockets.h
parentd4555f267e284d14e9e877f8f82da8bcc2d76d7a (diff)
common: Add mandatory length field to UDP receive calls
Current UDP receive reads up to MAX_UDP_LENGTH bytes into the passed in buffer, which may lead to buffer overflow if the write buffer is of insufficient size. Add mandatory length argument to UDP socket receive calls. Reported-by: Simone Margaritelli <simone@zimperium.com> Signed-off-by: Tom Tsou <tom.tsou@ettus.com>
Diffstat (limited to 'CommonLibs/Sockets.h')
-rw-r--r--CommonLibs/Sockets.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/CommonLibs/Sockets.h b/CommonLibs/Sockets.h
index c79f79a..0a70269 100644
--- a/CommonLibs/Sockets.h
+++ b/CommonLibs/Sockets.h
@@ -108,7 +108,7 @@ public:
@param buffer A char[MAX_UDP_LENGTH] procured by the caller.
@return The number of bytes received or -1 on non-blocking pass.
*/
- int read(char* buffer);
+ int read(char* buffer, size_t length);
/**
Receive a packet with a timeout.
@@ -116,7 +116,7 @@ public:
@param maximum wait time in milliseconds
@return The number of bytes received or -1 on timeout.
*/
- int read(char* buffer, unsigned timeout);
+ int read(char* buffer, size_t length, unsigned timeout);
/** Send a packet to a given destination, other than the default. */