Re: [1.4.20RC2] case exact match in decodeheader
Paul Lesniewski <[email protected]>
| Newsgroups | gmane.mail.squirrelmail.devel |
|---|---|
| Message-ID | <[email protected]> |
On Sat, Oct 3, 2009 at 7:35 PM, Takahiro Kambe <[email protected]> wrote: > Hi, > > my acquaintance find there is a case exact match in processing > decoding encoded header. > > This problem still exists in 1.4.20RC2 and affects replying/forwarding > a mail. > > Attached patch fix the problem. Case of "q|b" part has changed to the > same as other regular expressions. > - if (preg_match('/=\?([^?]+)\?(q|b)\?([^?]+)\?=/', $ret)) > + if (preg_match('/=\?([^?]+)\?(Q|B)\?([^?]+)\?=/Ui', $ret)) Kanbe-san, thank you. Looks like I forgot to add the i modifier in a couple places where eregi() was being replaced. However, unless you can prove otherwise, I believe the U modifier does nothing here, so it is being left out. This has been committed. Thank you. -- Paul Lesniewski SquirrelMail Team Please support Open Source Software by donating to SquirrelMail! http://squirrelmail.org/donate_paul_lesniewski.php ------------------------------------------------------------------------------ Come build with us! The BlackBerry® Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9-12, 2009. Register now! http://p.sf.net/sfu/devconf ----- squirrelmail-devel mailing list Posting guidelines: http://squirrelmail.org/postingguidelines List address: [email protected] List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel