错误DLL(ISAPI过滤器)

时间:2010-03-12 21:30:01

标签: c memory isapi

我写了这个ISAPI过滤器来重写URL,因为我们有一些移动位置的网站...基本上过滤器查看引用者,如果它是本地服务器,它会查看请求的URL并将其与完整比较引用。如果第一个路径相同,则不执行任何操作,但如果不是,则从完整引用者获取第一个路径并将其添加到URL。例如:来自http://myserver/wr/apps/default.htm的引荐来源的/Content/imgs/img.jpg将被重写为/wr/Content/imgs/img.jpg。

当我查看日志文件时,一切看起来都不错。但是,DLL会保留错误信息以下信息:错误应用程序w3wp.exe,版本6.0.3790.3959,错误模块URLRedirector.dll,版本0.0.0.0,错误地址0x0002df25。

以下是代码:

#include <windows.h>
#include <stdio.h>
#include <stdlib.h>
#include <httpfilt.h>
#include <time.h>
#include <string.h>


#ifdef _DEBUG
#define TO_FILE  // uncomment out to use a log file
#ifdef TO_FILE
#define DEST ghFile
#define DebugMsg(x) WriteToFile x;
HANDLE ghFile;
#define LOGFILE "W:\\Temp\\URLRedirector.log"
void WriteToFile (HANDLE hFile, char *szFormat, ...) {
    char szBuf[1024];
    DWORD dwWritten;
    va_list list;
    va_start (list, szFormat);
    vsprintf (szBuf, szFormat, list);
    hFile = CreateFile (LOGFILE, GENERIC_WRITE, 
                        0, NULL, OPEN_ALWAYS, 
                        FILE_ATTRIBUTE_NORMAL, NULL);
    if (hFile != INVALID_HANDLE_VALUE) {
        SetFilePointer (hFile, 0, NULL, FILE_END);
        WriteFile (hFile, szBuf, lstrlen (szBuf), &dwWritten, NULL);
        CloseHandle (hFile);
    }
    va_end (list);
}
#endif
#endif

BOOL WINAPI __stdcall GetFilterVersion(HTTP_FILTER_VERSION *pVer)
{
    /* Specify the types and order of notification */

    pVer->dwFlags = (SF_NOTIFY_ORDER_HIGH | SF_NOTIFY_SECURE_PORT | SF_NOTIFY_NONSECURE_PORT
                     | SF_NOTIFY_PREPROC_HEADERS | SF_NOTIFY_END_OF_NET_SESSION);

    pVer->dwFilterVersion = HTTP_FILTER_REVISION;

    strcpy(pVer->lpszFilterDesc, "URL Redirector, Version 1.0");

    return TRUE;
}

DWORD WINAPI __stdcall HttpFilterProc(HTTP_FILTER_CONTEXT *pfc, DWORD NotificationType, VOID *pvData)
{
    CHAR *pPhysPath;
    PHTTP_FILTER_URL_MAP pURLMap;
    PHTTP_FILTER_PREPROC_HEADERS pHeaderInfo;
    CHAR szReferrer[255], szServer[255], szURL[255], szNewURL[255];
    DWORD dwRSize = sizeof(szReferrer);
    DWORD dwSSize = sizeof(szServer);
    DWORD dwUSize = sizeof(szURL);
    int iTmp, iTmp2;
    CHAR *pos, tmp[255], *tmp2;

    switch (NotificationType) {

        case SF_NOTIFY_PREPROC_HEADERS :
            pHeaderInfo = (PHTTP_FILTER_PREPROC_HEADERS)pvData;

            if (pfc->GetServerVariable(pfc, "HTTP_REFERER", szReferrer, &dwRSize))
            {
                DebugMsg(( DEST,
                           "Referrer: %s\r\n", szReferrer ));

                if (pfc->GetServerVariable(pfc, "SERVER_NAME", szServer, &dwSSize))
                    DebugMsg(( DEST,
                               "Server Name: %s\r\n", szServer ));

                if (pHeaderInfo->GetHeader(pfc, "URL", szURL, &dwUSize))
                    DebugMsg(( DEST,
                               "URL: %s\r\n", szURL ));

                iTmp = strnstr(szReferrer, szServer, strlen(szReferrer));
                if(iTmp > 0)
                {
                    //Referred is our own server...
                    strcpy(tmp, szReferrer + iTmp);
                    DebugMsg(( DEST,
                               "tmp: %s - %d\r\n", tmp, strlen(tmp) ));
                    pos = strchr(tmp+1, '/');
                    DebugMsg(( DEST,
                               "pos: %s - %d\r\n", pos, strlen(pos) ));

                    iTmp2 = strlen(tmp) - strlen(pos) + 1;

                    strncpy(tmp2, tmp, iTmp2);
                    tmp2[iTmp2] = '\0';
                    DebugMsg(( DEST,
                               "tmp2: %s\r\n", tmp2));

                    if(strncmp(szURL, tmp2, iTmp2) != 0)
                    {
                        //First paths don't match, create new URL...
                        strncpy(szNewURL, tmp2, iTmp2-1);
                        strcat(szNewURL, szURL);
                        DebugMsg(( DEST,
                                   "newURL: %s\r\n", szNewURL));
                        pHeaderInfo->SetHeader(pfc, "URL", szNewURL);
                        return SF_STATUS_REQ_HANDLED_NOTIFICATION;
                    }
                }
            }

            break;

        default :

            break;    
    }

    return SF_STATUS_REQ_NEXT_NOTIFICATION;
}


/* simple function to compare two strings and return the position at which the compare ended */
static int strnstr ( const char *string, const char *strCharSet, int n)
{
    int len = (strCharSet  != NULL ) ? ((int)strlen(strCharSet )) : 0 ;
    int ret, I, J, found;

    if ( 0 == n || 0 == len )
    {
        return -1;
    }

    ret = -1;
    found = 0;
    for (I = 0 ; I <= n - len && found != 1 ; I++)
    {
        J = 0 ;
        for ( ; J < len ; J++ )
        {
            if (toupper(string[I + J]) != toupper(strCharSet [J]))
            {
                break; // Exit For(J)
            }
        }

        if ( J == len)
        {
            ret = I + (J);
            found = 1;
        } 
    }

    return ret;
}

3 个答案:

答案 0 :(得分:2)

如果您真的想编写和维护自定义ISAPI筛选器,我强烈推荐两件事:

  1. 使用Visual Studio进行调试

    如果问题是可重现的,请使用Visual Studio或其他调试程序附加到w3wp.exe以捕获异常。您需要做的是启动IIS并通过它运行请求,以便加载过滤器。确保DLL的pdb文件与DLL位于同一位置 。
    然后,您需要启动Visual Studio,然后附加到调试器中的w3wp.exe进程。然后运行导致故障的请求。在Visual Studio中,调试器将显示导致问题的确切行。

  2. 使用StackWalker

    如果问题不能可靠地重现,您需要安排在问题 发生时记录callstack转储,包括行号,以帮助诊断它。为此,使用codeplex Stackwalker项目。发生异常时,您的ISAPI可以将callstack转储到日志文件中,这样您就可以在代码中找到导致错误的行。

答案 1 :(得分:1)

我知道你在问一个编程问题,但你可以通过放弃编写ISAPI过滤器的努力来避免整个过程,而是使用现成的通用重写器,如IIRF

这是符合你想要的规则:

RewriteCond %{HTTP_REFERER} ^http://localserver/([^/]+)/etc/etc$
RewriteCond $1              !*1
RewriteRule ^/([^/]+)/([^/]+)\.(jpg|gif|png)$    /*1/$1/$2.$3    [L]

答案 2 :(得分:0)

一个潜在的问题是,如果szReferrerszServer是完全相同的值(不确定是否可行)。但如果确实发生了,那么strnstr将返回字符串值的长度。换句话说,它返回第一个字符串的长度。在那种情况下,strcpy进入tmp我认为只将零终止字节放入tmp。以下strchr(tmp+1, ‘/’);具有未定义的行为。从它返回的值可能是任何东西(它实际上可能导致访问冲突,因为它可以读取它应该超出的内存)。事后可能会崩溃。