Skip site navigation (1)Skip section navigation (2)
Date:      Thu,  6 Dec 2001 16:20:43 +0000 (GMT)
From:      Dominic Mitchell <dom@happygiraffe.net>
To:        FreeBSD-gnats-submit@freebsd.org
Subject:   ports/32558: www/sitecopy is broken
Message-ID:  <20011206162043.8977A6262@cathbad.happygiraffe.net>

next in thread | raw e-mail | index | archive | help

>Number:         32558
>Category:       ports
>Synopsis:       www/sitecopy is broken
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Dec 06 08:30:00 PST 2001
>Closed-Date:
>Last-Modified:
>Originator:     Dominic Mitchell
>Release:        FreeBSD 5.0-CURRENT i386
>Organization:
>Environment:
System: FreeBSD cathbad.happygiraffe.net 5.0-CURRENT FreeBSD 5.0-CURRENT #10: Thu Dec 6 11:16:43 GMT 2001 root@cathbad.happygiraffe.net:/usr/obj/usr/src/sys/CATHBAD i386

Compiled using:
libxml2-2.4.6       Xml parser library for GNOME

>Description:
This port is broken.  It crashes when trying to parse it's
~/.sitecopy/<site> file.  I have gotten a patch from the author, which
will be in the next version, which solves the problem.
>How-To-Repeat:
	% sitecopy -u www.mysite.com
>Fix:

Here's the patch.  Just dump it directly into www/sitecopy/files/patch-libneon-hip_xml.c.

--- libneon/hip_xml.c.orig	Thu Dec  6 16:09:42 2001
+++ libneon/hip_xml.c	Thu Dec  6 16:10:32 2001
@@ -768,12 +768,6 @@
     struct hip_xml_handler *hand, *next;
 
     sbuffer_destroy(p->buffer);
-    /* Clean up any states which may remain.
-     * If p.valid, then this should be only the root element. */
-    for (s = p->current; s!=NULL; s=parent) {
-	parent = s->parent;
-	destroy_state(s);
-    }
 
     /* Free up the handlers on the stack: the base of the stack
      * is the handler in the root element, so start there. */
@@ -781,7 +775,14 @@
 	next = hand->next;
 	free(hand);
     }
-	 
+
+    /* Clean up any states which may remain.
+     * If p.valid, then this should be only the root element. */
+    for (s = p->current; s!=NULL; s=parent) {
+	parent = s->parent;
+	destroy_state(s);
+    }
+ 
 #ifdef HAVE_EXPAT
     XML_ParserFree(p->parser);
 #else
>Release-Note:
>Audit-Trail:
>Unformatted:

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-ports" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20011206162043.8977A6262>