From 08e6eb1d5d8bb809507c405807b9d3411aa68d4e Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Thu, 8 May 2008 11:14:41 -0700 Subject: [PATCH] wininet: handle NULL input for FindCloseUrlCache. --- dlls/wininet/tests/urlcache.c | 11 +++++++++++ dlls/wininet/urlcache.c | 2 +- 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/wininet/tests/urlcache.c b/dlls/wininet/tests/urlcache.c index 6570d5a..08ffebb 100644 --- a/dlls/wininet/tests/urlcache.c +++ b/dlls/wininet/tests/urlcache.c @@ -166,7 +166,18 @@ static void test_urlcacheA(void) ok(!ret && GetLastError() == ERROR_FILE_NOT_FOUND, "local file should no longer exist\n"); } +static void test_FindCloseUrlCache(void) +{ + BOOL r; + DWORD err; + r = FindCloseUrlCache(NULL); + err = GetLastError(); + ok(0 == r, "expected 0, got %d\n", r); + ok(ERROR_INVALID_HANDLE == err, "expected %d, got %d\n", ERROR_INVALID_HANDLE, err); +} + START_TEST(urlcache) { test_urlcacheA(); + test_FindCloseUrlCache(); } diff --git a/dlls/wininet/urlcache.c b/dlls/wininet/urlcache.c index 6b00ebb..6e76893 100644 --- a/dlls/wininet/urlcache.c +++ b/dlls/wininet/urlcache.c @@ -3127,7 +3127,7 @@ BOOL WINAPI FindCloseUrlCache(HANDLE hEnumHandle) TRACE("(%p)\n", hEnumHandle); - if (pEntryHandle->dwMagic != URLCACHE_FIND_ENTRY_HANDLE_MAGIC) + if (!pEntryHandle || pEntryHandle->dwMagic != URLCACHE_FIND_ENTRY_HANDLE_MAGIC) { SetLastError(ERROR_INVALID_HANDLE); return FALSE; -- 1.5.3.6