From: Derick Rethans <derick@php.net>
Date: Wed, 12 Oct 2005 19:49:35 +0000 (+0000)
Subject: - Apperently you can't store NULL pointer values into Zend hashes - god know
X-Git-Tag: RELEASE_0_9_1~131
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=569f8afa2cb4953fa3d0dfb6ed81d50c94d1b31b;p=php

- Apperently you can't store NULL pointer values into Zend hashes - god know
  why not - so we won't store those in the cache anymore.
---

diff --git a/ext/date/php_date.c b/ext/date/php_date.c
index 32f2558889..26b3e44da8 100644
--- a/ext/date/php_date.c
+++ b/ext/date/php_date.c
@@ -192,9 +192,7 @@ static void _php_date_tzinfo_dtor(void *tzinfo)
 {
 	timelib_tzinfo **tzi = (timelib_tzinfo **)tzinfo;
 
-	if (*tzi) {
-		timelib_tzinfo_dtor(*tzi);
-	}
+	timelib_tzinfo_dtor(*tzi);
 }
 
 /* {{{ PHP_RINIT_FUNCTION */
@@ -291,7 +289,9 @@ static timelib_tzinfo *php_date_parse_tzfile(char *formal_tzname, timelib_tzdb *
 	}
 
 	tzi = timelib_parse_tzfile(formal_tzname, tzdb);
-	zend_hash_add(&DATEG(tzcache), formal_tzname, strlen(formal_tzname) + 1, (void *) &tzi, sizeof(timelib_tzinfo*), NULL);
+	if (tzi) {
+		zend_hash_add(&DATEG(tzcache), formal_tzname, strlen(formal_tzname) + 1, (void *) &tzi, sizeof(timelib_tzinfo*), NULL);
+	}
 	return tzi;
 }
 /* }}} */