Bug #7892

MIME encoding bug of NKF.nkf

Added by Kenta Murata about 1 year ago. Updated about 1 year ago.

[ruby-dev:47057]
Status:Open
Priority:Normal
Assignee:Yui NARUSE
Category:ext
Target version:next minor
ruby -v:ruby 2.0.0dev (2013-02-08 trunk 39161) [x86_64-darwin11.4.2] Backport:

Description

NKF の MIME encoding の結果が 1.8 と 1.9/2.0 で異なってます。

1.8 の場合

$ /usr/bin/ruby -rnkf -ve "puts NKF.nkf('-jW -M --cp932', '「あああああああああああ by ああああああああああ」のレシピ')"
ruby 1.8.7 (2012-02-08 patchlevel 358) [universal-darwin11.0]
=?ISO-2022-JP?B?GyRCIVYkIiQiJCIkIiQiJCIkIiQiJCIkIiQiGyhC?= by
=?ISO-2022-JP?B?GyRCJCIkIiQiJCIkIiQiJCIkIiQiJCIhVyROJWwlNyVUGyhC?=

1.9.3-p385 の場合

$ ruby -rnkf -ve "puts NKF.nkf('-jW -M --cp932', '「あああああああああああ by ああああああああああ」のレシピ')"
ruby 1.9.3p385 (2013-02-06 revision 39114) [x86_64-darwin11.4.2]
=?ISO-2022-JP?B?GyRCIVYkIiQiJCIkIiQiJCIkIiQiJCIkIiQiGyhC?= by
=?US-ASCII?Q??=
=?ISO-2022-JP?B?GyRCJCIkIiQiJCIkIiQiJCIkIiQiJCIhVyROJWwlNyVUGyhC?=

2.0.0-rc2 の場合

$ RBENVVERSION=2.0.0-rc2 rbenv exec ruby -rnkf -ve "puts NKF.nkf('-jW -M --cp932', '「あああああああああああ by ああああああああああ」のレシピ')"
ruby 2.0.0dev (2013-02-08 trunk 39161) [x86
64-darwin11.4.2]
=?ISO-2022-JP?B?GyRCIVYkIiQiJCIkIiQiJCIkIiQiJCIkIiQiGyhC?= by
=?US-ASCII?Q??=
=?ISO-2022-JP?B?GyRCJCIkIiQiJCIkIiQiJCIkIiQiJCIhVyROJWwlNyVUGyhC?=

History

#1 Updated by Yusuke Endoh about 1 year ago

  • Target version changed from 2.0.0 to next minor

#2 Updated by Yui NARUSE about 1 year ago

うーん……。

まず。どちらも元に戻らないので、バグです。
で、バグの根本的な原因は、「長い日本語 alphabets 日本語」という文字列をエンコードした際に、
「=?ISO-2022-JP?B?blahblah?= alphabets =?ISO-2022-JP?B?blah?=」とデコードしたいところ、
「=?ISO-2022-JP?B?blahblah?= alphabets<改行>=?ISO-2022-JP?B?blah?=」となってしまうケースを
想定していなかったから、ですね。

後述のパッチで直るような気もするんですが、根本的に改行戦略の考慮が足りてないので、
まじめにアルゴリズムを検討してとりあえずRubyで実装したいところですが、正直 MIME encode ってこれからも使います?

diff --git a/nkf.c b/nkf.c
index 705fb55..d3fde19 100644
--- a/nkf.c
+++ b/nkf.c
@@ -5421,28 +5421,6 @@ mimeputc(nkfchar c)
mimeoutstate.buf[mimeoutstate.count++] = (char)c;
return;
}
- if (nkfisspace(c)) {
- for (i=0;i<mimeout
state.count;i++) {
- if (SPMIMEOUTBUFLENGTH) {
- eofmime();
- for (i=0;i<mimeout
state.count;i++) {
- (*omputc)(mimeoutstate.buf[i]);
- base64count++;
- }
- mimeout
state.count = 0;
- }
- return;
- }
if (mimeoutstate.count>0 && SP<c && c!='=') {
mimeout
state.buf[mimeoutstate.count++] = (char)c;
if (mimeout
state.count>MIMEOUTBUFLENGTH) {

Also available in: Atom PDF