<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] fix charset-hook bugs



On Wednesday, 02 August 2006 at 12:05, Alain Bench wrote:
> Hello,
> 
>     While testing <create-alias> recoding, I noticed a bug in the IDN
> code, where conversion between IDN and display was unwantedly sensible
> to charset-hooks. Example: When user has "charset-hook . euc-jp",
> because he receives many mails containing EUC-JP charset with various
> wrong MIME labels, a domain "xn--ren-dma.org" is displayed "ren?«±.org"
> instead of the correct "rené.org". And when he enters "rené.org", it's
> idnized to wrong "xn--ren-c03j.org". Exact wrongs depend on $charset.
> 
>     I believe that such conversions, between constant "utf-8" and the
> $charset provided by user, both under full control, should not be
> subject to charset-hooks. It's not needed (any corrections can be done
> directly on $charset, or indirectly thru iconv-hooks), and can create
> unwanted interferences in some perfectly sensible setups.
> Charset-hooking iso-8859-1 to whatever else intends to correct wrong
> MIME charset=iso-8859-1 labels; It should not break any operations in an
> already correct iso-8859-1 $charset. Should be subject to charset-hooks
> *only* uncertain fromcodes coming from the wild Internet.
> 
>     The attached patch-1.5.12.ab.M_ICONV_HOOK_sanitize.1 corrects this
> by removing M_ICONV_HOOK_{FROM,TO} flags usage from IDN, PGP, and GPGME
> code, where not appropriate. It also removes completely the
> M_ICONV_HOOK_TO symbol and depending code, because charset-hooking a
> tocode is never wanted. It adds comments about role of flag
> M_ICONV_HOOK_FROM, and its misleading name (ICONV_HOOK, when it
> exclusively triggers charset-hooks), hoping less misusage in the future.
> And while at it, the patch corrects a typo and normalizes casing of some
> constant charset names.
> 
>     This patch applies after patch-1.5.12.msyk.iconvhook.1-ab
> (collisions in comments otherwise).

In tip now, thanks.

Attachment: pgpUbWfP0GMGx.pgp
Description: PGP signature