From: Antony Dovgal <tony2001@php.net>
Date: Tue, 15 Nov 2005 14:07:43 +0000 (+0000)
Subject: call zend_objects_destroy_object() explicitly
X-Git-Tag: RELEASE_2_0_1~5
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=21aeacfa9d697105587f691be2d17ce892879f57;p=php

call zend_objects_destroy_object() explicitly
fix segfault when intern->time is NULL
---

diff --git a/ext/date/php_date.c b/ext/date/php_date.c
index 5ed733ce1b..f904f4d3ac 100644
--- a/ext/date/php_date.c
+++ b/ext/date/php_date.c
@@ -1138,7 +1138,7 @@ static zend_object_value date_object_new_date(zend_class_entry *class_type TSRML
 	memset(intern, 0, sizeof(php_date_obj));
 	intern->std.ce = class_type;
 
-	retval.handle = zend_objects_store_put(intern, NULL, (zend_objects_free_object_storage_t) date_object_free_storage_date, NULL TSRMLS_CC);
+	retval.handle = zend_objects_store_put(intern, (zend_objects_store_dtor_t)zend_objects_destroy_object, (zend_objects_free_object_storage_t) date_object_free_storage_date, NULL TSRMLS_CC);
 	retval.handlers = &date_object_handlers_date;
 	
 	return retval;
@@ -1153,7 +1153,7 @@ static zend_object_value date_object_new_timezone(zend_class_entry *class_type T
 	memset(intern, 0, sizeof(php_timezone_obj));
 	intern->std.ce = class_type;
 
-	retval.handle = zend_objects_store_put(intern, NULL, (zend_objects_free_object_storage_t) date_object_free_storage_timezone, NULL TSRMLS_CC);
+	retval.handle = zend_objects_store_put(intern, (zend_objects_store_dtor_t)zend_objects_destroy_object, (zend_objects_free_object_storage_t) date_object_free_storage_timezone, NULL TSRMLS_CC);
 	retval.handlers = &date_object_handlers_timezone;
 	
 	return retval;
@@ -1163,10 +1163,12 @@ static void date_object_free_storage_date(void *object TSRMLS_DC)
 {
 	php_date_obj *intern = (php_date_obj *)object;
 
-	if (intern->time->tz_info) {
-		timelib_tzinfo_dtor(intern->time->tz_info);
+	if (intern->time) {
+		if (intern->time->tz_info) {
+			timelib_tzinfo_dtor(intern->time->tz_info);
+		}
+		timelib_time_dtor(intern->time);
 	}
-	timelib_time_dtor(intern->time);
 
 	efree(object);
 }