about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-04-01 23:18:08 +0000
committerEric Wong <e@80x24.org>2015-04-19 22:10:42 +0000
commit793a3e5d3bee5de25bdfa01f32abff96d742b591 (patch)
tree87f69f6fbd15e85f8e21448d28df2790e5cd1c0a
parent6e1d6f91068b3b666bc592a0666b51a0593bc7d5 (diff)
downloadkgio-793a3e5d3bee5de25bdfa01f32abff96d742b591.tar.gz
Studying malloc implementations for a while, it's completely
reasonable for "free" to alter errno when making syscalls
to release/compact memory.
-rw-r--r--ext/kgio/writev.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/ext/kgio/writev.c b/ext/kgio/writev.c
index 027cdb5..d3ec53e 100644
--- a/ext/kgio/writev.c
+++ b/ext/kgio/writev.c
@@ -90,8 +90,7 @@ static ssize_t custom_writev(int fd, const struct iovec *vec, int iov_cnt, size_
 
         result = write(fd, buf, total_len);
 
-        /* well, it seems that `free` could not change errno
-         * but lets save it anyway */
+        /* free() may alter errno */
         i = errno;
         free(buf);
         errno = i;