X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=net.c;h=0e41be1c3fa804c772f4547a16ac68ec8b42e262;hp=a9708fcb51bcb88f1bd58aa525f42c1a22f9625d;hb=ba30bb8d0bb025c0dad80e5ff47352b4bc130423;hpb=9503f937af157874ef1ac598a5b6dfe41326a171

diff --git a/net.c b/net.c
index a9708fcb..0e41be1c 100644
--- a/net.c
+++ b/net.c
@@ -6,27 +6,29 @@
 
 /** \file net.c networking-related helper functions */
 
+#include "error.h"
 #include "para.h"
 #include "net.h"
 #include "string.h"
-#include "error.h"
 
 
-/** holds information about one encrypted connection */
+/** Information about one encrypted connection. */
 struct crypt_data {
-	/** function used to decrypt received data */
+	/** Function used to decrypt received data. */
 	crypt_function *recv;
-	/** function used to encrypt data to be sent */
+	/** Function used to encrypt data to be sent. */
 	crypt_function *send;
-	/** context-dependent data, passed to \a recv() and \a send() */
+	/**
+	 * Context-dependent data (crypt keys), passed verbatim to the above
+	 * crypt functions.
+	 */
 	void *private_data;
 };
-/** array holding per fd crypt data per */
+/** Array holding per fd crypt data. */
 static struct crypt_data *crypt_data_array;
-/** current size of the crypt data array */
+/** Current size of the crypt data array. */
 static unsigned cda_size = 0;
 
-
 /**
  * activate encryption for one file descriptor
  *
@@ -126,15 +128,15 @@ static int sendall(int fd, const char *buf, size_t *len)
 }
 
 /**
- * encrypt and send buffer
+ * Encrypt and send a binary buffer.
  *
- * \param fd:  the file descriptor
- * \param buf the buffer to be encrypted and sent
- * \param len the length of \a buf
+ * \param fd The file descriptor.
+ * \param buf The buffer to be encrypted and sent.
+ * \param len The length of \a buf.
  *
- * Check if encrytpion is available. If yes, encrypt the given buffer.  Send out
- * the buffer, encrypted or not, and try to resend the remaing part in case of
- * short writes.
+ * Check if encryption is available. If yes, encrypt the given buffer.  Send
+ * out the buffer, encrypted or not, and try to resend the remaing part in case
+ * of short writes.
  *
  * \return Positive on success, \p -E_SEND on errors.
  */
@@ -149,7 +151,8 @@ int send_bin_buffer(int fd, const char *buf, size_t len)
 		cf = crypt_data_array[fd].send;
 	if (cf) {
 		void *private = crypt_data_array[fd].private_data;
-		unsigned char *outbuf = para_malloc(len);
+		/* RC4 may write more than len to the output buffer */
+		unsigned char *outbuf = para_malloc(ROUND_UP(len, 8));
 		(*cf)(len, (unsigned char *)buf, outbuf, private);
 		ret = sendall(fd, (char *)outbuf, &len);
 		free(outbuf);
@@ -290,35 +293,15 @@ int get_host_info(char *host, struct hostent **ret)
  *
  * \sa socket(2)
  */
-int get_socket(void)
+int get_stream_socket(int domain)
 {
 	int socket_fd;
 
-	if ((socket_fd = socket(AF_INET, SOCK_STREAM, 0)) == -1)
+	if ((socket_fd = socket(domain, SOCK_STREAM, 0)) == -1)
 		return -E_SOCKET;
 	return socket_fd;
 }
 
-/**
- * a wrapper around connect(2)
- *
- * \param fd the file descriptor
- * \param their_addr the address to connect
- *
- * \return \p -E_CONNECT on errors, 1 on success
- *
- * \sa connect(2)
- */
-int para_connect(int fd, struct sockaddr_in *their_addr)
-{
-	int ret;
-
-	if ((ret = connect(fd, (struct sockaddr *)their_addr,
-			sizeof(struct sockaddr))) == -1)
-		return -E_CONNECT;
-	return 1;
-}
-
 /**
  * paraslash's wrapper around the accept system call
  *
@@ -540,7 +523,7 @@ int recv_cred_buffer(int fd, char *buf, size_t size)
  * \return The file descriptor of the created socket, negative
  * on errors.
  *
- * \sa get_socket()
+ * \sa get_stream_socket()
  * \sa setsockopt(2)
  * \sa bind(2)
  * \sa listen(2)
@@ -548,7 +531,7 @@ int recv_cred_buffer(int fd, char *buf, size_t size)
 int init_tcp_socket(int port)
 {
 	struct sockaddr_in my_addr;
-	int fd, ret = get_socket();
+	int fd, ret = get_stream_socket(AF_INET);
 
 	if (ret < 0)
 		return ret;