Skip site navigation (1) Skip section navigation (2)

Re: [PATCH] Bug in XPATH() if expression returns a scalar value

From: Florian Pflug <fgp(at)phlo(dot)org>
To: Ross J(dot) Reedstrom <reedstrm(at)rice(dot)edu>
Cc: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Bug in XPATH() if expression returns a scalar value
Date: 2011-05-31 19:28:50
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
On May31, 2011, at 19:15 , Ross J. Reedstrom wrote:
> What you describe, making XPATH return something for the scalar
> functions, is sorely needed. Constraining the return values to be valid
> XML fragments is the sort of wart that makes XML processing in
> postgresql seem odd to those familiar with other tools, though.

I've now changes things so that the results of scalar-value
returning XPath expressions are correctly entity-encoded (i.e.,
a literal "<" gets translated to "&lt;").

After realizing that this is necessary even for node-set
returning XPath expressions (see my other mail from today),
because they may very well select text nodes, I came to the
conclusion that doing this unconditionally (well, except for
element nodes obviously) is the least surprising behaviour.

The following subsumes the behavior with this and the patch
from my other e-mail applied.

        (XPATH('namespace-uri(/*)', x))[1] AS namespace,
        (XPATH('/*/@value', x))[1] AS value,
        (XPATH('/*/text()', x))[1] AS text
        XMLATTRIBUTES('<n' AS xmlns, '<v' AS value),
))) v(x);

 namespace | value | text  
 &lt;n     | &lt;v | &lt;t

Without the patch from the other mail, the "namespace" result
stays the same, but "value" and "text" are "<v" and "<t"

Updated patch is attached

best regards,
Florian Pflug

PS: Btw, while trying this I think I found another problem. If you

        XMLELEMENT(NAME "root",
                   XMLATTRIBUTES('<n' AS xmlns,
                                 '<v' AS value))

you get

 <root xmlns="<n" value="&lt;v"/>

i.e. the "<" in the namespace URI isn't quoted properly.
Trying to cast that value to text and back to xml fails.
Funnily enough, if you skip the XPATH() call, things work properly

                  XMLATTRIBUTES('<n' AS xmlns,
                                '<v' AS value))


 <root xmlns="&lt;n" value="&lt;v"/>

I'll start a new thread for this issue...

Attachment: pg_xpath_returnvalue.v2.patch
Description: application/octet-stream (7.4 KB)

In response to

pgsql-hackers by date

Next:From: Peter EisentrautDate: 2011-05-31 19:38:08
Subject: Re: Please test peer (socket ident) auth on *BSD
Previous:From: Cédric VillemainDate: 2011-05-31 18:52:54
Subject: Re: Getting a bug tracker for the Postgres project

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group