X-Git-Url: http://git.indexdata.com/?a=blobdiff_plain;f=data1%2Fd1_read.c;h=0db4c89622b08ed14c8f2d4f152578debf455963;hb=43e4297c07b9c8b29bfc1ea647fc27456198f6ce;hp=52e2364312f3ea71f08b99c78c4392172d3765a0;hpb=4478d785b7769691261005c98063b98a5a5971b3;p=idzebra-moved-to-github.git

diff --git a/data1/d1_read.c b/data1/d1_read.c
index 52e2364..0db4c89 100644
--- a/data1/d1_read.c
+++ b/data1/d1_read.c
@@ -1,5 +1,5 @@
-/* $Id: d1_read.c,v 1.21 2006-08-14 10:40:06 adam Exp $
-   Copyright (C) 1995-2006
+/* $Id: d1_read.c,v 1.25 2007-04-16 08:44:31 adam Exp $
+   Copyright (C) 1995-2007
    Index Data ApS
 
 This file is part of the Zebra server.
@@ -174,6 +174,7 @@ data1_node *data1_mk_root (data1_handle dh, NMEM nmem, const char *name)
 {
     data1_absyn *absyn = data1_get_absyn(dh, name, 1);
     data1_node *res;
+
     if (!absyn)
     {
         yaz_log(YLOG_WARN, "Unable to acquire abstract syntax " "for '%s'",
@@ -592,7 +593,6 @@ data1_xattr *data1_read_xattr (data1_handle dh, NMEM m,
     for (;;)
     {
 	data1_xattr *p;
-	int len;
 	while (*amp || (c && d1_isspace(c)))
 	    c = ampr (get_byte, fh, amp);
 	if (*amp == 0 && (c == 0 || c == '>' || c == '/'))
@@ -609,10 +609,7 @@ data1_xattr *data1_read_xattr (data1_handle dh, NMEM m,
 	    wrbuf_putc (wrbuf, c);
 	    c = ampr (get_byte, fh, amp);
 	}
-	wrbuf_putc (wrbuf, '\0');
-	len = wrbuf_len(wrbuf);
-	p->name = (char*) nmem_malloc (m, len);
-	strcpy (p->name, wrbuf_buf(wrbuf));
+	p->name = nmem_strdup (m, wrbuf_cstr(wrbuf));
 	if (c == '=')
 	{
 	    c = ampr (get_byte, fh, amp);
@@ -649,10 +646,7 @@ data1_xattr *data1_read_xattr (data1_handle dh, NMEM m,
 		    c = ampr (get_byte, fh, amp);
 	        }
             }
-	    wrbuf_putc (wrbuf, '\0');
-	    len = wrbuf_len(wrbuf);
-	    p->value = (char*) nmem_malloc (m, len);
-	    strcpy (p->value, wrbuf_buf(wrbuf));
+	    p->value = nmem_strdup(m, wrbuf_cstr(wrbuf));
 	}
     }
     *ch = c;
@@ -965,7 +959,7 @@ data1_node *data1_read_node (data1_handle dh, const char **buf, NMEM m)
     data1_node *node;
 
     node = data1_read_nodex(dh, m, getc_mem, (void *) (buf), wrbuf);
-    wrbuf_free (wrbuf, 1);
+    wrbuf_destroy(wrbuf);
     return node;
 }
 
@@ -1020,14 +1014,15 @@ static int conv_item (NMEM m, yaz_iconv_t t,
     {
         char *outbuf = wrbuf->buf + wrbuf->pos;
         size_t outlen = wrbuf->size - wrbuf->pos;
-        if (yaz_iconv (t, &inbuf, &inlen, &outbuf, &outlen) ==
+        if (yaz_iconv(t, &inbuf, &inlen, &outbuf, &outlen) ==
             (size_t)(-1) && yaz_iconv_error(t) != YAZ_ICONV_E2BIG)
         {
             /* bad data. stop and skip conversion entirely */
             return -1;
         }
         else if (inlen == 0)
-        {   /* finished converting */
+        {   /* finished converting, flush it */
+            yaz_iconv(t, 0, 0, &outbuf, &outlen);
             wrbuf->pos = wrbuf->size - outlen;
             break;
         }
@@ -1075,8 +1070,7 @@ static void data1_iconv_s (data1_handle dh, NMEM m, data1_node *n,
                         conv_item(m, t, wrbuf, p->value, strlen(p->value))
                         == 0)
                     {
-                        wrbuf_puts (wrbuf, "");
-                        p->value = nmem_strdup (m, wrbuf->buf);
+                        p->value = nmem_strdup(m, wrbuf_cstr(wrbuf));
                     }
                 }
             }
@@ -1124,12 +1118,12 @@ int data1_iconv (data1_handle dh, NMEM m, data1_node *n,
         yaz_iconv_t t = yaz_iconv_open(tocode, fromcode);
         if (!t)
 	{
-            wrbuf_free(wrbuf, 1);
+            wrbuf_destroy(wrbuf);
             return -1;
 	}
         data1_iconv_s(dh, m, n, t, wrbuf, tocode);
         yaz_iconv_close(t);
-        wrbuf_free(wrbuf, 1);
+        wrbuf_destroy(wrbuf);
     }
     return 0;
 }