[3/4] winhttp: Manage a session global cookie cache.
Hans Leidekker
hans at codeweavers.com
Tue Sep 16 05:31:35 CDT 2008
diff --git a/dlls/winhttp/Makefile.in b/dlls/winhttp/Makefile.in
index 5617a1d..cf188c7 100644
--- a/dlls/winhttp/Makefile.in
+++ b/dlls/winhttp/Makefile.in
@@ -8,6 +8,7 @@ IMPORTS = wininet kernel32
DELAYIMPORTS = crypt32
C_SRCS = \
+ cookie.c \
handle.c \
main.c \
net.c \
diff --git a/dlls/winhttp/cookie.c b/dlls/winhttp/cookie.c
new file mode 100644
index 0000000..6c0f8b4
--- /dev/null
+++ b/dlls/winhttp/cookie.c
@@ -0,0 +1,281 @@
+/*
+ * Copyright 2008 Hans Leidekker for CodeWeavers
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+#include "config.h"
+#include <stdarg.h>
+
+#include "wine/debug.h"
+#include "wine/list.h"
+
+#include "windef.h"
+#include "winbase.h"
+#include "winhttp.h"
+
+#include "winhttp_private.h"
+
+WINE_DEFAULT_DEBUG_CHANNEL(winhttp);
+
+static domain_t *add_domain( session_t *session, WCHAR *name )
+{
+ domain_t *domain;
+
+ if (!(domain = heap_alloc_zero( sizeof(domain_t) ))) return NULL;
+
+ list_init( &domain->entry );
+ list_init( &domain->cookies );
+
+ domain->name = name;
+ list_add_tail( &session->cookie_cache, &domain->entry );
+
+ TRACE("%s\n", debugstr_w(domain->name));
+ return domain;
+}
+
+static cookie_t *find_cookie( domain_t *domain, const WCHAR *path, const WCHAR *name )
+{
+ struct list *item;
+ cookie_t *cookie;
+
+ LIST_FOR_EACH( item, &domain->cookies )
+ {
+ cookie = LIST_ENTRY( item, cookie_t, entry );
+ if (!strcmpW( cookie->path, path ) && !strcmpiW( cookie->name, name ))
+ {
+ TRACE("found %s=%s\n", debugstr_w(cookie->name), debugstr_w(cookie->value));
+ return cookie;
+ }
+ }
+ return NULL;
+}
+
+static BOOL domain_match( const WCHAR *name, domain_t *domain, BOOL partial )
+{
+ TRACE("comparing %s with %s\n", debugstr_w(name), debugstr_w(domain->name));
+
+ if (partial && !strstrW( name, domain->name )) return FALSE;
+ else if (!partial && strcmpW( name, domain->name )) return FALSE;
+ return TRUE;
+}
+
+static void free_cookie( cookie_t *cookie )
+{
+ heap_free( cookie->name );
+ heap_free( cookie->value );
+ heap_free( cookie->path );
+ heap_free( cookie );
+}
+
+static void delete_cookie( cookie_t *cookie )
+{
+ list_remove( &cookie->entry );
+ free_cookie( cookie );
+}
+
+void delete_domain( domain_t *domain )
+{
+ cookie_t *cookie;
+ struct list *item, *next;
+
+ LIST_FOR_EACH_SAFE( item, next, &domain->cookies )
+ {
+ cookie = LIST_ENTRY( item, cookie_t, entry );
+ delete_cookie( cookie );
+ }
+
+ list_remove( &domain->entry );
+ heap_free( domain->name );
+ heap_free( domain );
+}
+
+static BOOL add_cookie( session_t *session, cookie_t *cookie, WCHAR *domain_name, WCHAR *path )
+{
+ domain_t *domain = NULL;
+ cookie_t *old_cookie;
+ struct list *item;
+
+ LIST_FOR_EACH( item, &session->cookie_cache )
+ {
+ domain = LIST_ENTRY( item, domain_t, entry );
+ if (domain_match( domain_name, domain, FALSE )) break;
+ domain = NULL;
+ }
+ if (!domain)
+ {
+ if (!(domain = add_domain( session, domain_name ))) return FALSE;
+ }
+ else if ((old_cookie = find_cookie( domain, path, cookie->name ))) delete_cookie( old_cookie );
+
+ cookie->path = path;
+ list_add_tail( &domain->cookies, &cookie->entry );
+
+ TRACE("domain %s path %s <- %s=%s\n", debugstr_w(domain_name), debugstr_w(cookie->path),
+ debugstr_w(cookie->name), debugstr_w(cookie->value));
+ return TRUE;
+}
+
+static cookie_t *parse_cookie( const WCHAR *string )
+{
+ cookie_t *cookie;
+ const WCHAR *p;
+ int len;
+
+ if (!(cookie = heap_alloc_zero( sizeof(cookie_t) ))) return NULL;
+
+ list_init( &cookie->entry );
+
+ if (!(p = strchrW( string, '=' )))
+ {
+ WARN("no '=' in %s\n", debugstr_w(string));
+ return NULL;
+ }
+ if (p == string)
+ {
+ WARN("empty cookie name in %s\n", debugstr_w(string));
+ return NULL;
+ }
+ len = p - string;
+ if (!(cookie->name = heap_alloc( (len + 1) * sizeof(WCHAR) )))
+ {
+ heap_free( cookie );
+ return NULL;
+ }
+ memcpy( cookie->name, string, len * sizeof(WCHAR) );
+ cookie->name[len] = 0;
+
+ p++; /* skip '=' */
+ while (*p == ' ') p++;
+
+ len = strlenW( p );
+ if (!(cookie->value = heap_alloc( (len + 1) * sizeof(WCHAR) )))
+ {
+ free_cookie( cookie );
+ return NULL;
+ }
+ memcpy( cookie->value, p, len * sizeof(WCHAR) );
+ cookie->value[len] = 0;
+
+ return cookie;
+}
+
+BOOL set_cookies( request_t *request, const WCHAR *cookies )
+{
+ static const WCHAR pathW[] = {'p','a','t','h',0};
+ static const WCHAR domainW[] = {'d','o','m','a','i','n',0};
+
+ BOOL ret = FALSE;
+ WCHAR *buffer, *p, *q, *r;
+ WCHAR *cookie_domain = NULL, *cookie_path = NULL;
+ session_t *session = request->connect->session;
+ cookie_t *cookie;
+ int len;
+
+ len = strlenW( cookies );
+ if (!(buffer = heap_alloc( (len + 1) * sizeof(WCHAR) ))) return FALSE;
+ strcpyW( buffer, cookies );
+
+ p = buffer;
+ while (*p && *p != ';') p++;
+ if (*p == ';') *p++ = 0;
+ if (!(cookie = parse_cookie( buffer )))
+ {
+ heap_free( buffer );
+ return FALSE;
+ }
+ if ((q = strstrW( p, domainW ))) /* FIXME: do real attribute parsing */
+ {
+ while (*q && *q != '=') q++;
+ if (!*q) goto end;
+
+ r = ++q;
+ while (*r && *r != ';') r++;
+ len = r - q;
+
+ if (!(cookie_domain = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end;
+ memcpy( cookie_domain, q, len * sizeof(WCHAR) );
+ cookie_domain[len] = 0;
+
+ }
+ if ((q = strstrW( p, pathW )))
+ {
+ while (*q && *q != '=') q++;
+ if (!*q) goto end;
+
+ r = ++q;
+ while (*r && *r != ';') r++;
+ len = r - q;
+
+ if (!(cookie_path = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end;
+ memcpy( cookie_path, q, len * sizeof(WCHAR) );
+ cookie_path[len] = 0;
+ }
+ if (!cookie_domain && !(cookie_domain = strdupW( request->connect->servername ))) goto end;
+ if (!cookie_path && !(cookie_path = strdupW( request->path ))) goto end;
+
+ if ((p = strrchrW( cookie_path, '/' )) && p != cookie_path) *p = 0;
+ ret = add_cookie( session, cookie, cookie_domain, cookie_path );
+
+end:
+ if (!ret)
+ {
+ free_cookie( cookie );
+ heap_free( cookie_domain );
+ heap_free( cookie_path );
+ }
+ heap_free( buffer );
+ return ret;
+}
+
+BOOL add_cookie_headers( request_t *request )
+{
+ struct list *domain_cursor;
+ session_t *session = request->connect->session;
+
+ LIST_FOR_EACH( domain_cursor, &session->cookie_cache )
+ {
+ domain_t *domain = LIST_ENTRY( domain_cursor, domain_t, entry );
+ if (domain_match( request->connect->servername, domain, TRUE ))
+ {
+ struct list *cookie_cursor;
+ TRACE("found domain %s\n", debugstr_w(domain->name));
+
+ LIST_FOR_EACH( cookie_cursor, &domain->cookies )
+ {
+ cookie_t *cookie = LIST_ENTRY( cookie_cursor, cookie_t, entry );
+
+ TRACE("comparing path %s with %s\n", debugstr_w(request->path), debugstr_w(cookie->path));
+
+ if (strstrW( request->path, cookie->path ) == request->path)
+ {
+ const WCHAR format[] = {'C','o','o','k','i','e',':',' ','%','s','=','%','s',0};
+ int len;
+ WCHAR *header;
+
+ len = strlenW( cookie->name ) + strlenW( format ) + strlenW( cookie->value );
+ if (!(header = heap_alloc( (len + 1) * sizeof(WCHAR) ))) return FALSE;
+
+ sprintfW( header, format, cookie->name, cookie->value );
+
+ TRACE("%s\n", debugstr_w(header));
+ add_request_headers( request, header, len, WINHTTP_ADDREQ_FLAG_ADD );
+ heap_free( header );
+ }
+ }
+ }
+ }
+ return TRUE;
+}
diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c
index 485ec73..4468963 100644
--- a/dlls/winhttp/request.c
+++ b/dlls/winhttp/request.c
@@ -404,7 +404,7 @@ static BOOL process_header( request_t *request, LPCWSTR field, LPCWSTR value, DW
return TRUE;
}
-static BOOL add_request_headers( request_t *request, LPCWSTR headers, DWORD len, DWORD flags )
+BOOL add_request_headers( request_t *request, LPCWSTR headers, DWORD len, DWORD flags )
{
BOOL ret = FALSE;
WCHAR *buffer, *p, *q;
@@ -824,6 +824,11 @@ static BOOL send_request( request_t *request, LPCWSTR headers, DWORD headers_len
TRACE("failed to add request headers\n");
return FALSE;
}
+ if (!(request->hdr.disable_flags & WINHTTP_DISABLE_COOKIES) && !add_cookie_headers( request ))
+ {
+ WARN("failed to add cookie headers\n");
+ return FALSE;
+ }
if (!(ret = open_connection( request ))) goto end;
if (!(req = build_request_string( request ))) goto end;
@@ -1273,16 +1278,16 @@ static void drain_content( request_t *request )
}
}
-/* copy cookies from response headers to request headers */
-static void add_cookies( request_t *request )
+static void record_cookies( request_t *request )
{
unsigned int i;
for (i = 0; i < request->num_headers; i++)
{
- if (!strcmpiW( request->headers[i].field, attr_set_cookie ) && !request->headers[i].is_request)
+ header_t *set_cookie = &request->headers[i];
+ if (!strcmpiW( set_cookie->field, attr_set_cookie ) && !set_cookie->is_request)
{
- process_header( request, attr_cookie, request->headers[i].value, WINHTTP_ADDREQ_FLAG_ADD, TRUE );
+ set_cookies( request, set_cookie->value );
}
}
}
@@ -1305,13 +1310,14 @@ static BOOL receive_response( request_t *request, BOOL async )
if (!query_headers( request, query, NULL, &request->content_length, &size, NULL ))
request->content_length = ~0UL;
+ if (!(request->hdr.disable_flags & WINHTTP_DISABLE_COOKIES)) record_cookies( request );
+
if (status == 301 || status == 302)
{
if (request->hdr.disable_flags & WINHTTP_DISABLE_REDIRECTS) break;
- drain_content( request );
+ drain_content( request );
if (!(ret = handle_redirect( request ))) break;
- if (!(request->hdr.disable_flags & WINHTTP_DISABLE_COOKIES)) add_cookies( request );
clear_response_headers( request );
ret = send_request( request, NULL, 0, NULL, 0, 0, 0, FALSE ); /* recurse synchronously */
diff --git a/dlls/winhttp/session.c b/dlls/winhttp/session.c
index 35cac0c..04b9b47 100644
--- a/dlls/winhttp/session.c
+++ b/dlls/winhttp/session.c
@@ -65,9 +65,16 @@ BOOL WINAPI WinHttpCheckPlatform( void )
static void session_destroy( object_header_t *hdr )
{
session_t *session = (session_t *)hdr;
+ struct list *item, *next;
+ domain_t *domain;
TRACE("%p\n", session);
+ LIST_FOR_EACH_SAFE( item, next, &session->cookie_cache )
+ {
+ domain = LIST_ENTRY( item, domain_t, entry );
+ delete_domain( domain );
+ }
heap_free( session->agent );
heap_free( session->proxy_server );
heap_free( session->proxy_bypass );
@@ -173,6 +180,7 @@ HINTERNET WINAPI WinHttpOpen( LPCWSTR agent, DWORD access, LPCWSTR proxy, LPCWST
session->hdr.refs = 1;
session->access = access;
session->hdr.redirect_policy = WINHTTP_OPTION_REDIRECT_POLICY_DISALLOW_HTTPS_TO_HTTP;
+ list_init( &session->cookie_cache );
if (agent && !(session->agent = strdupW( agent ))) goto end;
if (proxy && !(session->proxy_server = strdupW( proxy ))) goto end;
diff --git a/dlls/winhttp/winhttp_private.h b/dlls/winhttp/winhttp_private.h
index ee01f4d..0270ea8 100644
--- a/dlls/winhttp/winhttp_private.h
+++ b/dlls/winhttp/winhttp_private.h
@@ -70,6 +70,21 @@ struct _object_header_t
typedef struct
{
+ struct list entry;
+ WCHAR *name;
+ struct list cookies;
+} domain_t;
+
+typedef struct
+{
+ struct list entry;
+ WCHAR *name;
+ WCHAR *value;
+ WCHAR *path;
+} cookie_t;
+
+typedef struct
+{
object_header_t hdr;
LPWSTR agent;
DWORD access;
@@ -77,6 +92,7 @@ typedef struct
LPWSTR proxy_bypass;
LPWSTR proxy_username;
LPWSTR proxy_password;
+ struct list cookie_cache;
} session_t;
typedef struct
@@ -195,6 +211,11 @@ BOOL netconn_secure_connect( netconn_t * );
BOOL netconn_send( netconn_t *, const void *, size_t, int, int * );
const void *netconn_get_certificate( netconn_t * );
+BOOL set_cookies( request_t *, const WCHAR * );
+BOOL add_cookie_headers( request_t * );
+BOOL add_request_headers( request_t *, LPCWSTR, DWORD, DWORD );
+void delete_domain( domain_t * );
+
static inline void *heap_alloc( SIZE_T size )
{
return HeapAlloc( GetProcessHeap(), 0, size );
More information about the wine-patches
mailing list