X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fcommon%2Freadwrite.c;h=42a048fdabd2d3abaf0e32d561f9cb901930baff;hp=a559de76b1ea5f85387ab178eee47a21b7f101f1;hb=a3eae3c98f047a8e795a01471adf8f4f7563947b;hpb=6cd525e813795a1d5e38feac8dedf2c73ffb1274

diff --git a/src/common/readwrite.c b/src/common/readwrite.c
index a559de76b..42a048fda 100644
--- a/src/common/readwrite.c
+++ b/src/common/readwrite.c
@@ -15,9 +15,12 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include <unistd.h>
-#include <errno.h>
+#define _LGPL_SOURCE
 #include <assert.h>
+#include <errno.h>
+#include <limits.h>
+#include <unistd.h>
+
 #include "readwrite.h"
 
 /*
@@ -28,11 +31,22 @@
  * error occured.
  * The error can be checked by querying errno.
  */
+LTTNG_HIDDEN
 ssize_t lttng_read(int fd, void *buf, size_t count)
 {
 	size_t i = 0;
 	ssize_t ret;
 
+	assert(buf);
+
+	/*
+	 * Deny a read count that can be bigger then the returned value max size.
+	 * This makes the function to never return an overflow value.
+	 */
+	if (count > SSIZE_MAX) {
+		return -EINVAL;
+	}
+
 	do {
 		ret = read(fd, buf + i, count - i);
 		if (ret < 0) {
@@ -55,11 +69,22 @@ error:
 	}
 }
 
+LTTNG_HIDDEN
 ssize_t lttng_write(int fd, const void *buf, size_t count)
 {
 	size_t i = 0;
 	ssize_t ret;
 
+	assert(buf);
+
+	/*
+	 * Deny a write count that can be bigger then the returned value max size.
+	 * This makes the function to never return an overflow value.
+	 */
+	if (count > SSIZE_MAX) {
+		return -EINVAL;
+	}
+
 	do {
 		ret = write(fd, buf + i, count - i);
 		if (ret < 0) {