Re: cvs: php-src(PHP_5_0) /ext/xsl xsltprocessor.c

Nope you shouldn't. I thought it existed in 5.0.0 too. At 10:41 AM 8/10/2004 +0200, Christian Stocker wrote:

On 10.8.2004 10:25 Uhr, Andi Gutmans wrote:

News?

Should I really add it? The bug was not existent in 5.0.0 and there was no bugreport. chregu

At 08:02 AM 8/10/2004 +0000, Christian Stocker wrote:

chregu          Tue Aug 10 04:02:01 2004 EDT

  Modified files:              (Branch: PHP_5_0)
    /php-src/ext/xsl    xsltprocessor.c

Log: MFH (Fix segfault on error in xslt) http://cvs.php.net/diff.php/php-src/ext/xsl/xsltprocessor.c?r1=1.29.2.2&r2=1.29.2.3&ty=u Index: php-src/ext/xsl/xsltprocessor.c diff -u php-src/ext/xsl/xsltprocessor.c:1.29.2.2 php-src/ext/xsl/xsltprocessor.c:1.29.2.3

--- php-src/ext/xsl/xsltprocessor.c:1.29.2.2    Sun Aug  8 14:05:47 2004
+++ php-src/ext/xsl/xsltprocessor.c     Tue Aug 10 04:02:00 2004

@@ -17,7 +17,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: xsltprocessor.c,v 1.29.2.2 2004/08/08 18:05:47 chregu Exp $ */ +/* $Id: xsltprocessor.c,v 1.29.2.3 2004/08/10 08:02:00 chregu Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -400,7 +400,10 @@

        char **params = NULL;
        int clone;

-

+       if (style == NULL) {
+               php_error_docref(NULL TSRMLS_CC, E_WARNING, "No stylesheet associated to this object");
+               return NULL;
+       }
        if (intern->parameter) {
                params = php_xsl_xslt_make_params(intern->parameter, 0 TSRMLS_CC);
        }

-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php

-- christian stocker | Bitflux GmbH | schoeneggstrasse 5 | ch-8004 zurich phone +41 1 240 56 70 | mobile +41 76 561 88 60 | fax +41 1 240 56 71 http://www.bitflux.ch | chregu@bitflux.ch | gnupg-keyid 0x5CE1DECB