Make the maximum /list reply length a configurable limit.

[ngircd-alex.git] / src / ngircd / conn-zip.c

diff --git a/src/ngircd/conn-zip.c b/src/ngircd/conn-zip.c
index 0b9161f35f85e4fa0346053ab8ea1aeb92abfcb4..0a3c17c89b5e07f90945ef245bb072edccb312b0 100644 (file)
--- a/src/ngircd/conn-zip.c
+++ b/src/ngircd/conn-zip.c
@@ -7,12 +7,15 @@
  * the Free Software Foundation; either version 2 of the License, or
  * (at your option) any later version.
  * Please read the file COPYING, README and AUTHORS for more information.
- *
- * Connection compression using ZLIB
  */
 
 #include "portab.h"
 
+/**
+ * @file
+ * Connection compression using ZLIB
+ */
+
 #define CONN_MODULE
 
 #ifdef ZLIB
@@ -80,10 +83,12 @@ Zip_InitConn( CONN_ID Idx )
  * compression ratios.
  * If the (pre-)compression buffer is full, we try to flush it ("actually
  * compress some data") and to add the new (uncompressed) data afterwards.
+ * This function closes the connection on error.
  * @param Idx Connection handle.
  * @param Data Pointer to the data.
  * @param Len Length of the data to add.
- * @return true on success, false otherwise. */
+ * @return true on success, false otherwise.
+ */
 GLOBAL bool
 Zip_Buffer( CONN_ID Idx, const char *Data, size_t Len )
 {
@@ -102,9 +107,11 @@ Zip_Buffer( CONN_ID Idx, const char *Data, size_t Len )
        /* check again; if zip buf is still too large do not append data:
         * otherwise the zip wbuf would grow too large */
        buflen = array_bytes(&My_Connections[Idx].zip.wbuf);
-       if (buflen + Len >= WRITEBUFFER_SLINK_LEN)
+       if (buflen + Len >= WRITEBUFFER_SLINK_LEN) {
+               Log(LOG_ALERT, "Zip Write buffer space exhausted: %lu bytes", buflen + Len);
+               Conn_Close(Idx, "Zip Write buffer space exhausted", NULL, false);
                return false;
-
+       }
        return array_catb(&My_Connections[Idx].zip.wbuf, Data, Len);
 } /* Zip_Buffer */
 
@@ -112,6 +119,7 @@ Zip_Buffer( CONN_ID Idx, const char *Data, size_t Len )
 /**
  * Compress data in ZIP buffer and move result to the write buffer of
  * the connection.
+ * This function closes the connection on error.
  * @param Idx Connection handle.
  * @return true on success, false otherwise.
  */
@@ -150,7 +158,7 @@ Zip_Flush( CONN_ID Idx )
        if (out->avail_out <= 0) {
                /* Not all data was compressed, because data became
                 * bigger while compressing it. */
-               Log (LOG_ALERT, "Compression error: buffer overvlow!?");
+               Log(LOG_ALERT, "Compression error: buffer overflow!?");
                Conn_Close(Idx, "Compression error!", NULL, false);
                return false;
        }
@@ -180,6 +188,7 @@ Zip_Flush( CONN_ID Idx )
  * uncompress data and copy it to read buffer.
  * Returns true if data has been unpacked or no
  * compressed data is currently pending in the zread buffer.
+ * This function closes the connection on error.
  * @param Idx Connection handle.
  * @return true on success, false otherwise.
  */
@@ -230,9 +239,11 @@ Unzip_Buffer( CONN_ID Idx )
 #endif
        assert(unzipbuf_used <= READBUFFER_LEN);
        if (!array_catb(&My_Connections[Idx].rbuf, (char*) unzipbuf,
-                       (size_t)unzipbuf_used))
+                       (size_t)unzipbuf_used)) {
+               Log (LOG_ALERT, "Decompression error: can't copy data!?");
+               Conn_Close(Idx, "Decompression error!", NULL, false);
                return false;
-
+       }
        if( in->avail_in > 0 ) {
                array_moveleft(&My_Connections[Idx].zip.rbuf, 1, in_len );
        } else {