Roy T. Fielding wrote:
On Jan 5, 2006, at 4:49 AM, [EMAIL PROTECTED] wrote:+ In order to handle empty boundaries, we'll look for the + boundary plus the \n. */ + + boundary_line = apr_pstrcat(p, "--", mail->boundary, "\n", NULL); /* The start boundary */ - bound = ap_strstr(mail->body, mail->boundary); + bound = ap_strstr(mail->body, boundary_line);That seems a bit risky -- MIME parts are supposed to have CRLF for line terminators, but that code will only search for LF on Unix. Would it make more sense to use a regex?
Why not + boundary_line = apr_pstrcat(p, "--", mail->boundary, "\r\n", NULL); which should be far faster than a regex evaluation?
