From: Xinchen Hui <laruence@php.net>
Date: Tue, 13 May 2014 05:45:24 +0000 (+0800)
Subject: Remove zval **
X-Git-Tag: POST_PHPNG_MERGE~378^2~2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=97c836e421f47729cf3b6ae7f7840a10d382f4f8;p=php

Remove zval **
---

diff --git a/ext/standard/dns_win32.c b/ext/standard/dns_win32.c
index 5d00b07af0..c4895610ff 100644
--- a/ext/standard/dns_win32.c
+++ b/ext/standard/dns_win32.c
@@ -138,7 +138,7 @@ PHP_FUNCTION(dns_check_record)
 /* }}} */
 
 /* {{{ php_parserr */
-static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw, zval **subarray)
+static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw, zval *subarray)
 {
 	int type;
 	u_long ttl;
@@ -154,14 +154,15 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 		return;
 	}
 
-	ZVAL_NEW_ARR(*subarray);
-	add_assoc_string(*subarray, "host", pRec->pName);
-	add_assoc_string(*subarray, "class", "IN");
-	add_assoc_long(*subarray, "ttl", ttl);
+	array_init(subarray);
+
+	add_assoc_string(subarray, "host", pRec->pName);
+	add_assoc_string(subarray, "class", "IN");
+	add_assoc_long(subarray, "ttl", ttl);
 
 	if (raw) {
-		add_assoc_long(*subarray, "type", type);
-		add_assoc_stringl(*subarray, "data", (char*) &pRec->Data, (uint) pRec->wDataLength);
+		add_assoc_long(subarray, "type", type);
+		add_assoc_stringl(subarray, "data", (char*) &pRec->Data, (uint) pRec->wDataLength);
 		return;
 	}
 
@@ -169,33 +170,33 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 		case DNS_TYPE_A: {
 			IN_ADDR ipaddr;
 			ipaddr.S_un.S_addr = (pRec->Data.A.IpAddress);
-			add_assoc_string(*subarray, "type", "A");
-			add_assoc_string(*subarray, "ip", inet_ntoa(ipaddr));
+			add_assoc_string(subarray, "type", "A");
+			add_assoc_string(subarray, "ip", inet_ntoa(ipaddr));
 			break;
 		}
 
 		case DNS_TYPE_MX:
-			add_assoc_string(*subarray, "type", "MX");
-			add_assoc_long(*subarray, "pri", pRec->Data.Srv.wPriority);
+			add_assoc_string(subarray, "type", "MX");
+			add_assoc_long(subarray, "pri", pRec->Data.Srv.wPriority);
 			/* no break; */
 
 		case DNS_TYPE_CNAME:
 			if (type == DNS_TYPE_CNAME) {
-				add_assoc_string(*subarray, "type", "CNAME");
+				add_assoc_string(subarray, "type", "CNAME");
 			}
 			/* no break; */
 
 		case DNS_TYPE_NS:
 			if (type == DNS_TYPE_NS) {
-				add_assoc_string(*subarray, "type", "NS");
+				add_assoc_string(subarray, "type", "NS");
 			}
 			/* no break; */
 
 		case DNS_TYPE_PTR:
 			if (type == DNS_TYPE_PTR) {
-				add_assoc_string(*subarray, "type", "PTR");
+				add_assoc_string(subarray, "type", "PTR");
 			}
-			add_assoc_string(*subarray, "target", pRec->Data.MX.pNameExchange);
+			add_assoc_string(subarray, "target", pRec->Data.MX.pNameExchange);
 			break;
 
 		/* Not available on windows, the query is possible but there is no DNS_HINFO_DATA structure */
@@ -208,12 +209,12 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 				zend_string *txt;
 				char *txt_dst;
 				long txt_len = 0;
-				zval *entries;
+				zval entries;
 
-				add_assoc_string(*subarray, "type", "TXT");
+				add_assoc_string(subarray, "type", "TXT");
+				
+				array_init(entries);
 				
-				ZVAL_NEW_ARR(entries);
-
 				for (i = 0; i < count; i++) {
 					txt_len += strlen(data_txt->pStringArray[i]) + 1;
 				}
@@ -223,12 +224,12 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 				for (i = 0; i < count; i++) {
 					int len = strlen(data_txt->pStringArray[i]);
 					memcpy(txt_dst, data_txt->pStringArray[i], len);
-					add_next_index_stringl(entries, data_txt->pStringArray[i], len);
+					add_next_index_stringl(&entries, data_txt->pStringArray[i], len);
 					txt_dst += len;
 				}
-				txt->len = txt_dst - txt->val;
-				add_assoc_str(*subarray, "txt", txt);
-				add_assoc_zval(*subarray, "entries", entries);
+				tct->len = txt_dst - txt->val;
+				add_assoc_str(subarray, "txt", txt);
+				add_assoc_zval(subarray, "entries", &entries);
 			}
 			break;
 
@@ -236,15 +237,15 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 			{
 				DNS_SOA_DATA *data_soa = &pRec->Data.Soa;
 
-				add_assoc_string(*subarray, "type", "SOA");
+				add_assoc_string(subarray, "type", "SOA");
 
-				add_assoc_string(*subarray, "mname", data_soa->pNamePrimaryServer);
-				add_assoc_string(*subarray, "rname", data_soa->pNameAdministrator);
-				add_assoc_long(*subarray, "serial", data_soa->dwSerialNo);
-				add_assoc_long(*subarray, "refresh", data_soa->dwRefresh);
-				add_assoc_long(*subarray, "retry", data_soa->dwRetry);
-				add_assoc_long(*subarray, "expire", data_soa->dwExpire);
-				add_assoc_long(*subarray, "minimum-ttl", data_soa->dwDefaultTtl);
+				add_assoc_string(subarray, "mname", data_soa->pNamePrimaryServer);
+				add_assoc_string(subarray, "rname", data_soa->pNameAdministrator);
+				add_assoc_long(subarray, "serial", data_soa->dwSerialNo);
+				add_assoc_long(subarray, "refresh", data_soa->dwRefresh);
+				add_assoc_long(subarray, "retry", data_soa->dwRetry);
+				add_assoc_long(subarray, "expire", data_soa->dwExpire);
+				add_assoc_long(subarray, "minimum-ttl", data_soa->dwDefaultTtl);
 			}
 			break;
 
@@ -292,8 +293,8 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 				}
 				tp[0] = '\0';
 
-				add_assoc_string(*subarray, "type", "AAAA");
-				add_assoc_string(*subarray, "ipv6", buf);
+				add_assoc_string(subarray, "type", "AAAA");
+				add_assoc_string(subarray, "ipv6", buf);
 			}
 			break;
 
@@ -307,11 +308,11 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 			{
 				DNS_SRV_DATA *data_srv = &pRec->Data.Srv;
 
-				add_assoc_string(*subarray, "type", "SRV");
-				add_assoc_long(*subarray, "pri", data_srv->wPriority);
-				add_assoc_long(*subarray, "weight", data_srv->wWeight);
-				add_assoc_long(*subarray, "port", data_srv->wPort);
-				add_assoc_string(*subarray, "target", data_srv->pNameTarget);
+				add_assoc_string(subarray, "type", "SRV");
+				add_assoc_long(subarray, "pri", data_srv->wPriority);
+				add_assoc_long(subarray, "weight", data_srv->wWeight);
+				add_assoc_long(subarray, "port", data_srv->wPort);
+				add_assoc_string(subarray, "target", data_srv->pNameTarget);
 			}
 			break;
 
@@ -320,13 +321,13 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 			{
 				DNS_NAPTR_DATA * data_naptr = &pRec->Data.Naptr;
 
-				add_assoc_string(*subarray, "type", "NAPTR");
-				add_assoc_long(*subarray, "order", data_naptr->wOrder);
-				add_assoc_long(*subarray, "pref", data_naptr->wPreference);
-				add_assoc_string(*subarray, "flags", data_naptr->pFlags);
-				add_assoc_string(*subarray, "services", data_naptr->pService);
-				add_assoc_string(*subarray, "regex", data_naptr->pRegularExpression);
-				add_assoc_string(*subarray, "replacement", data_naptr->pReplacement);
+				add_assoc_string(subarray, "type", "NAPTR");
+				add_assoc_long(subarray, "order", data_naptr->wOrder);
+				add_assoc_long(subarray, "pref", data_naptr->wPreference);
+				add_assoc_string(subarray, "flags", data_naptr->pFlags);
+				add_assoc_string(subarray, "services", data_naptr->pService);
+				add_assoc_string(subarray, "regex", data_naptr->pRegularExpression);
+				add_assoc_string(subarray, "replacement", data_naptr->pReplacement);
 			}
 			break;
 #endif
@@ -334,7 +335,7 @@ static void php_parserr(PDNS_RECORD pRec, int type_to_fetch, int store, int raw,
 		default:
 			/* unknown type */
 			zval_ptr_dtor(subarray);
-			*subarray = NULL;
+			ZVAL_UNDEF(subarray);
 			return;
 	}
 
@@ -464,20 +465,20 @@ PHP_FUNCTION(dns_get_record)
 
 			for (pRec = pResult; pRec; pRec = pRec->pNext) {
 				DNS_SRV_DATA *srv = &pRec->Data.Srv;
-				zval *retval = NULL;
+				zval retval;
 
 				if (pRec->Flags.S.Section == DnsSectionAnswer) {
 					php_parserr(pRec, type_to_fetch, store_results, raw, &retval);
-					if (retval != NULL && store_results) {
-						add_next_index_zval(return_value, retval);
+					if (!Z_ISUNDEF(retval) && store_results) {
+						add_next_index_zval(return_value, &retval);
 					}
 				}
 
 				if (authns && pRec->Flags.S.Section == DnsSectionAuthority) {
 
 					php_parserr(pRec, type_to_fetch, 1, raw, &retval);
-					if (retval != NULL) {
-						add_next_index_zval(authns, retval);
+					if (!Z_ISUNDEF(retval)) {
+						add_next_index_zval(authns, &retval);
 					}
 				}
 
@@ -491,8 +492,8 @@ PHP_FUNCTION(dns_get_record)
 #endif
 				if (addtl && pRec->Flags.S.Section == DnsSectionAdditional) {
 					php_parserr(pRec, type_to_fetch, 1, raw, &retval);
-					if (retval != NULL) {
-						add_next_index_zval(addtl, retval);
+					if (!Z_ISUNDEF(retval)) {
+						add_next_index_zval(addtl, &retval);
 					}
 				}
 			}