fixed mlXPathCompOpEvalPositionalPredicate problem with object caching

* xpath.c: fixed mlXPathCompOpEvalPositionalPredicate problem
  with object caching (bug #469410)

svn path=/trunk/; revision=3653

1 files changed, 8 insertions, 1 deletions

diff --git a/xpath.c b/xpath.c
index 9c4db87c..22a3f9d0 100644
--- a/xpath.c
+++ b/xpath.c
@@ -11706,8 +11706,15 @@ xmlXPathCompOpEvalPositionalPredicate(xmlXPathParserContextPtr ctxt,
 	    valuePush(ctxt, contextObj);
 	    res = xmlXPathCompOpEvalToBoolean(ctxt, exprOp, 1);
 	    
-	    if ((ctxt->error != XPATH_EXPRESSION_OK) || (res == -1))
+	    if ((ctxt->error != XPATH_EXPRESSION_OK) || (res == -1)) {
+	        xmlXPathObjectPtr tmp;
+		/* pop the result */
+		tmp = valuePop(ctxt);
+		xmlXPathReleaseObject(xpctxt, tmp);
+		/* then pop off contextObj, which will be freed later */
+		valuePop(ctxt);
 		goto evaluation_error;
+	    }
 
 	    if (res)
 		pos++;