Version bump.

[mLib] / pkbuf.c

diff --git a/pkbuf.c b/pkbuf.c
index 629f53140ace1cf59ee3d95f36c3df24965e2c1d..5c3d58d116a4300d7471bd9781c0d0fe3c308824 100644 (file)
--- a/pkbuf.c
+++ b/pkbuf.c
@@ -1,6 +1,6 @@
 /* -*-c-*-
  *
 /* -*-c-*-
  *

- * $Id: pkbuf.c,v 1.4 2001/02/03 16:23:33 mdw Exp $
+ * $Id: pkbuf.c,v 1.6 2003/04/23 12:47:44 mdw Exp $

  *
  * Simple packet buffering
  *
  *
  * Simple packet buffering
  *


@@ -30,6 +30,12 @@
 /*----- Revision history --------------------------------------------------* 
  *
  * $Log: pkbuf.c,v $
 /*----- Revision history --------------------------------------------------* 
  *
  * $Log: pkbuf.c,v $

+ * Revision 1.6  2003/04/23 12:47:44  mdw
+ * Various bugfixes.
+ *
+ * Revision 1.5  2002/01/13 13:33:51  mdw
+ * Packet handler functions now have a @typedef@ name.
+ *

  * Revision 1.4  2001/02/03 16:23:33  mdw
  * Bug fix: handle a disable during a close-induced flush without dumping
  * core.
  * Revision 1.4  2001/02/03 16:23:33  mdw
  * Bug fix: handle a disable during a close-induced flush without dumping
  * core.


@@ -83,7 +89,7 @@
 void pkbuf_flush(pkbuf *pk, octet *p, size_t len)
 {
   size_t l;
 void pkbuf_flush(pkbuf *pk, octet *p, size_t len)
 {
   size_t l;

-  size_t keep;
+  size_t o, keep;

 
   if (pk->f & PKBUF_CLOSE) {
     pk->func(0, 0, pk, 0, pk->p);
 
   if (pk->f & PKBUF_CLOSE) {
     pk->func(0, 0, pk, 0, pk->p);


@@ -97,22 +103,23 @@ void pkbuf_flush(pkbuf *pk, octet *p, size_t len)
     len = pk->len;
   }
   l = p + len - pk->buf;
     len = pk->len;
   }
   l = p + len - pk->buf;

-  p = pk->buf;
+  o = 0;

 
   /* --- Now grind through any packets which have accumulated --- */
 
 
   /* --- Now grind through any packets which have accumulated --- */
 

-  while (l > pk->want) {
+  pk->len = l;
+  while (l >= pk->want) {

     size_t sz = pk->want;
 
     /* --- Pass a packet to the user handler --- */
 
     keep = 0;
     size_t sz = pk->want;
 
     /* --- Pass a packet to the user handler --- */
 
     keep = 0;

-    pk->func(p, sz, pk, &keep, pk->p);
+    pk->func(pk->buf + o, sz, pk, &keep, pk->p);

 
     /* --- Adjust all the pointers for the next packet --- */
 
     sz -= keep;
 
     /* --- Adjust all the pointers for the next packet --- */
 
     sz -= keep;

-    p += sz;
+    o += sz;

     l -= sz;
 
     /* --- Abort here if disabled --- */
     l -= sz;
 
     /* --- Abort here if disabled --- */


@@ -123,8 +130,8 @@ void pkbuf_flush(pkbuf *pk, octet *p, size_t len)
 
   /* --- Shunt data around in the buffer --- */
 
 
   /* --- Shunt data around in the buffer --- */
 

-  if (p > pk->buf && l != 0)
-    memmove(pk->buf, p, l);
+  if (o > 0 && l != 0)
+    memmove(pk->buf, pk->buf + o, l);

   pk->len = l;
 }
 
   pk->len = l;
 }
 


@@ -164,10 +171,8 @@ void pkbuf_close(pkbuf *pk)
 
 size_t pkbuf_free(pkbuf *pk, octet **p)
 {
 
 size_t pkbuf_free(pkbuf *pk, octet **p)
 {

-  if (!pk->buf) {
-    fprintf(stderr, "*** allocating new buffer\n");
+  if (!pk->buf)

     pk->buf = x_alloc(pk->a, pk->sz);
     pk->buf = x_alloc(pk->a, pk->sz);

-  }

   *p = pk->buf + pk->len;
   return (pk->sz - pk->len);
 }
   *p = pk->buf + pk->len;
   return (pk->sz - pk->len);
 }


@@ -218,7 +223,7 @@ void pkbuf_want(pkbuf *pk, size_t want)
 {
   pk->want = want;
   if (want > pk->sz) {
 {
   pk->want = want;
   if (want > pk->sz) {

-    do pk->sz <<= 1; while (want < pk->sz);
+    do pk->sz <<= 1; while (want > pk->sz);

     if (pk->buf) {
       if (pk->len)
        pk->buf = x_realloc(pk->a, pk->buf, pk->sz, pk->len);
     if (pk->buf) {
       if (pk->len)
        pk->buf = x_realloc(pk->a, pk->buf, pk->sz, pk->len);


@@ -233,9 +238,7 @@ void pkbuf_want(pkbuf *pk, size_t want)
 /* --- @pkbuf_init@ --- *
  *
  * Arguments:  @pkbuf *pk@ = pointer to buffer block
 /* --- @pkbuf_init@ --- *
  *
  * Arguments:  @pkbuf *pk@ = pointer to buffer block

- *             @void (*func)(octet *b, size_t sz, pkbuf *pk,@
- *                          @size_t *keep, void *p)@ =
- *                     handler function
+ *             @pkbuf *func@ = handler function

  *             @void *p@ = argument pointer for @func@
  *
  * Returns:    ---
  *             @void *p@ = argument pointer for @func@
  *
  * Returns:    ---


@@ -244,10 +247,7 @@ void pkbuf_want(pkbuf *pk, size_t want)
  *             the provided function for handling.
  */
 
  *             the provided function for handling.
  */
 

-void pkbuf_init(pkbuf *pk,
-               void (*func)(octet */*b*/, size_t /*sz*/,
-                            pkbuf */*pk*/, size_t */*keep*/, void */*p*/),
-               void *p)
+void pkbuf_init(pkbuf *pk, pkbuf_func *func, void *p)

 {
   pk->func = func;
   pk->p = p;
 {
   pk->func = func;
   pk->p = p;