From e1791bedd5084754de2b32b45c144d3d8b6a3758 Mon Sep 17 00:00:00 2001 From: Rob Richards Date: Thu, 6 Oct 2005 18:48:19 +0000 Subject: [PATCH] MFH: use sizeof instead of define --- ext/wddx/php_wddx_api.h | 3 --- ext/wddx/wddx.c | 6 +++--- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/ext/wddx/php_wddx_api.h b/ext/wddx/php_wddx_api.h index 3c62f70205..edd788498e 100644 --- a/ext/wddx/php_wddx_api.h +++ b/ext/wddx/php_wddx_api.h @@ -47,9 +47,6 @@ #define WDDX_VAR_S "" #define WDDX_VAR_E "" -#define WDDX_NUMBER_LEN 17 -#define WDDX_VAR_S_LEN 13 - #define php_wddx_add_chunk(packet, str) smart_str_appends(packet, str) #define php_wddx_add_chunk_ex(packet, str, len) smart_str_appendl(packet, str, len) #define php_wddx_add_chunk_static(packet, str) smart_str_appendl(packet, str, sizeof(str)-1) diff --git a/ext/wddx/wddx.c b/ext/wddx/wddx.c index 14da1cb952..2c1dd1dff0 100644 --- a/ext/wddx/wddx.c +++ b/ext/wddx/wddx.c @@ -422,7 +422,7 @@ static void php_wddx_serialize_number(wddx_packet *packet, zval *var) tmp = *var; zval_copy_ctor(&tmp); convert_to_string(&tmp); - snprintf(tmp_buf, Z_STRLEN(tmp) + WDDX_NUMBER_LEN + 1, WDDX_NUMBER, Z_STRVAL(tmp)); + snprintf(tmp_buf, sizeof(tmp_buf), WDDX_NUMBER, Z_STRVAL(tmp)); zval_dtor(&tmp); php_wddx_add_chunk(packet, tmp_buf); @@ -624,8 +624,8 @@ void php_wddx_serialize_var(wddx_packet *packet, zval *var, char *name, int name if (name) { name_esc = php_escape_html_entities(name, name_len, &name_esc_len, 0, ENT_QUOTES, NULL TSRMLS_CC); - tmp_buf = emalloc(name_esc_len + WDDX_VAR_S_LEN + 1); - snprintf(tmp_buf, name_esc_len + WDDX_VAR_S_LEN + 1, WDDX_VAR_S, name_esc); + tmp_buf = emalloc(name_esc_len + sizeof(WDDX_VAR_S)); + snprintf(tmp_buf, name_esc_len + sizeof(WDDX_VAR_S), WDDX_VAR_S, name_esc); php_wddx_add_chunk(packet, tmp_buf); efree(tmp_buf); efree(name_esc); -- 2.40.0