removed some debugs
[citadel.git] / citadel / clientsocket.c
index a4b521864087357dff41622fd57e027be94bae25..a88c614d91240409266e9d461b69f58450cf17f6 100644 (file)
@@ -1,55 +1,26 @@
-/*
- * This module handles client-side sockets opened by the Citadel server (for
- * the client side of Internet protocols, etc.)   It does _not_ handle client
- * sockets for the Citadel client; for that you must look in ipc_c_tcp.c
- * (which, uncoincidentally, bears a striking similarity to this file).
- *
- * Copyright (c) 1987-2011 by the citadel.org team
- *
- * This program is open source software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as published
- * by the Free Software Foundation; either version 3 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- */
-
-#include "sysdep.h"
+// This module handles client-side sockets opened by the Citadel server (for
+// the client side of Internet protocols, etc.)   It does _not_ handle client
+// sockets for the Citadel client; for that you must look in ipc_c_tcp.c
+// (which, uncoincidentally, bears a striking similarity to this file).
+//
+// Copyright (c) 1987-2017 by the citadel.org team
+//
+// This program is open source software.  Use, duplication, or disclosure
+// is subject to the terms of the GNU General Public License, version 3.
+// The program is distributed without any warranty, expressed or implied.
+
 #include <stdlib.h>
 #include <unistd.h>
+#include <netdb.h>
 #include <stdio.h>
-#include <signal.h>
-#include <sys/types.h>
-#include <sys/time.h>
+#ifdef __FreeBSD__
 #include <sys/socket.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
-#include <netdb.h>
-#include <string.h>
-#include <pwd.h>
-#include <errno.h>
-#include <stdarg.h>
-#include <syslog.h>
-#include <libcitadel.h>
-#include "citadel.h"
-#include "server.h"
-#ifndef HAVE_SNPRINTF
-#include "snprintf.h"
 #endif
-#include "sysdep_decls.h"
-#include "config.h"
-#include "clientsocket.h"
+#include <libcitadel.h>
 #include "ctdl_module.h"
+#include "clientsocket.h"
 
-int sock_connect(char *host, char *service)
-{
+int sock_connect(char *host, char *service) {
        struct in6_addr serveraddr;
        struct addrinfo hints;
        struct addrinfo *res = NULL;
@@ -67,36 +38,32 @@ int sock_connect(char *host, char *service)
        hints.ai_family = AF_UNSPEC;
        hints.ai_socktype = SOCK_STREAM;
 
-       /*
-        * Handle numeric IPv4 and IPv6 addresses
-        */
+       // Handle numeric IPv4 and IPv6 addresses
        rc = inet_pton(AF_INET, host, &serveraddr);
-       if (rc == 1) {                                          /* dotted quad */
+       if (rc == 1) {                                          // dotted quad
                hints.ai_family = AF_INET;
                hints.ai_flags |= AI_NUMERICHOST;
-       } else {
+       }
+       else {
                rc = inet_pton(AF_INET6, host, &serveraddr);
-               if (rc == 1) {                                  /* IPv6 address */
+               if (rc == 1) {                                  // IPv6 address
                        hints.ai_family = AF_INET6;
                        hints.ai_flags |= AI_NUMERICHOST;
                }
        }
 
-       /* Begin the connection process */
-
+       // Begin the connection process
        rc = getaddrinfo(host, service, &hints, &res);
        if (rc != 0) {
                syslog(LOG_ERR, "%s: %s", host, gai_strerror(rc));
                return(-1);
        }
 
-       /*
-        * Try all available addresses until we connect to one or until we run out.
-        */
+       // Try all available addresses until we connect to one or until we run out.
        for (ai = res; ai != NULL; ai = ai->ai_next) {
                sock = socket(ai->ai_family, ai->ai_socktype, ai->ai_protocol);
                if (sock < 0) {
-                       syslog(LOG_ERR, "socket() failed: %s", strerror(errno));
+                       syslog(LOG_ERR, "%s: %m", host);
                        freeaddrinfo(res);
                        return(-1);
                }
@@ -106,7 +73,7 @@ int sock_connect(char *host, char *service)
                        return(sock);
                }
                else {
-                       syslog(LOG_ERR, "connect() failed: %s", strerror(errno));
+                       syslog(LOG_ERR, "%s: %m", host);
                        close(sock);
                }
        }
@@ -115,76 +82,59 @@ int sock_connect(char *host, char *service)
 }
 
 
-
-/*
- * Read data from the client socket.
- *
- * sock                socket fd to read from
- * buf         buffer to read into 
- * bytes       number of bytes to read
- * timeout     Number of seconds to wait before timing out
- *
- * Possible return values:
- *      1       Requested number of bytes has been read.
- *      0       Request timed out.
- *     -1      Connection is broken, or other error.
- */
-int socket_read_blob(int *Socket, StrBuf * Target, int bytes, int timeout)
-{
-       CitContext *CCC = MyContext();
+// Read data from the client socket.
+//
+// sock                socket fd to read from
+// buf         buffer to read into 
+// bytes       number of bytes to read
+// timeout     Number of seconds to wait before timing out
+//
+// Possible return values:
+//      1       Requested number of bytes has been read.
+//      0       Request timed out.
+//     -1      Connection is broken, or other error.
+int socket_read_blob(int *Socket, StrBuf *Target, int bytes, int timeout) {
        const char *Error;
        int retval = 0;
 
-
-       retval = StrBufReadBLOBBuffered(Target,
-                                       CCC->SBuf.Buf,
-                                       &CCC->SBuf.ReadWritePointer,
-                                       Socket, 1, bytes, O_TERM, &Error);
-       
+       retval = StrBufReadBLOBBuffered(Target, CC->SBuf.Buf, &CC->SBuf.ReadWritePointer, Socket, 1, bytes, O_TERM, &Error); 
        if (retval < 0) {
-               syslog(LOG_CRIT, "socket_read_blob() failed: %s", Error);
+               syslog(LOG_ERR, "clientsocket: socket_read_blob() failed: %s", Error);
        }
        return retval;
 }
 
 
-int CtdlSockGetLine(int *sock, StrBuf * Target, int nSec)
-{
-       CitContext *CCC = MyContext();
+int CtdlSockGetLine(int *sock, StrBuf *Target, int nSec) {
        const char *Error;
        int rc;
 
        FlushStrBuf(Target);
        rc = StrBufTCP_read_buffered_line_fast(Target,
-                                              CCC->SBuf.Buf,
-                                              &CCC->SBuf.ReadWritePointer,
+                                              CC->SBuf.Buf,
+                                              &CC->SBuf.ReadWritePointer,
                                               sock, nSec, 1, &Error);
-       if ((rc < 0) && (Error != NULL))
-               syslog(LOG_CRIT, "CtdlSockGetLine() failed: %s", Error);
+       if ((rc < 0) && (Error != NULL)) {
+               syslog(LOG_ERR, "clientsocket: CtdlSockGetLine() failed: %s", Error);
+       }
        return rc;
 }
 
 
-/*
- * client_getln()   ...   Get a LF-terminated line of text from the client.
- * (This is implemented in terms of client_read() and could be
- * justifiably moved out of sysdep.c)
- */
-int sock_getln(int *sock, char *buf, int bufsize)
-{
+// client_getln()   ...   Get a LF-terminated line of text from the client.
+int sock_getln(int *sock, char *buf, int bufsize) {
        int i, retval;
-       CitContext *CCC = MyContext();
        const char *pCh;
 
-       FlushStrBuf(CCC->sMigrateBuf);
-       retval = CtdlSockGetLine(sock, CCC->sMigrateBuf, 5);
+       FlushStrBuf(CC->sMigrateBuf);
+       retval = CtdlSockGetLine(sock, CC->sMigrateBuf, 5);
 
-       i = StrLength(CCC->sMigrateBuf);
-       pCh = ChrPtr(CCC->sMigrateBuf);
+       i = StrLength(CC->sMigrateBuf);
+       pCh = ChrPtr(CC->sMigrateBuf);
 
        memcpy(buf, pCh, i + 1);
 
-       FlushStrBuf(CCC->sMigrateBuf);
+       FlushStrBuf(CC->sMigrateBuf);
        if (retval < 0) {
                safestrncpy(&buf[i], "000", bufsize - i);
                i += 3;
@@ -193,14 +143,14 @@ int sock_getln(int *sock, char *buf, int bufsize)
 }
 
 
-/*
- * sock_write() - send binary to server.
- * Returns the number of bytes written, or -1 for error.
- */
-int sock_write(int *sock, const char *buf, int nbytes) 
-{ return sock_write_timeout(sock, buf, nbytes, 50); }
-int sock_write_timeout(int *sock, const char *buf, int nbytes, int timeout)
-{
+// sock_write() - send binary to server.
+// Returns the number of bytes written, or -1 for error.
+int sock_write(int *sock, const char *buf, int nbytes) {
+       return sock_write_timeout(sock, buf, nbytes, 50);
+}
+
+
+int sock_write_timeout(int *sock, const char *buf, int nbytes, int timeout) {
        int nSuccessLess = 0;
        int bytes_written = 0;
        int retval;
@@ -213,18 +163,14 @@ int sock_write_timeout(int *sock, const char *buf, int nbytes, int timeout)
        fdflags = fcntl(*sock, F_GETFL);
        IsNonBlock = (fdflags & O_NONBLOCK) == O_NONBLOCK;
 
-       while ((nSuccessLess < timeout) && 
-              (*sock != -1) && 
-              (bytes_written < nbytes)) 
-       {
-               if (IsNonBlock){
+       while ((nSuccessLess < timeout) && (*sock != -1) && (bytes_written < nbytes)) {
+               if (IsNonBlock) {
                        tv.tv_sec = selectresolution;
                        tv.tv_usec = 0;
                        
                        FD_ZERO(&rfds);
                        FD_SET(*sock, &rfds);
                        if (select(*sock + 1, NULL, &rfds, NULL, &tv) == -1) {
-///                            *Error = strerror(errno);
                                close (*sock);
                                *sock = -1;
                                return -1;
@@ -249,7 +195,6 @@ int sock_write_timeout(int *sock, const char *buf, int nbytes, int timeout)
                        FD_ZERO(&rfds);
                        FD_SET(*sock, &rfds);
                        if (select(*sock + 1, NULL, &rfds, NULL, &tv) == -1) {
-///                            *Error = strerror(errno);
                                close (*sock);
                                *sock = -1;
                                return -1;
@@ -260,27 +205,20 @@ int sock_write_timeout(int *sock, const char *buf, int nbytes, int timeout)
 }
 
 
-
-/*
- * client_getln()   ...   Get a LF-terminated line of text from the client.
- * (This is implemented in terms of client_read() and could be
- * justifiably moved out of sysdep.c)
- */
-int sock_getln_err(int *sock, char *buf, int bufsize, int *rc, int nSec)
-{
+// client_getln()   ...   Get a LF-terminated line of text from the client.
+int sock_getln_err(int *sock, char *buf, int bufsize, int *rc, int nSec) {
        int i, retval;
-       CitContext *CCC = MyContext();
        const char *pCh;
 
-       FlushStrBuf(CCC->sMigrateBuf);
-       *rc = retval = CtdlSockGetLine(sock, CCC->sMigrateBuf, nSec);
+       FlushStrBuf(CC->sMigrateBuf);
+       *rc = retval = CtdlSockGetLine(sock, CC->sMigrateBuf, nSec);
 
-       i = StrLength(CCC->sMigrateBuf);
-       pCh = ChrPtr(CCC->sMigrateBuf);
+       i = StrLength(CC->sMigrateBuf);
+       pCh = ChrPtr(CC->sMigrateBuf);
 
        memcpy(buf, pCh, i + 1);
 
-       FlushStrBuf(CCC->sMigrateBuf);
+       FlushStrBuf(CC->sMigrateBuf);
        if (retval < 0) {
                safestrncpy(&buf[i], "000", bufsize - i);
                i += 3;
@@ -288,13 +226,11 @@ int sock_getln_err(int *sock, char *buf, int bufsize, int *rc, int nSec)
        return i;
 }
 
-/*
- * Multiline version of sock_gets() ... this is a convenience function for
- * client side protocol implementations.  It only returns the first line of
- * a multiline response, discarding the rest.
- */
-int ml_sock_gets(int *sock, char *buf, int nSec)
-{
+
+// Multiline version of sock_gets() ... this is a convenience function for
+// client side protocol implementations.  It only returns the first line of
+// a multiline response, discarding the rest.
+int ml_sock_gets(int *sock, char *buf, int nSec) {
        int rc = 0;
        char bigbuf[1024];
        int g;
@@ -319,12 +255,9 @@ int ml_sock_gets(int *sock, char *buf, int nSec)
 }
 
 
-/*
- * sock_puts() - send line to server - implemented in terms of serv_write()
- * Returns the number of bytes written, or -1 for error.
- */
-int sock_puts(int *sock, char *buf)
-{
+// sock_puts() - send line to server - implemented in terms of serv_write()
+// Returns the number of bytes written, or -1 for error.
+int sock_puts(int *sock, char *buf) {
        int i, j;
 
        i = sock_write(sock, buf, strlen(buf));