All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Syntax error in iptables.xslt
@ 2007-08-02 15:05 Dan Nicholson
  0 siblings, 0 replies; 4+ messages in thread
From: Dan Nicholson @ 2007-08-02 15:05 UTC (permalink / raw)
  To: netfilter-devel

Using the iptables.xslt stylesheet as shipped in iptables-1.3.8 is causing
syntax errors when run through xsltproc.

$ echo | xsltproc iptables.xslt -
compilation error: file iptables.xslt line 118 element param
The value '$node' of the attribute 'name' is not a valid QName.
$ xsltproc --version
Using libxml 20627, libxslt 10120 and libexslt 813
xsltproc was compiled against libxml 20627, libxslt 10120 and libexslt 813
libxslt 10120 was compiled against libxml 20627
libexslt 813 was compiled against libxml 20627

The problem is that in <xsl:param name="$node"/>, "$node" is not a valid
QName.

http://www.w3.org/TR/xslt#variables

Patch below converts to a valid name and the one expected in the rest of
the stylesheet when using <xsl:with-param name="node"/>.

--
Dan

Index: iptables.xslt
===================================================================
--- iptables.xslt	(revision 6982)
+++ iptables.xslt	(working copy)
@@ -116,7 +116,7 @@
   </xsl:template>
   
   <xsl:template name="counters">
-    <xsl:param name="$node"/>
+    <xsl:param name="node"/>
     <xsl:text>[</xsl:text>
     <xsl:if test="string-length($node/@packet-count)"><xsl:value-of select="$node/@packet-count"/></xsl:if>
     <xsl:if test="string-length($node/@packet-count)=0">0</xsl:if>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] Syntax error in iptables.xslt
@ 2007-08-02 15:42 Dan Nicholson
  2007-08-03 11:13 ` Amin Azez
  2007-08-06  8:50 ` Pablo Neira Ayuso
  0 siblings, 2 replies; 4+ messages in thread
From: Dan Nicholson @ 2007-08-02 15:42 UTC (permalink / raw)
  To: netfilter-devel

Using the iptables.xslt stylesheet as shipped in iptables-1.3.8 is causing
syntax errors when run through xsltproc.

$ echo | xsltproc iptables.xslt -
compilation error: file iptables.xslt line 118 element param
The value '$node' of the attribute 'name' is not a valid QName.
$ xsltproc --version
Using libxml 20627, libxslt 10120 and libexslt 813
xsltproc was compiled against libxml 20627, libxslt 10120 and libexslt 813
libxslt 10120 was compiled against libxml 20627
libexslt 813 was compiled against libxml 20627

The problem is that in <xsl:param name="$node"/>, "$node" is not a valid
QName.

http://www.w3.org/TR/xslt#variables

Patch below converts to a valid name and the one expected in the rest of
the stylesheet when using <xsl:with-param name="node"/>.

--
Dan

Index: iptables.xslt
===================================================================
--- iptables.xslt	(revision 6982)
+++ iptables.xslt	(working copy)
@@ -116,7 +116,7 @@
   </xsl:template>
   
   <xsl:template name="counters">
-    <xsl:param name="$node"/>
+    <xsl:param name="node"/>
     <xsl:text>[</xsl:text>
     <xsl:if test="string-length($node/@packet-count)"><xsl:value-of select="$node/@packet-count"/></xsl:if>
     <xsl:if test="string-length($node/@packet-count)=0">0</xsl:if>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Syntax error in iptables.xslt
  2007-08-02 15:42 [PATCH] Syntax error in iptables.xslt Dan Nicholson
@ 2007-08-03 11:13 ` Amin Azez
  2007-08-06  8:50 ` Pablo Neira Ayuso
  1 sibling, 0 replies; 4+ messages in thread
From: Amin Azez @ 2007-08-03 11:13 UTC (permalink / raw)
  To: Dan Nicholson, Patrick McHardy; +Cc: netfilter-devel

* Dan Nicholson wrote, On 02/08/07 16:42:
> Patch below converts to a valid name and the one expected in the rest of
> the stylesheet when using <xsl:with-param name="node"/>.
> 
> --
> Dan
> 
> Index: iptables.xslt
> ===================================================================
> --- iptables.xslt	(revision 6982)
> +++ iptables.xslt	(working copy)
> @@ -116,7 +116,7 @@
>    </xsl:template>
>    
>    <xsl:template name="counters">
> -    <xsl:param name="$node"/>
> +    <xsl:param name="node"/>
>      <xsl:text>[</xsl:text>
>      <xsl:if test="string-length($node/@packet-count)"><xsl:value-of select="$node/@packet-count"/></xsl:if>
>      <xsl:if test="string-length($node/@packet-count)=0">0</xsl:if>
> 
> 

Thanks for spotting that. It would have had the effect of not restoring
counters.

Patrick, please accept this patch.

Signed-off by: Sam Liddicott <azez@ufomechanic.net>

Sam

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Syntax error in iptables.xslt
  2007-08-02 15:42 [PATCH] Syntax error in iptables.xslt Dan Nicholson
  2007-08-03 11:13 ` Amin Azez
@ 2007-08-06  8:50 ` Pablo Neira Ayuso
  1 sibling, 0 replies; 4+ messages in thread
From: Pablo Neira Ayuso @ 2007-08-06  8:50 UTC (permalink / raw)
  To: Dan Nicholson; +Cc: netfilter-devel

Dan Nicholson wrote:
> The problem is that in <xsl:param name="$node"/>, "$node" is not a valid
> QName.
> 
> http://www.w3.org/TR/xslt#variables
> 
> Patch below converts to a valid name and the one expected in the rest of
> the stylesheet when using <xsl:with-param name="node"/>.

Applied, thanks Dan.

-- 
"Será preciso viajar a través de los ojos de los idiotas" -- Poeta en
Nueva York -- Federico García Lorca.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-08-06  8:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-02 15:42 [PATCH] Syntax error in iptables.xslt Dan Nicholson
2007-08-03 11:13 ` Amin Azez
2007-08-06  8:50 ` Pablo Neira Ayuso
  -- strict thread matches above, loose matches on Subject: below --
2007-08-02 15:05 Dan Nicholson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.