Always compact the buffer if it has reached MAXBUFSIZE.
authorGuus Sliepen <guus@tinc-vpn.org>
Sun, 22 May 2011 11:15:05 +0000 (13:15 +0200)
committerGuus Sliepen <guus@tinc-vpn.org>
Sun, 22 May 2011 11:24:01 +0000 (13:24 +0200)
src/buffer.c
src/buffer.h
src/meta.c

index 249e3e0..315faf1 100644 (file)
@@ -22,8 +22,8 @@
 #include "buffer.h"
 #include "xalloc.h"
 
-void buffer_compact(buffer_t *buffer) {
-       if(buffer->offset / 7 > buffer->len / 8) {
+void buffer_compact(buffer_t *buffer, int maxsize) {
+       if(buffer->len >= maxsize || buffer->offset / 7 > buffer->len / 8) {
                memmove(buffer->data, buffer->data + buffer->offset, buffer->len - buffer->offset);
                buffer->len -= buffer->offset;
                buffer->offset = 0;
index 73c1f69..a96c15a 100644 (file)
@@ -8,7 +8,7 @@ typedef struct buffer_t {
        int offset;
 } buffer_t;
 
-extern void buffer_compact(buffer_t *buffer);
+extern void buffer_compact(buffer_t *buffer, int maxsize);
 extern char *buffer_prepare(buffer_t *buffer, int size);
 extern void buffer_add(buffer_t *buffer, const char *data, int size);
 extern char *buffer_readline(buffer_t *buffer);
index cf6b9d4..5735da7 100644 (file)
@@ -85,10 +85,10 @@ bool receive_meta(connection_t *c) {
           - If not, keep stuff in buffer and exit.
         */
 
-       buffer_compact(&c->inbuf);
+       buffer_compact(&c->inbuf, MAXBUFSIZE);
 
        if(sizeof inbuf <= c->inbuf.len) {
-               logger(LOG_ERR, "Input buffer full for %s (%s)\n");
+               logger(LOG_ERR, "Input buffer full for %s (%s)\n", c, c->hostname);
                return false;
        }