November 2007 - Wine-Commits

Huw Davies : inetcomm: Implement IMimeBody:[G|S]etCurrentEncoding.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: dd66f04c218052f7fde5366c92289118f8b41512 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dd66f04c218052f7fde5366c9… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:04:08 2007 +0000 inetcomm: Implement IMimeBody:[G|S]etCurrentEncoding. --- dlls/inetcomm/mimeole.c | 18 ++++++++++++++---- dlls/inetcomm/tests/mimeole.c | 5 +++++ 2 files changed, 19 insertions(+), 4 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index dea1597..685356c 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -103,6 +103,7 @@ typedef struct MimeBody DWORD next_prop_id; char *content_pri_type; char *content_sub_type; + ENCODINGTYPE encoding; } MimeBody; static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) @@ -762,16 +763,24 @@ static HRESULT WINAPI MimeBody_GetCurrentEncoding( IMimeBody* iface, ENCODINGTYPE* pietEncoding) { - FIXME("stub\n"); - return E_NOTIMPL; + MimeBody *This = impl_from_IMimeBody(iface); + + TRACE("(%p)->(%p)\n", This, pietEncoding); + + *pietEncoding = This->encoding; + return S_OK; } static HRESULT WINAPI MimeBody_SetCurrentEncoding( IMimeBody* iface, ENCODINGTYPE ietEncoding) { - FIXME("stub\n"); - return E_NOTIMPL; + MimeBody *This = impl_from_IMimeBody(iface); + + TRACE("(%p)->(%d)\n", This, ietEncoding); + + This->encoding = ietEncoding; + return S_OK; } static HRESULT WINAPI MimeBody_GetEstimatedSize( @@ -924,6 +933,7 @@ HRESULT MimeBody_create(IUnknown *outer, void **obj) This->next_prop_id = FIRST_CUSTOM_PROP_ID; This->content_pri_type = NULL; This->content_sub_type = NULL; + This->encoding = IET_7BIT; *obj = (IMimeBody *)&This->lpVtbl; return S_OK; diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index 2c814d2..32dec5c 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -89,6 +89,7 @@ static void test_CreateBody(void) IStream *in; LARGE_INTEGER off; ULARGE_INTEGER pos; + ENCODINGTYPE enc; hr = CoCreateInstance(&CLSID_IMimeBody, NULL, CLSCTX_INPROC_SERVER, &IID_IMimeBody, (void**)&body); ok(hr == S_OK, "ret %08x\n", hr); @@ -107,6 +108,10 @@ static void test_CreateBody(void) hr = IMimeBody_InitNew(body); ok(hr == S_OK, "ret %08x\n", hr); + hr = IMimeBody_GetCurrentEncoding(body, &enc); + ok(hr == S_OK, "ret %08x\n", hr); + ok(enc == IET_7BIT, "encoding %d\n", enc); + hr = IMimeBody_Load(body, in); ok(hr == S_OK, "ret %08x\n", hr); off.QuadPart = 0;

1 0

Huw Davies : inetcomm: Implement IMimeBody:SetData.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: ef59756ace4f21f496f029d7c5f9ccf4c0d3db0f URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef59756ace4f21f496f029d7c… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:04:51 2007 +0000 inetcomm: Implement IMimeBody:SetData. --- dlls/inetcomm/mimeole.c | 44 +++++++++++++++++++++++++++++++++++++- dlls/inetcomm/regsvr.c | 4 ++- dlls/inetcomm/tests/Makefile.in | 2 +- dlls/inetcomm/tests/mimeole.c | 9 ++++++++ 4 files changed, 55 insertions(+), 4 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 685356c..16fe3f4 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -104,6 +104,8 @@ typedef struct MimeBody char *content_pri_type; char *content_sub_type; ENCODINGTYPE encoding; + void *data; + IID data_iid; } MimeBody; static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) @@ -431,6 +433,16 @@ static void empty_new_prop_list(struct list *list) } } +static void release_data(REFIID riid, void *data) +{ + if(!data) return; + + if(IsEqualIID(riid, &IID_IStream)) + IStream_Release((IStream *)data); + else + FIXME("Unhandled data format %s\n", debugstr_guid(riid)); +} + static HRESULT WINAPI MimeBody_QueryInterface(IMimeBody* iface, REFIID riid, void** ppvObject) @@ -480,6 +492,9 @@ static ULONG WINAPI MimeBody_Release(IMimeBody* iface) HeapFree(GetProcessHeap(), 0, This->content_pri_type); HeapFree(GetProcessHeap(), 0, This->content_sub_type); + + release_data(&This->data_iid, This->data); + HeapFree(GetProcessHeap(), 0, This); } @@ -818,8 +833,31 @@ static HRESULT WINAPI MimeBody_SetData( REFIID riid, LPVOID pvObject) { - FIXME("stub\n"); - return E_NOTIMPL; + MimeBody *This = impl_from_IMimeBody(iface); + TRACE("(%p)->(%d, %s, %s, %s %p)\n", This, ietEncoding, debugstr_a(pszPriType), debugstr_a(pszSubType), + debugstr_guid(riid), pvObject); + + if(IsEqualIID(riid, &IID_IStream)) + IStream_AddRef((IStream *)pvObject); + else + { + FIXME("Unhandled object type %s\n", debugstr_guid(riid)); + return E_INVALIDARG; + } + + if(This->data) + FIXME("release old data\n"); + + This->data_iid = *riid; + This->data = pvObject; + + IMimeBody_SetCurrentEncoding(iface, ietEncoding); + + /* FIXME: Update the content type. + If pszPriType == NULL use 'application' + If pszSubType == NULL use 'octet-stream' */ + + return S_OK; } static HRESULT WINAPI MimeBody_EmptyData( @@ -934,6 +972,8 @@ HRESULT MimeBody_create(IUnknown *outer, void **obj) This->content_pri_type = NULL; This->content_sub_type = NULL; This->encoding = IET_7BIT; + This->data = NULL; + This->data_iid = IID_NULL; *obj = (IMimeBody *)&This->lpVtbl; return S_OK; diff --git a/dlls/inetcomm/regsvr.c b/dlls/inetcomm/regsvr.c index 9bd9930..97759fc 100644 --- a/dlls/inetcomm/regsvr.c +++ b/dlls/inetcomm/regsvr.c @@ -18,7 +18,6 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#define INITGUID #include <stdarg.h> #include <string.h> @@ -28,7 +27,10 @@ #include "winuser.h" #include "winreg.h" #include "winerror.h" +#include "objbase.h" +#include "ocidl.h" +#include "initguid.h" #include "inetcomm_private.h" #include "mimeole.h" diff --git a/dlls/inetcomm/tests/Makefile.in b/dlls/inetcomm/tests/Makefile.in index 93e8b3b..9c001cc 100644 --- a/dlls/inetcomm/tests/Makefile.in +++ b/dlls/inetcomm/tests/Makefile.in @@ -4,7 +4,7 @@ SRCDIR = @srcdir@ VPATH = @srcdir@ TESTDLL = inetcomm.dll IMPORTS = inetcomm ole32 kernel32 -EXTRALIBS = +EXTRALIBS = -luuid CTESTS = \ mimeole.c diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index 32dec5c..c425c21 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -125,6 +125,15 @@ static void test_CreateBody(void) hr = IMimeBody_IsContentType(body, NULL, "mixed"); ok(hr == S_OK, "ret %08x\n", hr); + hr = IMimeBody_SetData(body, IET_8BIT, "text", "plain", &IID_IStream, in); + ok(hr == S_OK, "ret %08x\n", hr); + hr = IMimeBody_IsContentType(body, "text", "plain"); + todo_wine + ok(hr == S_OK, "ret %08x\n", hr); + hr = IMimeBody_GetCurrentEncoding(body, &enc); + ok(hr == S_OK, "ret %08x\n", hr); + ok(enc == IET_8BIT, "encoding %d\n", enc); + IMimeBody_Release(body); }

1 0

Rob Shearman : advapi32: Move credential registry reading code to separate functions.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: ca83f61d2ef7ca4472a8be25104d7819d8f534ce URL: http://source.winehq.org/git/wine.git/?a=commit;h=ca83f61d2ef7ca4472a8be251… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Nov 26 19:40:51 2007 +0000 advapi32: Move credential registry reading code to separate functions. --- dlls/advapi32/cred.c | 220 ++++++++++++++++++++++++-------------------------- 1 files changed, 106 insertions(+), 114 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ca83f61d2ef7ca4472a8b…

1 0

Huw Davies : inetcomm: Store a header's parameters as a list.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: 0bcd30773e95f60998b8d642cbfe70a0cf7814a6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0bcd30773e95f60998b8d642c… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:03:08 2007 +0000 inetcomm: Store a header's parameters as a list. --- dlls/inetcomm/mimeole.c | 80 +++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 80 insertions(+), 0 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 8e4df70..71c621f 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -79,8 +79,16 @@ static const property_t default_props[] = typedef struct { struct list entry; + char *name; + char *value; +} param_t; + +typedef struct +{ + struct list entry; const property_t *prop; PROPVARIANT value; + struct list params; } header_t; typedef struct MimeBody @@ -223,6 +231,7 @@ static header_t *read_prop(MimeBody *body, char **ptr) ret = HeapAlloc(GetProcessHeap(), 0, sizeof(*ret)); ret->prop = prop; PropVariantInit(&ret->value); + list_init(&ret->params); *ptr = colon + 1; return ret; @@ -253,6 +262,57 @@ static void unfold_header(char *header, int len) *(start - 1) = '\0'; } +static void add_param(header_t *header, const char *p) +{ + const char *key = p, *value, *cp = p; + param_t *param; + char *name; + + TRACE("got param %s\n", p); + + while (*key == ' ' || *key == '\t' ) key++; + + cp = strchr(key, '='); + if(!cp) + { + WARN("malformed parameter - skipping\n"); + return; + } + + name = HeapAlloc(GetProcessHeap(), 0, cp - key + 1); + memcpy(name, key, cp - key); + name[cp - key] = '\0'; + + value = cp + 1; + + param = HeapAlloc(GetProcessHeap(), 0, sizeof(*param)); + param->name = name; + param->value = strdupA(value); + list_add_tail(&header->params, &param->entry); +} + +static void split_params(header_t *header, char *value) +{ + char *cp = value, *start = value; + int in_quote = 0; + int done_value = 0; + + while(*cp) + { + if(!in_quote && *cp == ';') + { + *cp = '\0'; + if(done_value) add_param(header, start); + done_value = 1; + start = cp + 1; + } + else if(*cp == '"') + in_quote = !in_quote; + cp++; + } + if(done_value) add_param(header, start); +} + static void read_value(header_t *header, char **cur) { char *end = *cur, *value; @@ -271,6 +331,12 @@ static void read_value(header_t *header, char **cur) unfold_header(value, len); TRACE("value %s\n", debugstr_a(value)); + if(header->prop->flags & MPF_HASPARAMS) + { + split_params(header, value); + TRACE("value w/o params %s\n", debugstr_a(value)); + } + header->value.vt = VT_LPSTR; header->value.pszVal = value; @@ -297,6 +363,19 @@ static HRESULT parse_headers(MimeBody *body, IStream *stm) return hr; } +static void emptry_param_list(struct list *list) +{ + param_t *param, *cursor2; + + LIST_FOR_EACH_ENTRY_SAFE(param, cursor2, list, param_t, entry) + { + list_remove(&param->entry); + HeapFree(GetProcessHeap(), 0, param->name); + HeapFree(GetProcessHeap(), 0, param->value); + HeapFree(GetProcessHeap(), 0, param); + } +} + static void empty_header_list(struct list *list) { header_t *header, *cursor2; @@ -305,6 +384,7 @@ static void empty_header_list(struct list *list) { list_remove(&header->entry); PropVariantClear(&header->value); + emptry_param_list(&header->params); HeapFree(GetProcessHeap(), 0, header); } }

1 0

Huw Davies : inetcomm: Implement IMimeBody:IsContentType.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: 19cb4bcd3640694c8901622d343f47bb08ec99f0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=19cb4bcd3640694c8901622d3… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:03:35 2007 +0000 inetcomm: Implement IMimeBody:IsContentType. --- dlls/inetcomm/mimeole.c | 53 +++++++++++++++++++++++++++++++++++++++- dlls/inetcomm/tests/mimeole.c | 7 +++++ 2 files changed, 58 insertions(+), 2 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 71c621f..dea1597 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -101,6 +101,8 @@ typedef struct MimeBody struct list headers; struct list new_props; /* FIXME: This should be in a PropertySchema */ DWORD next_prop_id; + char *content_pri_type; + char *content_sub_type; } MimeBody; static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) @@ -343,6 +345,30 @@ static void read_value(header_t *header, char **cur) *cur = end; } +static void init_content_type(MimeBody *body, header_t *header) +{ + char *slash; + DWORD len; + + if(header->prop->id != PID_HDR_CNTTYPE) + { + ERR("called with header %s\n", header->prop->name); + return; + } + + slash = strchr(header->value.pszVal, '/'); + if(!slash) + { + WARN("malformed context type value\n"); + return; + } + len = slash - header->value.pszVal; + body->content_pri_type = HeapAlloc(GetProcessHeap(), 0, len + 1); + memcpy(body->content_pri_type, header->value.pszVal, len); + body->content_pri_type[len] = '\0'; + body->content_sub_type = strdupA(slash + 1); +} + static HRESULT parse_headers(MimeBody *body, IStream *stm) { char *header_buf, *cur_header_ptr; @@ -357,6 +383,9 @@ static HRESULT parse_headers(MimeBody *body, IStream *stm) { read_value(header, &cur_header_ptr); list_add_tail(&body->headers, &header->entry); + + if(header->prop->id == PID_HDR_CNTTYPE) + init_content_type(body, header); } HeapFree(GetProcessHeap(), 0, header_buf); @@ -448,6 +477,8 @@ static ULONG WINAPI MimeBody_Release(IMimeBody* iface) empty_header_list(&This->headers); empty_new_prop_list(&This->new_props); + HeapFree(GetProcessHeap(), 0, This->content_pri_type); + HeapFree(GetProcessHeap(), 0, This->content_sub_type); HeapFree(GetProcessHeap(), 0, This); } @@ -631,8 +662,24 @@ static HRESULT WINAPI MimeBody_IsContentType( LPCSTR pszPriType, LPCSTR pszSubType) { - FIXME("stub\n"); - return E_NOTIMPL; + MimeBody *This = impl_from_IMimeBody(iface); + + TRACE("(%p)->(%s, %s)\n", This, debugstr_a(pszPriType), debugstr_a(pszSubType)); + if(pszPriType) + { + const char *pri = This->content_pri_type; + if(!pri) pri = "text"; + if(strcasecmp(pri, pszPriType)) return S_FALSE; + } + + if(pszSubType) + { + const char *sub = This->content_sub_type; + if(!sub) sub = "plain"; + if(strcasecmp(sub, pszSubType)) return S_FALSE; + } + + return S_OK; } static HRESULT WINAPI MimeBody_BindToObject( @@ -875,6 +922,8 @@ HRESULT MimeBody_create(IUnknown *outer, void **obj) list_init(&This->headers); list_init(&This->new_props); This->next_prop_id = FIRST_CUSTOM_PROP_ID; + This->content_pri_type = NULL; + This->content_sub_type = NULL; *obj = (IMimeBody *)&This->lpVtbl; return S_OK; diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index 0683356..2c814d2 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -113,6 +113,13 @@ static void test_CreateBody(void) IStream_Seek(in, off, STREAM_SEEK_CUR, &pos); ok(pos.LowPart == 328, "pos %u\n", pos.LowPart); + hr = IMimeBody_IsContentType(body, "multipart", "mixed"); + ok(hr == S_OK, "ret %08x\n", hr); + hr = IMimeBody_IsContentType(body, "text", "plain"); + ok(hr == S_FALSE, "ret %08x\n", hr); + hr = IMimeBody_IsContentType(body, NULL, "mixed"); + ok(hr == S_OK, "ret %08x\n", hr); + IMimeBody_Release(body); }

1 0

Huw Davies : inetcomm: Unfold headers.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: 03ca389550d97f2c3408bf02eeb818e38b656dd4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=03ca389550d97f2c3408bf02e… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:02:35 2007 +0000 inetcomm: Unfold headers. --- dlls/inetcomm/mimeole.c | 28 ++++++++++++++++++++++++++++ 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 5d33537..8e4df70 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -228,6 +228,31 @@ static header_t *read_prop(MimeBody *body, char **ptr) return ret; } +static void unfold_header(char *header, int len) +{ + char *start = header, *cp = header; + + do { + while(*cp == ' ' || *cp == '\t') + { + cp++; + len--; + } + if(cp != start) + memmove(start, cp, len + 1); + + cp = strstr(start, "\r\n"); + len -= (cp - start); + start = cp; + *start = ' '; + start++; + len--; + cp += 2; + } while(*cp == ' ' || *cp == '\t'); + + *(start - 1) = '\0'; +} + static void read_value(header_t *header, char **cur) { char *end = *cur, *value; @@ -243,6 +268,9 @@ static void read_value(header_t *header, char **cur) memcpy(value, *cur, len); value[len] = '\0'; + unfold_header(value, len); + TRACE("value %s\n", debugstr_a(value)); + header->value.vt = VT_LPSTR; header->value.pszVal = value;

1 0

Huw Davies : inetcomm: Copy RFC822 headers into a memory block for later parsing.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: 98dfca50bbc4109a921c4fcf5aa7d7376abcb4b6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=98dfca50bbc4109a921c4fcf5… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:01:12 2007 +0000 inetcomm: Copy RFC822 headers into a memory block for later parsing. --- dlls/inetcomm/mimeole.c | 90 +++++++++++++++++++++++++++++++++++++++-- dlls/inetcomm/tests/mimeole.c | 45 ++++++++++++++++++++ 2 files changed, 131 insertions(+), 4 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 56ee5d9..28460ba 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -2,6 +2,7 @@ * MIME OLE Interfaces * * Copyright 2006 Robert Shearman for CodeWeavers + * Copyright 2007 Huw Davies for CodeWeavers * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -49,6 +50,86 @@ static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) return (MimeBody *)((char*)iface - FIELD_OFFSET(MimeBody, lpVtbl)); } +#define PARSER_BUF_SIZE 1024 + +/***************************************************** + * copy_headers_to_buf [internal] + * + * Copies the headers into a '\0' terminated memory block and leave + * the stream's current position set to after the blank line. + */ +static HRESULT copy_headers_to_buf(IStream *stm, char **ptr) +{ + char *buf = NULL; + DWORD size = PARSER_BUF_SIZE, offset = 0, last_end = 0; + HRESULT hr; + int done = 0; + + *ptr = NULL; + + do + { + char *end; + DWORD read; + + if(!buf) + buf = HeapAlloc(GetProcessHeap(), 0, size + 1); + else + { + size *= 2; + buf = HeapReAlloc(GetProcessHeap(), 0, buf, size + 1); + } + if(!buf) + { + hr = E_OUTOFMEMORY; + goto fail; + } + + hr = IStream_Read(stm, buf + offset, size - offset, &read); + if(FAILED(hr)) goto fail; + + offset += read; + buf[offset] = '\0'; + + if(read == 0) done = 1; + + while(!done && (end = strstr(buf + last_end, "\r\n"))) + { + DWORD new_end = end - buf + 2; + if(new_end - last_end == 2) + { + LARGE_INTEGER off; + off.QuadPart = new_end; + IStream_Seek(stm, off, STREAM_SEEK_SET, NULL); + buf[new_end] = '\0'; + done = 1; + } + else + last_end = new_end; + } + } while(!done); + + *ptr = buf; + return S_OK; + +fail: + HeapFree(GetProcessHeap(), 0, buf); + return hr; +} + +static HRESULT parse_headers(MimeBody *body, IStream *stm) +{ + char *header_buf; + HRESULT hr; + + hr = copy_headers_to_buf(stm, &header_buf); + if(FAILED(hr)) return hr; + + HeapFree(GetProcessHeap(), 0, header_buf); + return hr; +} + + static HRESULT WINAPI MimeBody_QueryInterface(IMimeBody* iface, REFIID riid, void** ppvObject) @@ -119,8 +200,9 @@ static HRESULT WINAPI MimeBody_Load( IMimeBody* iface, LPSTREAM pStm) { - FIXME("(%p)->(%p): stub\n", iface, pStm); - return E_NOTIMPL; + MimeBody *This = impl_from_IMimeBody(iface); + TRACE("(%p)->(%p)\n", iface, pStm); + return parse_headers(This, pStm); } static HRESULT WINAPI MimeBody_Save( @@ -143,8 +225,8 @@ static HRESULT WINAPI MimeBody_GetSizeMax( static HRESULT WINAPI MimeBody_InitNew( IMimeBody* iface) { - FIXME("stub\n"); - return E_NOTIMPL; + TRACE("%p->()\n", iface); + return S_OK; } static HRESULT WINAPI MimeBody_GetPropInfo( diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index 156884e..0683356 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -32,6 +32,32 @@ #include "wine/test.h" +static char msg1[] = + "MIME-Version: 1.0\r\n" + "Content-Type: multipart/mixed;\r\n" + " boundary=\"------------1.5.0.6\";\r\n" + " stuff=\"du;nno\"\r\n" + "foo: bar\r\n" + "From: Huw Davies <huw(a)codeweavers.com>\r\n" + "From: Me <xxx(a)codeweavers.com>\r\n" + "To: wine-patches <wine-patches(a)winehq.org>\r\n" + "Cc: Huw Davies <huw(a)codeweavers.com>,\r\n" + " \"Fred Bloggs\" <fred(a)bloggs.com>\r\n" + "foo: baz\r\n" + "bar: fum\r\n" + "\r\n" + "This is a multi-part message in MIME format.\r\n" + "--------------1.5.0.6\r\n" + "Content-Type: text/plain; format=fixed; charset=UTF-8\r\n" + "Content-Transfer-Encoding: 8bit\r\n" + "\r\n" + "Stuff\r\n" + "--------------1.5.0.6\r\n" + "Content-Type: text/plain; charset=\"us-ascii\"\r\n" + "Content-Transfer-Encoding: 7bit\r\n" + "\r\n" + "More stuff\r\n" + "--------------1.5.0.6--\r\n"; static void test_CreateVirtualStream(void) { @@ -60,6 +86,9 @@ static void test_CreateBody(void) HRESULT hr; IMimeBody *body; HBODY handle = (void *)0xdeadbeef; + IStream *in; + LARGE_INTEGER off; + ULARGE_INTEGER pos; hr = CoCreateInstance(&CLSID_IMimeBody, NULL, CLSCTX_INPROC_SERVER, &IID_IMimeBody, (void**)&body); ok(hr == S_OK, "ret %08x\n", hr); @@ -68,6 +97,22 @@ static void test_CreateBody(void) ok(hr == MIME_E_NO_DATA, "ret %08x\n", hr); ok(handle == NULL, "handle %p\n", handle); + hr = CreateStreamOnHGlobal(NULL, TRUE, &in); + ok(hr == S_OK, "ret %08x\n", hr); + IStream_Write(in, msg1, sizeof(msg1) - 1, NULL); + off.QuadPart = 0; + IStream_Seek(in, off, STREAM_SEEK_SET, NULL); + + /* Need to call InitNew before Load otherwise Load crashes with native inetcomm */ + hr = IMimeBody_InitNew(body); + ok(hr == S_OK, "ret %08x\n", hr); + + hr = IMimeBody_Load(body, in); + ok(hr == S_OK, "ret %08x\n", hr); + off.QuadPart = 0; + IStream_Seek(in, off, STREAM_SEEK_CUR, &pos); + ok(pos.LowPart == 328, "pos %u\n", pos.LowPart); + IMimeBody_Release(body); }

1 0

Huw Davies : inetcomm: Parse headers into a list.
by Alexandre Julliard Nov. 27, 2007

Nov. 27, 2007

Module: wine Branch: master Commit: 31297b6a3b20a73be36978b6036846a035b3fd07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=31297b6a3b20a73be36978b60… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 26 12:01:52 2007 +0000 inetcomm: Parse headers into a list. --- dlls/inetcomm/mimeole.c | 173 ++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 172 insertions(+), 1 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 28460ba..5d33537 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -31,18 +31,68 @@ #include "ole2.h" #include "mimeole.h" +#include "wine/list.h" #include "wine/debug.h" #include "inetcomm_private.h" WINE_DEFAULT_DEBUG_CHANNEL(inetcomm); +typedef struct +{ + LPCSTR name; + DWORD id; + DWORD flags; /* MIMEPROPFLAGS */ + VARTYPE default_vt; +} property_t; + +typedef struct +{ + struct list entry; + property_t prop; +} property_list_entry_t; + +static const property_t default_props[] = +{ + {"References", PID_HDR_REFS, 0, VT_LPSTR}, + {"Subject", PID_HDR_SUBJECT, 0, VT_LPSTR}, + {"From", PID_HDR_FROM, MPF_ADDRESS, VT_LPSTR}, + {"Message-ID", PID_HDR_MESSAGEID, 0, VT_LPSTR}, + {"Return-Path", PID_HDR_RETURNPATH, MPF_ADDRESS, VT_LPSTR}, + {"Date", PID_HDR_DATE, 0, VT_LPSTR}, + {"Received", PID_HDR_RECEIVED, 0, VT_LPSTR}, + {"Reply-To", PID_HDR_REPLYTO, MPF_ADDRESS, VT_LPSTR}, + {"X-Mailer", PID_HDR_XMAILER, 0, VT_LPSTR}, + {"Bcc", PID_HDR_BCC, MPF_ADDRESS, VT_LPSTR}, + {"MIME-Version", PID_HDR_MIMEVER, MPF_MIME, VT_LPSTR}, + {"Content-Type", PID_HDR_CNTTYPE, MPF_MIME | MPF_HASPARAMS, VT_LPSTR}, + {"Content-Transfer-Encoding", PID_HDR_CNTXFER, MPF_MIME, VT_LPSTR}, + {"Content-ID", PID_HDR_CNTID, MPF_MIME, VT_LPSTR}, + {"Content-Disposition", PID_HDR_CNTDISP, MPF_MIME, VT_LPSTR}, + {"To", PID_HDR_TO, MPF_ADDRESS, VT_LPSTR}, + {"Cc", PID_HDR_CC, MPF_ADDRESS, VT_LPSTR}, + {"Sender", PID_HDR_SENDER, MPF_ADDRESS, VT_LPSTR}, + {"In-Reply-To", PID_HDR_INREPLYTO, 0, VT_LPSTR}, + {NULL, 0, 0, 0} +}; + +typedef struct +{ + struct list entry; + const property_t *prop; + PROPVARIANT value; +} header_t; + typedef struct MimeBody { const IMimeBodyVtbl *lpVtbl; LONG refs; HBODY handle; + + struct list headers; + struct list new_props; /* FIXME: This should be in a PropertySchema */ + DWORD next_prop_id; } MimeBody; static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) @@ -50,6 +100,15 @@ static inline MimeBody *impl_from_IMimeBody( IMimeBody *iface ) return (MimeBody *)((char*)iface - FIELD_OFFSET(MimeBody, lpVtbl)); } +static LPSTR strdupA(LPCSTR str) +{ + char *ret; + int len = strlen(str); + ret = HeapAlloc(GetProcessHeap(), 0, len + 1); + memcpy(ret, str, len + 1); + return ret; +} + #define PARSER_BUF_SIZE 1024 /***************************************************** @@ -117,18 +176,122 @@ fail: return hr; } +static header_t *read_prop(MimeBody *body, char **ptr) +{ + char *colon = strchr(*ptr, ':'); + const property_t *prop; + header_t *ret; + + if(!colon) return NULL; + + *colon = '\0'; + + for(prop = default_props; prop->name; prop++) + { + if(!strcasecmp(*ptr, prop->name)) + { + TRACE("%s: found match with default property id %d\n", *ptr, prop->id); + break; + } + } + + if(!prop->name) + { + property_list_entry_t *prop_entry; + LIST_FOR_EACH_ENTRY(prop_entry, &body->new_props, property_list_entry_t, entry) + { + if(!strcasecmp(*ptr, prop_entry->prop.name)) + { + TRACE("%s: found match with already added new property id %d\n", *ptr, prop_entry->prop.id); + prop = &prop_entry->prop; + break; + } + } + if(!prop->name) + { + prop_entry = HeapAlloc(GetProcessHeap(), 0, sizeof(*prop_entry)); + prop_entry->prop.name = strdupA(*ptr); + prop_entry->prop.id = body->next_prop_id++; + prop_entry->prop.flags = 0; + prop_entry->prop.default_vt = VT_LPSTR; + list_add_tail(&body->new_props, &prop_entry->entry); + prop = &prop_entry->prop; + TRACE("%s: allocating new prop id %d\n", *ptr, prop_entry->prop.id); + } + } + + ret = HeapAlloc(GetProcessHeap(), 0, sizeof(*ret)); + ret->prop = prop; + PropVariantInit(&ret->value); + *ptr = colon + 1; + + return ret; +} + +static void read_value(header_t *header, char **cur) +{ + char *end = *cur, *value; + DWORD len; + + do { + end = strstr(end, "\r\n"); + end += 2; + } while(*end == ' ' || *end == '\t'); + + len = end - *cur; + value = HeapAlloc(GetProcessHeap(), 0, len + 1); + memcpy(value, *cur, len); + value[len] = '\0'; + + header->value.vt = VT_LPSTR; + header->value.pszVal = value; + + *cur = end; +} + static HRESULT parse_headers(MimeBody *body, IStream *stm) { - char *header_buf; + char *header_buf, *cur_header_ptr; HRESULT hr; + header_t *header; hr = copy_headers_to_buf(stm, &header_buf); if(FAILED(hr)) return hr; + cur_header_ptr = header_buf; + while((header = read_prop(body, &cur_header_ptr))) + { + read_value(header, &cur_header_ptr); + list_add_tail(&body->headers, &header->entry); + } + HeapFree(GetProcessHeap(), 0, header_buf); return hr; } +static void empty_header_list(struct list *list) +{ + header_t *header, *cursor2; + + LIST_FOR_EACH_ENTRY_SAFE(header, cursor2, list, header_t, entry) + { + list_remove(&header->entry); + PropVariantClear(&header->value); + HeapFree(GetProcessHeap(), 0, header); + } +} + +static void empty_new_prop_list(struct list *list) +{ + property_list_entry_t *prop, *cursor2; + + LIST_FOR_EACH_ENTRY_SAFE(prop, cursor2, list, property_list_entry_t, entry) + { + list_remove(&prop->entry); + HeapFree(GetProcessHeap(), 0, (char *)prop->prop.name); + HeapFree(GetProcessHeap(), 0, prop); + } +} static HRESULT WINAPI MimeBody_QueryInterface(IMimeBody* iface, REFIID riid, @@ -174,6 +337,9 @@ static ULONG WINAPI MimeBody_Release(IMimeBody* iface) refs = InterlockedDecrement(&This->refs); if (!refs) { + empty_header_list(&This->headers); + empty_new_prop_list(&This->new_props); + HeapFree(GetProcessHeap(), 0, This); } @@ -582,6 +748,8 @@ static IMimeBodyVtbl body_vtbl = MimeBody_GetHandle }; +#define FIRST_CUSTOM_PROP_ID 0x100 + HRESULT MimeBody_create(IUnknown *outer, void **obj) { MimeBody *This; @@ -596,6 +764,9 @@ HRESULT MimeBody_create(IUnknown *outer, void **obj) This->lpVtbl = &body_vtbl; This->refs = 1; This->handle = NULL; + list_init(&This->headers); + list_init(&This->new_props); + This->next_prop_id = FIRST_CUSTOM_PROP_ID; *obj = (IMimeBody *)&This->lpVtbl; return S_OK;

1 0

Alexander Nicolaysen Sørnes : vendor: Make tables sortable
by Chris Morgan Nov. 27, 2007

Nov. 27, 2007

Module: appdb Branch: master Commit: 9d5882d8008df54680183d656a092b76fb693895 URL: http://source.winehq.org/git/appdb.git/?a=commit;h=9d5882d8008df54680183d65… Author: Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> Date: Mon Nov 26 23:47:51 2007 +0100 vendor: Make tables sortable --- include/vendor.php | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/include/vendor.php b/include/vendor.php index 4ec6597..409467f 100644 --- a/include/vendor.php +++ b/include/vendor.php @@ -213,7 +213,12 @@ class Vendor { echo '<input type="hidden" name="iVendorId" value="'.$this->iVendorId.'">',"\n"; } - function objectGetEntries($bQueued, $bRejected, $iRows = 0, $iStart = 0) + public static function objectGetSortableFields() + { + return array('vendorName'); + } + + function objectGetEntries($bQueued, $bRejected, $iRows = 0, $iStart = 0, $sOrderBy = 'vendorName', $bAscending = TRUE) { /* Vendor queueing is not implemented yet */ if($bQueued) @@ -223,11 +228,13 @@ class Vendor { if($bRejected) return FALSE; + $sOrder = $bAscending ? 'ASC' : 'DESC'; + if(!$iRows) $iRows = Vendor::objectGetEntriesCount($bQueued, $bRejected); $hResult = query_parameters("SELECT * FROM vendor - ORDER BY vendorName LIMIT ?,?", + ORDER BY $sOrderBy $sOrder LIMIT ?,?", $iStart, $iRows); if(!$hResult) @@ -238,9 +245,9 @@ class Vendor { function objectGetHeader() { - $oTableRow = new TableRow(); + $oTableRow = new TableRowSortable(); - $oTableRow->AddTextCell("Name"); + $oTableRow->AddSortableTextCell('Name', 'vendorName'); $oTableRow->AddTextCell("Website");

1 0

Alexander Nicolaysen Sørnes : distribution: Make tables sortable
by Chris Morgan Nov. 27, 2007

Nov. 27, 2007

Module: appdb Branch: master Commit: 6361b26cb206e1d973987aec585931d852d9dab1 URL: http://source.winehq.org/git/appdb.git/?a=commit;h=6361b26cb206e1d973987aec… Author: Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> Date: Mon Nov 26 23:43:27 2007 +0100 distribution: Make tables sortable --- include/distribution.php | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/include/distribution.php b/include/distribution.php index e82ab54..ef2c5be 100644 --- a/include/distribution.php +++ b/include/distribution.php @@ -492,9 +492,9 @@ class distribution { function objectGetHeader() { - $oTableRow = new TableRow(); + $oTableRow = new TableRowSortable(); - $oTableRow->AddTextCell("Distribution name"); + $oTableRow->AddSortableTextCell("Distribution name", "name"); $oTableRow->AddTextCell("Distribution url"); @@ -505,7 +505,12 @@ class distribution { return $oTableRow; } - function objectGetEntries($bQueued, $bRejected, $iRows = 0, $iStart = 0) + public static function objectGetSortableFields() + { + return array('name'); + } + + function objectGetEntries($bQueued, $bRejected, $iRows = 0, $iStart = 0, $sOrderBy = "name", $bAscending = TRUE) { /* Not implemented */ if($bRejected) @@ -516,12 +521,14 @@ class distribution { if($bQueued && !distribution::canEdit()) return NULL; + $sOrder = $bAscending ? "ASC" : "DESC"; + /* If row limit is 0 we want to fetch all rows */ if(!$iRows) $iRows = distribution::objectGetEntriesCount($bQueued, $bRejected); $sQuery = "SELECT * FROM distributions - WHERE queued = '?' ORDER BY name LIMIT ?,?"; + WHERE queued = '?' ORDER BY $sOrderBy $sOrder LIMIT ?,?"; return query_parameters($sQuery, $bQueued ? "true" : "false", $iStart, $iRows);

1 0