Re: [xml] xmlParseDTD breaks now

Date view Thread view Subject view Author view

From: Daniel Veillard (Daniel.Veillard@w3.org)
Date: Tue Sep 26 2000 - 18:38:53 EDT


On Tue, Sep 26, 2000 at 02:59:29PM -0700, Wayne Davison wrote:
>
> On Tue, 26 Sep 2000, Daniel Veillard wrote:
> > But I dislike the fact that the URI (and the base in the previous case)
> > are not parsed to check that their content is a valid URI ...
>
> How about this change (instead of your suggested one)?
>
> Index: uri.c
> @@ -1398,6 +1398,10 @@
> if (ret != 0)
> goto done;
> }
> + if (base == NULL) {
> + val = xmlSaveUri(ref);
> + goto done;
> + }
> bas = xmlCreateURI();
> if (bas == NULL)
> goto done;

  Won't work I'm afraid, xmlParseURIReference will fail and ret != 0
won't let this code execute :-)
  Patch enclosed, i think this solves the problem in a clean way.
Feedback welcome ...

Daniel

-- 
Daniel.Veillard@w3.org | W3C, INRIA Rhone-Alpes  | Today's Bookmarks :
Tel : +33 476 615 257  | 655, avenue de l'Europe | Linux XML libxml WWW
Fax : +33 476 615 207  | 38330 Montbonnot FRANCE | Gnome rpm2html rpmfind
 http://www.w3.org/People/all#veillard%40w3.org  | RPM badminton Kaffe


----
Message from the list xml@rpmfind.net
Archived at : http://xmlsoft.org/messages/
to unsubscribe: echo "unsubscribe xml" | mail  majordomo@rpmfind.net


Date view Thread view Subject view Author view

This archive was generated by hypermail 2b29 : Tue Sep 26 2000 - 18:43:46 EDT