Fixed a function overloading related bug.

# Yet I suspect the problem is in another place...

diff --git a/ext/mbstring/mbstring.c b/ext/mbstring/mbstring.c
index 4a4c6363a052a920c7a6bdf3d53c529034be0aa7..83d39eadc9c893fd18d47d1844d87ac2e08b12b0 100644 (file)
--- a/ext/mbstring/mbstring.c
+++ b/ext/mbstring/mbstring.c
@@ -916,11 +916,12 @@ PHP_RINIT_FUNCTION(mbstring)
                                if (zend_hash_find(EG(function_table), p->orig_func, 
                                                                   strlen(p->orig_func)+1, (void **)&orig) != SUCCESS) {
                                        php_error_docref("ref.mbstring" TSRMLS_CC, E_ERROR, "mbstring couldn't find function %s.", p->orig_func);
-                               }
-                               zend_hash_add(EG(function_table), p->save_func, strlen(p->save_func)+1, orig, sizeof(zend_function), NULL);
-                               if (zend_hash_update(EG(function_table), p->orig_func, strlen(p->orig_func)+1,
+                               } else {
+                                       zend_hash_add(EG(function_table), p->save_func, strlen(p->save_func)+1, orig, sizeof(zend_function), NULL);
+                                       if (zend_hash_update(EG(function_table), p->orig_func, strlen(p->orig_func)+1,
                                                                         func, sizeof(zend_function), NULL) == FAILURE){
-                                       php_error_docref("ref.mbstring" TSRMLS_CC, E_ERROR, "mbstring couldn't replace function %s.", p->orig_func);
+                                               php_error_docref("ref.mbstring" TSRMLS_CC, E_ERROR, "mbstring couldn't replace function %s.", p->orig_func);
+                                       }
                                }
                        }
                        p++;