Re: mod_isapi/3598: ISAPI Redirect, then Apache raise a AV Error`

[wr...@apache.org]

[In order for any reply to be added to the PR database, you need]
[to include <ap...@Apache.Org> in the Cc line and make sure the]
[subject line starts with the report component and number, with ]
[or without any 'Re:' prefixes (such as "general/1098:" or      ]
["Re: general/1098:").  If the subject doesn't match this       ]
[pattern, your message will be misfiled and ignored.  The       ]
["apbugs" address is not added to the Cc line of messages from  ]
[the database automatically because of the potential for mail   ]
[loops.  If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request from a  ]
[developer.  Reply only with text; DO NOT SEND ATTACHMENTS!     ]


Synopsis: ISAPI Redirect, then Apache raise a AV Error`

State-Changed-From-To: open-closed
State-Changed-By: wrowe
State-Changed-When: Tue Sep 12 19:30:07 PDT 2000
State-Changed-Why:

> In mod_isapi.c, the ap_table_set(....) of redirect ought
> to ap_table_add(..), For ISAPI will release the memory 
> of Redirect URL.

Doesn't seem so, since that table itself lives in the same
pool as the request rec.  That is, at least for
HSE_REQ_SEND_URL_REDIRECT_RESP...

On the other hand, HSE_REQ_SEND_URL uses the syntax:
        ap_internal_redirect((char*)lpvBuffer, r);	

This could be a problem, since the ISAPI .dll owns the
lpvBuffer... and although I've been able to repeat only
once (and couldn't catch it in the debugger), I'm committing
a fix to 1.3.13-dev so we ap_pstrdup the argument before
redirecting.

Hope it solves the problem