#forgot something

diff --git a/ext/com/COM.c b/ext/com/COM.c
index 4354e9bcc22878fa87b7af29d27b71993ddaf4c7..bba515b5ec436362154b4ae508d8e2099174b3bb 100644 (file)
--- a/ext/com/COM.c
+++ b/ext/com/COM.c
@@ -1595,12 +1595,12 @@ static void do_COM_propput(pval *return_value, comval *obj, pval *arg_property,
                RETURN_NULL();
        }
 
-       dispparams.cArgs = 0;
-       dispparams.cNamedArgs = 0;
+       if (return_value) {
+               dispparams.cArgs = 0;
+               dispparams.cNamedArgs = 0;
 
-       hr = php_COM_invoke(obj, dispid, DISPATCH_PROPERTYGET, &dispparams, var_result, &ErrString TSRMLS_CC);
+               hr = php_COM_invoke(obj, dispid, DISPATCH_PROPERTYGET, &dispparams, var_result, &ErrString TSRMLS_CC);
 
-       if (return_value) {
                if (SUCCEEDED(hr)) {
                        php_variant_to_pval(var_result, return_value, codepage TSRMLS_CC);
                } else {

diff --git a/ext/rpc/com/com_wrapper.c b/ext/rpc/com/com_wrapper.c
index 4354e9bcc22878fa87b7af29d27b71993ddaf4c7..bba515b5ec436362154b4ae508d8e2099174b3bb 100644 (file)
--- a/ext/rpc/com/com_wrapper.c
+++ b/ext/rpc/com/com_wrapper.c
@@ -1595,12 +1595,12 @@ static void do_COM_propput(pval *return_value, comval *obj, pval *arg_property,
                RETURN_NULL();
        }
 
-       dispparams.cArgs = 0;
-       dispparams.cNamedArgs = 0;
+       if (return_value) {
+               dispparams.cArgs = 0;
+               dispparams.cNamedArgs = 0;
 
-       hr = php_COM_invoke(obj, dispid, DISPATCH_PROPERTYGET, &dispparams, var_result, &ErrString TSRMLS_CC);
+               hr = php_COM_invoke(obj, dispid, DISPATCH_PROPERTYGET, &dispparams, var_result, &ErrString TSRMLS_CC);
 
-       if (return_value) {
                if (SUCCEEDED(hr)) {
                        php_variant_to_pval(var_result, return_value, codepage TSRMLS_CC);
                } else {