cvs: php-src(PHP_5_3) /ext/mbstring/libmbfl/filters mbfilter_cp936.c /ext/mbstring/tests bug46843.phpt

moriyoshi Mon Feb 2 04:53:10 2009 UTC Added files: (Branch: PHP_5_3) /php-src/ext/mbstring/tests bug46843.phpt Modified files: /php-src/ext/mbstring/libmbfl/filters mbfilter_cp936.c Log: - MFH: Fix bug #46843 (CP936 euro symbol is not converted properly) http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c?r1=1.1&r2=1.1.10.1&diff_format=u Index: php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c diff -u php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1 php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1.10.1 --- php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1 Sat Aug 23 06:18:36 2003 +++ php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c Mon Feb 2 04:53:10 2009 @@ -192,7 +192,7 @@ } } if (s >= 0) { - if (s < 0x80) { /* latin */ + if (s <= 0x80) { /* latin */ CK((*filter->output_function)(s, filter->data)); } else { CK((*filter->output_function)((s >> 8) & 0xff, filter->data)); http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/tests/bug46843.phpt?view=markup&rev=1.1 Index: php-src/ext/mbstring/tests/bug46843.phpt +++ php-src/ext/mbstring/tests/bug46843.phpt --TEST-- Bug #46843 (CP936 euro symbol is not converted properly) --SKIPIF-- <?php extension_loaded('mbstring') or die('skip mbstring not available'); ?> --FILE-- <?php var_dump(bin2hex(mb_convert_encoding("\x80", 'UCS-2BE', 'CP936'))); var_dump(bin2hex(mb_convert_encoding("\x20\xac", 'CP936', 'UCS-2BE'))); ?> --EXPECT-- string(4) "20ac" string(2) "80"