cvs: php4 /ext/fribidi fribidi.c
| From: | Tal Peer | Date: | Sun, 15 Sep 2002 09:58:35 +0000 |
| Subject: | cvs: php4 /ext/fribidi fribidi.c | ||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-14057@lists.php.net to get a copy of this message | ||
tal Sun Sep 15 05:58:35 2002 EDT
Modified files:
/php4/ext/fribidi fribidi.c
Log:
* More work on returning errors properly and freeing resources
Index: php4/ext/fribidi/fribidi.c
diff -u php4/ext/fribidi/fribidi.c:1.15 php4/ext/fribidi/fribidi.c:1.16
--- php4/ext/fribidi/fribidi.c:1.15 Sun Sep 15 05:32:29 2002
+++ php4/ext/fribidi/fribidi.c Sun Sep 15 05:58:35 2002
@@ -12,11 +12,12 @@
| obtain it through the world-wide-web, please send a note to |
| license@php.net so we can mail you a copy immediately. |
+----------------------------------------------------------------------+
- | Author: Onn Ben-Zvi <onn@zend.com>, onnb@mercury.co.il |
+ | Authors: Onn Ben-Zvi <onnb@mercury.co.il> |
+ | Tal Peer <tal@php.net>
+----------------------------------------------------------------------+
*/
-/* $Id: fribidi.c,v 1.15 2002/09/15 09:32:29 tal Exp $ */
+/* $Id: fribidi.c,v 1.16 2002/09/15 09:58:35 tal Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -236,7 +237,14 @@
fribidi_unicode_to_isiri_3342(u_visual_str, len , outString);
break;
default:
- zend_error(E_ERROR,"unknown character set %d<br />", Z_LVAL_PP(parameter3));
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Unknown charset");
+ efree(u_logical_str);
+ efree(u_visual_str);
+ efree(position_L_to_V_list);
+ efree(position_V_to_L_list);
+ efree(embedding_level_list);
+ efree(outString);
+ RETURN_FALSE;
}