Added Tor

ports/glibc-compat/src/readv.c

+/* Copyright 2015 The Native Client Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file. */
+
+#include <sys/uio.h>
+#include <unistd.h> // unix(nacl) standards
+#include <errno.h>
+
+ssize_t readv(int fd, const struct iovec *iov, int iovcnt){

[binji, 2015/08/24 22:28:00]

nit: add space between ) and {

here and elsewhere

[deepankar-tyagi, 2015/08/25 18:14:43]

Done.

+  if(iovcnt < 0){

[binji, 2015/08/24 22:28:01]

nit: space after if

[deepankar-tyagi, 2015/08/25 18:14:43]

Done.

+    // #TODO(dt) add check for IOV_MAX
+    errno = EINVAL;
+    return -1;
+  }
+  ssize_t bytes_read = 0 ;

[binji, 2015/08/24 22:28:01]

nit: remove space before ;

[deepankar-tyagi, 2015/08/25 18:14:43]

Done.

+  ssize_t bytes_supposed_to_be_read = 0 ;
+  int i;
+  for (i = 0; i < (iovcnt); i++) {
+    bytes_read +=  read(fd, iov[i].iov_base, iov[i].iov_len);

[binji, 2015/08/24 22:28:00]

nit: remove double space

[deepankar-tyagi, 2015/08/25 18:14:43]

Done.

+    bytes_supposed_to_be_read += iov[i].iov_len ;
+    // #TODO(dt) add check for (max) ssize_t allowed
+    if (bytes_read != bytes_supposed_to_be_read){
+      // deliberately not setting errno here as it might override
+      // errno set by read(...)
+      return -1;
+    }
+  }
+  // #TODO(dt) (mark for)update st_atime if required.

[binji, 2015/08/24 22:28:00]

shouldn't be necessary -- call to read(...) above should have done it, right?

[deepankar-tyagi, 2015/08/25 18:14:43]

haven't looked at read(...) implementation yet, for now removed the comment.But
I think it shouldn't be a concern

+  return bytes_read;
+}