wine-commits April 2013

wine-commits@winehq.org

2 participants
579 discussions

Piotr Caban : wininet: Add test of reading gzipped content from cache.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: 8dfd89c2e00cfad8373b5f598111237ecf154030 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8dfd89c2e00cfad8373b5f598… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 17:00:36 2013 +0200 wininet: Add test of reading gzipped content from cache. --- dlls/wininet/tests/http.c | 140 +++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 140 insertions(+), 0 deletions(-) diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c index ceddde1..664fbbf 100644 --- a/dlls/wininet/tests/http.c +++ b/dlls/wininet/tests/http.c @@ -1815,6 +1815,8 @@ struct server_info { int port; }; +static int test_cache_gzip; + static DWORD CALLBACK server_thread(LPVOID param) { struct server_info *si = param; @@ -2029,6 +2031,16 @@ static DWORD CALLBACK server_thread(LPVOID param) static const char no_cache_response[] = "HTTP/1.1 200 OK\r\nCache-Control: junk, \t No-StOrE\r\n\r\nsome content"; send(c, no_cache_response, sizeof(no_cache_response)-1, 0); } + if (strstr(buffer, "GET /test_cache_gzip")) + { + static const char gzip_response[] = "HTTP/1.1 200 OK\r\nContent-Encoding: gzip\r\nContent-Type: text/html\r\n\r\n" + "\x1f\x8b\x08\x00\x00\x00\x00\x00\x00\x03\x4b\xaf\xca\x2c\x50\x28" + "\x49\x2d\x2e\xe1\x02\x00\x62\x92\xc7\x6c\x0a\x00\x00\x00"; + if(!test_cache_gzip++) + send(c, gzip_response, sizeof(gzip_response), 0); + else + send(c, notokmsg, sizeof(notokmsg)-1, 0); + } if (strstr(buffer, "GET /test_premature_disconnect")) trace("closing connection\n"); @@ -2765,6 +2777,133 @@ static void test_no_cache(int port) InternetCloseHandle(ses); } +static void test_cache_read_gzipped(int port) +{ + static const char cache_url_fmt[] = "http://localhost:%d%s"; + static const char get_gzip[] = "/test_cache_gzip"; + static const char content[] = "gzip test\n"; + static const char text_html[] = "text/html"; + static const char raw_header[] = "HTTP/1.1 200 OK\r\nContent-Type: text/html\r\n\r\n"; + + HINTERNET ses, con, req; + DWORD read, size; + char cache_url[256], buf[256]; + BOOL ret; + + trace("Testing reading compressed content from cache\n"); + + ses = InternetOpen("winetest", INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, 0); + ok(ses != NULL,"InternetOpen failed with error %u\n", GetLastError()); + + con = InternetConnect(ses, "localhost", port, NULL, NULL, INTERNET_SERVICE_HTTP, 0, 0); + ok(con != NULL, "InternetConnect failed with error %u\n", GetLastError()); + + req = HttpOpenRequest(con, NULL, get_gzip, NULL, NULL, NULL, 0, 0); + ok(req != NULL, "HttpOpenRequest failed\n"); + + ret = TRUE; + ret = InternetSetOption(req, INTERNET_OPTION_HTTP_DECODING, &ret, sizeof(ret)); + if(!ret && GetLastError()==ERROR_INTERNET_INVALID_OPTION) { + win_skip("INTERNET_OPTION_HTTP_DECODING not supported\n"); + InternetCloseHandle(req); + InternetCloseHandle(con); + InternetCloseHandle(ses); + return; + } + ok(ret, "InternetSetOption(INTERNET_OPTION_HTTP_DECODING) failed: %d\n", GetLastError()); + + ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0); + ok(ret, "HttpSendRequest failed with error %u\n", GetLastError()); + size = 0; + while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read) + size += read; + ok(size == 10, "read %d bytes of data\n", size); + buf[size] = 0; + ok(!strncmp(buf, content, size), "incorrect page content: %s\n", buf); + + size = sizeof(buf); + ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_TYPE, buf, &size, 0); + ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError()); + buf[size] = 0; + ok(!strncmp(text_html, buf, size), "buf = %s\n", buf); + + size = sizeof(buf); + ret = HttpQueryInfoA(req, HTTP_QUERY_RAW_HEADERS_CRLF, buf, &size, 0); + ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError()); + buf[size] = 0; + ok(!strncmp(raw_header, buf, size), "buf = %s\n", buf); + InternetCloseHandle(req); + + req = HttpOpenRequest(con, NULL, get_gzip, NULL, NULL, NULL, INTERNET_FLAG_FROM_CACHE, 0); + ok(req != NULL, "HttpOpenRequest failed\n"); + + ret = TRUE; + ret = InternetSetOption(req, INTERNET_OPTION_HTTP_DECODING, &ret, sizeof(ret)); + ok(ret, "InternetSetOption(INTERNET_OPTION_HTTP_DECODING) failed: %d\n", GetLastError()); + + ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0); + ok(ret, "HttpSendRequest failed with error %u\n", GetLastError()); + size = 0; + while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read) + size += read; + todo_wine ok(size == 10, "read %d bytes of data\n", size); + buf[size] = 0; + ok(!strncmp(buf, content, size), "incorrect page content: %s\n", buf); + + size = sizeof(buf); + ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_ENCODING, buf, &size, 0); + ok(!ret && GetLastError()==ERROR_HTTP_HEADER_NOT_FOUND, + "HttpQueryInfo(HTTP_QUERY_CONTENT_ENCODING) returned %d, %d\n", + ret, GetLastError()); + + size = sizeof(buf); + ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_TYPE, buf, &size, 0); + todo_wine ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError()); + buf[size] = 0; + todo_wine ok(!strncmp(text_html, buf, size), "buf = %s\n", buf); + InternetCloseHandle(req); + + /* Decompression doesn't work while reading from cache */ + test_cache_gzip = 0; + sprintf(cache_url, cache_url_fmt, port, get_gzip); + DeleteUrlCacheEntry(cache_url); + + req = HttpOpenRequest(con, NULL, get_gzip, NULL, NULL, NULL, 0, 0); + ok(req != NULL, "HttpOpenRequest failed\n"); + + ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0); + ok(ret, "HttpSendRequest failed with error %u\n", GetLastError()); + size = 0; + while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read) + size += read; + ok(size == 31, "read %d bytes of data\n", size); + InternetCloseHandle(req); + + req = HttpOpenRequest(con, NULL, get_gzip, NULL, NULL, NULL, INTERNET_FLAG_FROM_CACHE, 0); + ok(req != NULL, "HttpOpenRequest failed\n"); + + ret = TRUE; + ret = InternetSetOption(req, INTERNET_OPTION_HTTP_DECODING, &ret, sizeof(ret)); + ok(ret, "InternetSetOption(INTERNET_OPTION_HTTP_DECODING) failed: %d\n", GetLastError()); + + ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0); + ok(ret, "HttpSendRequest failed with error %u\n", GetLastError()); + size = 0; + while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read) + size += read; + todo_wine ok(size == 31, "read %d bytes of data\n", size); + + size = sizeof(buf); + ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_ENCODING, buf, &size, 0); + todo_wine ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_ENCODING) failed: %d\n", GetLastError()); + InternetCloseHandle(req); + + InternetCloseHandle(con); + InternetCloseHandle(ses); + + DeleteUrlCacheEntry(cache_url); +} + static void test_HttpSendRequestW(int port) { static const WCHAR header[] = {'U','A','-','C','P','U',':',' ','x','8','6',0}; @@ -3269,6 +3408,7 @@ static void test_http_connection(void) test_no_content(si.port); test_conn_close(si.port); test_no_cache(si.port); + test_cache_read_gzipped(si.port); test_premature_disconnect(si.port); /* send the basic request again to shutdown the server thread */

1 0

Piotr Caban : wininet: Get rid of rawHeaders field in request_t structure.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: 422fb313d36952f181b54b1c5c9b29b2b5bcaba0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=422fb313d36952f181b54b1c5… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 17:00:18 2013 +0200 wininet: Get rid of rawHeaders field in request_t structure. --- dlls/wininet/http.c | 71 ++++++---------------------------------------- dlls/wininet/internet.h | 1 - 2 files changed, 10 insertions(+), 62 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index aaf1c86..94b63c8 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -701,7 +701,7 @@ static WCHAR* build_response_header(http_request_t *request, BOOL use_cr) if(!req) return NULL; - if(request->rawHeaders) + if (request->status_code) { req[n++] = request->version; sprintfW(buf, status_fmt, request->status_code); @@ -1847,7 +1847,6 @@ static void HTTPREQ_Destroy(object_header_t *hdr) heap_free(request->path); heap_free(request->verb); - heap_free(request->rawHeaders); heap_free(request->version); heap_free(request->statusText); @@ -2179,7 +2178,7 @@ static DWORD HTTPREQ_QueryOption(object_header_t *hdr, DWORD option, void *buffe if(req->proxy) flags |= INTERNET_REQFLAG_VIA_PROXY; - if(!req->rawHeaders) + if(!req->status_code) flags |= INTERNET_REQFLAG_NO_HEADERS; TRACE("INTERNET_OPTION_REQUEST_FLAGS returning %x\n", flags); @@ -5742,10 +5741,7 @@ static DWORD HTTP_GetResponseHeaders(http_request_t *request, INT *len) INT rc = 0; char bufferA[MAX_REPLY_LEN]; LPWSTR status_code = NULL, status_text = NULL; - DWORD res = ERROR_HTTP_INVALID_SERVER_RESPONSE, cchMaxRawHeaders = 1024; - LPWSTR lpszRawHeaders = NULL; - LPWSTR temp; - DWORD cchRawHeaders = 0; + DWORD res = ERROR_HTTP_INVALID_SERVER_RESPONSE; BOOL codeHundred = FALSE; TRACE("-->\n"); @@ -5802,9 +5798,6 @@ static DWORD HTTP_GetResponseHeaders(http_request_t *request, INT *len) request->version = heap_strdupW(g_szHttp1_0); request->statusText = heap_strdupW(szOK); - heap_free(request->rawHeaders); - request->rawHeaders = heap_strdupW(szDefaultHeader); - goto lend; } } while (codeHundred); @@ -5823,25 +5816,10 @@ static DWORD HTTP_GetResponseHeaders(http_request_t *request, INT *len) *(status_code-1) = ' '; *(status_text-1) = ' '; - /* regenerate raw headers */ - lpszRawHeaders = heap_alloc((cchMaxRawHeaders + 1) * sizeof(WCHAR)); - if (!lpszRawHeaders) goto lend; - - while (cchRawHeaders + buflen + strlenW(szCrLf) > cchMaxRawHeaders) - cchMaxRawHeaders *= 2; - temp = heap_realloc(lpszRawHeaders, (cchMaxRawHeaders+1)*sizeof(WCHAR)); - if (temp == NULL) goto lend; - lpszRawHeaders = temp; - memcpy(lpszRawHeaders+cchRawHeaders, buffer, (buflen-1)*sizeof(WCHAR)); - cchRawHeaders += (buflen-1); - memcpy(lpszRawHeaders+cchRawHeaders, szCrLf, sizeof(szCrLf)); - cchRawHeaders += sizeof(szCrLf)/sizeof(szCrLf[0])-1; - lpszRawHeaders[cchRawHeaders] = '\0'; - /* Parse each response line */ do { - buflen = MAX_REPLY_LEN; + buflen = MAX_REPLY_LEN; if (!read_line(request, bufferA, &buflen) && buflen) { LPWSTR * pFieldAndValue; @@ -5854,52 +5832,23 @@ static DWORD HTTP_GetResponseHeaders(http_request_t *request, INT *len) pFieldAndValue = HTTP_InterpretHttpHeader(buffer); if (pFieldAndValue) { - while (cchRawHeaders + buflen + strlenW(szCrLf) > cchMaxRawHeaders) - cchMaxRawHeaders *= 2; - temp = heap_realloc(lpszRawHeaders, (cchMaxRawHeaders+1)*sizeof(WCHAR)); - if (temp == NULL) goto lend; - lpszRawHeaders = temp; - memcpy(lpszRawHeaders+cchRawHeaders, buffer, (buflen-1)*sizeof(WCHAR)); - cchRawHeaders += (buflen-1); - memcpy(lpszRawHeaders+cchRawHeaders, szCrLf, sizeof(szCrLf)); - cchRawHeaders += sizeof(szCrLf)/sizeof(szCrLf[0])-1; - lpszRawHeaders[cchRawHeaders] = '\0'; - HTTP_ProcessHeader(request, pFieldAndValue[0], pFieldAndValue[1], HTTP_ADDREQ_FLAG_ADD ); - HTTP_FreeTokens(pFieldAndValue); } } - else - { - cbreaks++; - if (cbreaks >= 2) - break; - } + else + { + cbreaks++; + if (cbreaks >= 2) + break; + } }while(1); - /* make sure the response header is terminated with an empty line. Some apps really - truly care about that empty line being there for some reason. Just add it to the - header. */ - if (cchRawHeaders + strlenW(szCrLf) > cchMaxRawHeaders) - { - cchMaxRawHeaders = cchRawHeaders + strlenW(szCrLf); - temp = heap_realloc(lpszRawHeaders, (cchMaxRawHeaders + 1) * sizeof(WCHAR)); - if (temp == NULL) goto lend; - lpszRawHeaders = temp; - } - - memcpy(&lpszRawHeaders[cchRawHeaders], szCrLf, sizeof(szCrLf)); - - heap_free(request->rawHeaders); - request->rawHeaders = lpszRawHeaders; - TRACE("raw headers: %s\n", debugstr_w(lpszRawHeaders)); res = ERROR_SUCCESS; lend: - if (res) heap_free(lpszRawHeaders); *len = rc; TRACE("<--\n"); return res; diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index b828f83..246eb95 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -343,7 +343,6 @@ typedef struct server_t *proxy; LPWSTR path; LPWSTR verb; - LPWSTR rawHeaders; netconn_t *netconn; DWORD security_flags; DWORD connect_timeout;

1 0

Piotr Caban : wininet: Add current raw headers to cache.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: 566b14479c2d5959a23dfb3d854bc569b336c9ae URL: http://source.winehq.org/git/wine.git/?a=commit;h=566b14479c2d5959a23dfb3d8… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 17:00:06 2013 +0200 wininet: Add current raw headers to cache. --- dlls/wininet/http.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 281fe66..aaf1c86 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -2276,12 +2276,15 @@ static void commit_cache_entry(http_request_t *req) req->hCacheFile = NULL; if(HTTP_GetRequestURL(req, url)) { - DWORD headersLen; + WCHAR *header; + DWORD header_len; - headersLen = req->rawHeaders ? strlenW(req->rawHeaders) : 0; + header = build_response_header(req, TRUE); + header_len = (header ? strlenW(header) : 0); CommitUrlCacheEntryW(url, req->cacheFile, req->expires, req->last_modified, NORMAL_CACHE_ENTRY, - req->rawHeaders, headersLen, NULL, 0); + header, header_len, NULL, 0); + heap_free(header); } }

1 0

Piotr Caban : wininet: Delete Content-Encoding header when uncompressing data.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: 7883976722e447dc65bcb92776228e9489e3e1be URL: http://source.winehq.org/git/wine.git/?a=commit;h=7883976722e447dc65bcb9277… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 16:59:49 2013 +0200 wininet: Delete Content-Encoding header when uncompressing data. --- dlls/wininet/http.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index f5ec392..281fe66 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -2822,8 +2822,10 @@ static DWORD set_content_length(http_request_t *request) static const WCHAR gzipW[] = {'g','z','i','p',0}; encoding_idx = HTTP_GetCustomHeaderIndex(request, szContent_Encoding, 0, FALSE); - if(encoding_idx != -1 && !strcmpiW(request->custHeaders[encoding_idx].lpszValue, gzipW)) + if(encoding_idx != -1 && !strcmpiW(request->custHeaders[encoding_idx].lpszValue, gzipW)) { + HTTP_DeleteCustomHeader(request, encoding_idx); return init_gzip_stream(request); + } } return ERROR_SUCCESS;

1 0

Piotr Caban : wininet: Take HTTP_QUERY_FLAG_REQUEST_HEADERS flag into account when creating HTTP_QUERY_RAW_HEADERS data information .
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: 259ffa2a1af83b03c8d4d0107d40666dcb7f2b18 URL: http://source.winehq.org/git/wine.git/?a=commit;h=259ffa2a1af83b03c8d4d0107… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 16:59:33 2013 +0200 wininet: Take HTTP_QUERY_FLAG_REQUEST_HEADERS flag into account when creating HTTP_QUERY_RAW_HEADERS data information. --- dlls/wininet/http.c | 61 ++++++++++++++++++++++---------------------------- 1 files changed, 27 insertions(+), 34 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 682ed36..f5ec392 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -631,20 +631,21 @@ static void HTTP_FixURL(http_request_t *request) } } -static LPWSTR HTTP_BuildHeaderRequestString( http_request_t *request, LPCWSTR verb, LPCWSTR path, LPCWSTR version ) +static WCHAR* build_request_header(http_request_t *request, const WCHAR *verb, + const WCHAR *path, const WCHAR *version, BOOL use_cr) { LPWSTR requestString; DWORD len, n; LPCWSTR *req; UINT i; - LPWSTR p; static const WCHAR szSpace[] = { ' ',0 }; static const WCHAR szColon[] = { ':',' ',0 }; - static const WCHAR sztwocrlf[] = {'\r','\n','\r','\n', 0}; + static const WCHAR szCr[] = { '\r',0 }; + static const WCHAR szLf[] = { '\n',0 }; /* allocate space for an array of all the string pointers to be added */ - len = (request->nCustHeaders)*4 + 10; + len = (request->nCustHeaders)*5 + 10; req = heap_alloc(len*sizeof(LPCWSTR)); /* add the verb, path and HTTP version string */ @@ -654,39 +655,34 @@ static LPWSTR HTTP_BuildHeaderRequestString( http_request_t *request, LPCWSTR ve req[n++] = path; req[n++] = szSpace; req[n++] = version; + if (use_cr) + req[n++] = szCr; + req[n++] = szLf; /* Append custom request headers */ for (i = 0; i < request->nCustHeaders; i++) { if (request->custHeaders[i].wFlags & HDR_ISREQUEST) { - req[n++] = szCrLf; req[n++] = request->custHeaders[i].lpszField; req[n++] = szColon; req[n++] = request->custHeaders[i].lpszValue; + if (use_cr) + req[n++] = szCr; + req[n++] = szLf; TRACE("Adding custom header %s (%s)\n", debugstr_w(request->custHeaders[i].lpszField), debugstr_w(request->custHeaders[i].lpszValue)); } } - - if( n >= len ) - ERR("oops. buffer overrun\n"); - + if (use_cr) + req[n++] = szCr; + req[n++] = szLf; req[n] = NULL; + requestString = HTTP_build_req( req, 4 ); heap_free( req ); - - /* - * Set (header) termination string for request - * Make sure there's exactly two new lines at the end of the request - */ - p = &requestString[strlenW(requestString)-1]; - while ( (*p == '\n') || (*p == '\r') ) - p--; - strcpyW( p+1, sztwocrlf ); - return requestString; } @@ -3421,13 +3417,13 @@ static DWORD HTTP_HttpQueryInfoW(http_request_t *request, DWORD dwInfoLevel, DWORD res = ERROR_INVALID_PARAMETER; if (request_only) - headers = HTTP_BuildHeaderRequestString(request, request->verb, request->path, request->version); + headers = build_request_header(request, request->verb, request->path, request->version, TRUE); else headers = build_response_header(request, TRUE); + if (!headers) + return ERROR_OUTOFMEMORY; - if (headers) - len = strlenW(headers) * sizeof(WCHAR); - + len = strlenW(headers) * sizeof(WCHAR); if (len + sizeof(WCHAR) > *lpdwBufferLength) { len += sizeof(WCHAR); @@ -3435,13 +3431,7 @@ static DWORD HTTP_HttpQueryInfoW(http_request_t *request, DWORD dwInfoLevel, } else if (lpBuffer) { - if (headers) - memcpy(lpBuffer, headers, len + sizeof(WCHAR)); - else - { - len = strlenW(szCrLf) * sizeof(WCHAR); - memcpy(lpBuffer, szCrLf, sizeof(szCrLf)); - } + memcpy(lpBuffer, headers, len + sizeof(WCHAR)); TRACE("returning data: %s\n", debugstr_wn(lpBuffer, len / sizeof(WCHAR))); res = ERROR_SUCCESS; } @@ -3455,7 +3445,10 @@ static DWORD HTTP_HttpQueryInfoW(http_request_t *request, DWORD dwInfoLevel, LPWSTR headers; DWORD len; - headers = build_response_header(request, FALSE); + if (request_only) + headers = build_request_header(request, request->verb, request->path, request->version, FALSE); + else + headers = build_response_header(request, FALSE); if (!headers) return ERROR_OUTOFMEMORY; @@ -4070,7 +4063,7 @@ static DWORD HTTP_SecureProxyConnect(http_request_t *request) TRACE("\n"); - requestString = HTTP_BuildHeaderRequestString( request, connectW, server->host_port, g_szHttp1_1 ); + requestString = build_request_header( request, connectW, server->host_port, g_szHttp1_1, TRUE ); len = WideCharToMultiByte( CP_ACP, 0, requestString, -1, NULL, 0, NULL, NULL ); @@ -4829,11 +4822,11 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, if (request->proxy) { WCHAR *url = build_proxy_path_url(request); - requestString = HTTP_BuildHeaderRequestString(request, request->verb, url, request->version); + requestString = build_request_header(request, request->verb, url, request->version, TRUE); heap_free(url); } else - requestString = HTTP_BuildHeaderRequestString(request, request->verb, request->path, request->version); + requestString = build_request_header(request, request->verb, request->path, request->version, TRUE); TRACE("Request header -> %s\n", debugstr_w(requestString) );

1 0

Piotr Caban : wininet: Create raw HTTP headers dynamically.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: e092233396bceb950c64394b386fb3acad505935 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e092233396bceb950c64394b3… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 16:58:00 2013 +0200 wininet: Create raw HTTP headers dynamically. --- dlls/wininet/http.c | 149 ++++++++++++++++++++++++++------------------------ 1 files changed, 77 insertions(+), 72 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 73aa3f7..682ed36 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -585,61 +585,9 @@ static DWORD init_gzip_stream(http_request_t *req) #endif /*********************************************************************** - * HTTP_Tokenize (internal) - * - * Tokenize a string, allocating memory for the tokens. - */ -static LPWSTR * HTTP_Tokenize(LPCWSTR string, LPCWSTR token_string) -{ - LPWSTR * token_array; - int tokens = 0; - int i; - LPCWSTR next_token; - - if (string) - { - /* empty string has no tokens */ - if (*string) - tokens++; - /* count tokens */ - for (i = 0; string[i]; i++) - { - if (!strncmpW(string+i, token_string, strlenW(token_string))) - { - DWORD j; - tokens++; - /* we want to skip over separators, but not the null terminator */ - for (j = 0; j < strlenW(token_string) - 1; j++) - if (!string[i+j]) - break; - i += j; - } - } - } - - /* add 1 for terminating NULL */ - token_array = heap_alloc((tokens+1) * sizeof(*token_array)); - token_array[tokens] = NULL; - if (!tokens) - return token_array; - for (i = 0; i < tokens; i++) - { - int len; - next_token = strstrW(string, token_string); - if (!next_token) next_token = string+strlenW(string); - len = next_token - string; - token_array[i] = heap_alloc((len+1)*sizeof(WCHAR)); - memcpy(token_array[i], string, len*sizeof(WCHAR)); - token_array[i][len] = '\0'; - string = next_token+strlenW(token_string); - } - return token_array; -} - -/*********************************************************************** * HTTP_FreeTokens (internal) * - * Frees memory returned from HTTP_Tokenize. + * Frees table of pointers. */ static void HTTP_FreeTokens(LPWSTR * token_array) { @@ -742,6 +690,59 @@ static LPWSTR HTTP_BuildHeaderRequestString( http_request_t *request, LPCWSTR ve return requestString; } +static WCHAR* build_response_header(http_request_t *request, BOOL use_cr) +{ + static const WCHAR colonW[] = { ':',' ',0 }; + static const WCHAR crW[] = { '\r',0 }; + static const WCHAR lfW[] = { '\n',0 }; + static const WCHAR status_fmt[] = { ' ','%','u',' ',0 }; + + const WCHAR **req; + WCHAR *ret, buf[14]; + DWORD i, n = 0; + + req = heap_alloc((request->nCustHeaders*5+8)*sizeof(WCHAR*)); + if(!req) + return NULL; + + if(request->rawHeaders) + { + req[n++] = request->version; + sprintfW(buf, status_fmt, request->status_code); + req[n++] = buf; + req[n++] = request->statusText; + if (use_cr) + req[n++] = crW; + req[n++] = lfW; + } + + for(i = 0; i < request->nCustHeaders; i++) + { + if(!(request->custHeaders[i].wFlags & HDR_ISREQUEST) + && strcmpW(request->custHeaders[i].lpszField, szStatus)) + { + req[n++] = request->custHeaders[i].lpszField; + req[n++] = colonW; + req[n++] = request->custHeaders[i].lpszValue; + if(use_cr) + req[n++] = crW; + req[n++] = lfW; + + TRACE("Adding custom header %s (%s)\n", + debugstr_w(request->custHeaders[i].lpszField), + debugstr_w(request->custHeaders[i].lpszValue)); + } + } + if(use_cr) + req[n++] = crW; + req[n++] = lfW; + req[n] = NULL; + + ret = HTTP_build_req(req, 0); + heap_free(req); + return ret; +} + static void HTTP_ProcessCookies( http_request_t *request ) { int HeaderIndex; @@ -3422,7 +3423,7 @@ static DWORD HTTP_HttpQueryInfoW(http_request_t *request, DWORD dwInfoLevel, if (request_only) headers = HTTP_BuildHeaderRequestString(request, request->verb, request->path, request->version); else - headers = request->rawHeaders; + headers = build_response_header(request, TRUE); if (headers) len = strlenW(headers) * sizeof(WCHAR); @@ -3446,38 +3447,42 @@ static DWORD HTTP_HttpQueryInfoW(http_request_t *request, DWORD dwInfoLevel, } *lpdwBufferLength = len; - if (request_only) heap_free(headers); + heap_free(headers); return res; } case HTTP_QUERY_RAW_HEADERS: { - LPWSTR * ppszRawHeaderLines = HTTP_Tokenize(request->rawHeaders, szCrLf); - DWORD i, size = 0; - LPWSTR pszString = lpBuffer; + LPWSTR headers; + DWORD len; - for (i = 0; ppszRawHeaderLines[i]; i++) - size += strlenW(ppszRawHeaderLines[i]) + 1; + headers = build_response_header(request, FALSE); + if (!headers) + return ERROR_OUTOFMEMORY; - if (size + 1 > *lpdwBufferLength/sizeof(WCHAR)) + len = strlenW(headers) * sizeof(WCHAR); + if (len > *lpdwBufferLength) { - HTTP_FreeTokens(ppszRawHeaderLines); - *lpdwBufferLength = (size + 1) * sizeof(WCHAR); + *lpdwBufferLength = len; + heap_free(headers); return ERROR_INSUFFICIENT_BUFFER; } - if (pszString) + + if (lpBuffer) { - for (i = 0; ppszRawHeaderLines[i]; i++) + DWORD i; + + TRACE("returning data: %s\n", debugstr_wn(headers, len / sizeof(WCHAR))); + + for (i=0; i<len; i++) { - DWORD len = strlenW(ppszRawHeaderLines[i]); - memcpy(pszString, ppszRawHeaderLines[i], (len+1)*sizeof(WCHAR)); - pszString += len+1; + if (headers[i] == '\n') + headers[i] = 0; } - *pszString = '\0'; - TRACE("returning data: %s\n", debugstr_wn(lpBuffer, size)); + memcpy(lpBuffer, headers, len + sizeof(WCHAR)); } - *lpdwBufferLength = size * sizeof(WCHAR); - HTTP_FreeTokens(ppszRawHeaderLines); + *lpdwBufferLength = len - sizeof(WCHAR); + heap_free(headers); return ERROR_SUCCESS; } case HTTP_QUERY_STATUS_TEXT:

1 0

Piotr Caban : msvcrt: Move _fstat32 and _fstat64i32 functions to msvcrt.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: f99742a5393e296581a07c6aef115d8248f83b17 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f99742a5393e296581a07c6ae… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 14:29:59 2013 +0200 msvcrt: Move _fstat32 and _fstat64i32 functions to msvcrt. --- dlls/msvcr100/msvcr100.c | 32 +----------------------- dlls/msvcr100/msvcr100.spec | 4 +- dlls/msvcr110/msvcr110.spec | 4 +- dlls/msvcr80/msvcr80.spec | 4 +- dlls/msvcr90/msvcr90.spec | 4 +- dlls/msvcrt/file.c | 54 +++++++++++++++++++++++++++++++++++++++++++ dlls/msvcrt/msvcrt.spec | 2 + 7 files changed, 66 insertions(+), 38 deletions(-) diff --git a/dlls/msvcr100/msvcr100.c b/dlls/msvcr100/msvcr100.c index 4449310..149f132 100644 --- a/dlls/msvcr100/msvcr100.c +++ b/dlls/msvcr100/msvcr100.c @@ -78,8 +78,8 @@ typedef void (__cdecl *free_func_t)(void*); extern char* __cdecl __unDName(char *,const char*,int,malloc_func_t,free_func_t,unsigned short int); /********************************************************************* - * * stat64_to_stat32 [internal] - * */ + * stat64_to_stat32 [internal] + */ static void stat64_to_stat32(const struct _stat64 *buf64, struct _stat32 *buf) { buf->st_dev = buf64->st_dev; @@ -210,20 +210,6 @@ void* CDECL _recalloc(void* mem, size_t num, size_t size) } /********************************************************************* - * _fstat32 (MSVCR100.@) - */ -int CDECL _fstat32(int fd, struct _stat32* buf) -{ - int ret; - struct _stat64 buf64; - - ret = _fstat64(fd, &buf64); - if (!ret) - stat64_to_stat32(&buf64, buf); - return ret; -} - -/********************************************************************* * _stat32 (MSVCR100.@) */ int CDECL _stat32(const char *path, struct _stat32* buf) @@ -310,20 +296,6 @@ static void stat64_to_stat64i32(const struct _stat64 *buf64, struct _stat64i32 * } /********************************************************************* - * _fstat64i32 (MSVCR100.@) - */ -int CDECL _fstat64i32(int fd, struct _stat64i32* buf) -{ - int ret; - struct _stat64 buf64; - - ret = _fstat64(fd, &buf64); - if (!ret) - stat64_to_stat64i32(&buf64, buf); - return ret; -} - -/********************************************************************* * _stat64i32 (MSVCR100.@) */ int CDECL _stat64i32(const char* path, struct _stat64i32 * buf) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index e60e0fb..370feb2 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -841,10 +841,10 @@ @ cdecl _fseeki64(ptr int64 long) msvcrt._fseeki64 @ stub _fseeki64_nolock @ cdecl _fsopen(str str long) msvcrt._fsopen -@ cdecl _fstat32(long ptr) +@ cdecl _fstat32(long ptr) msvcrt._fstat32 @ stub _fstat32i64 @ cdecl _fstat64(long ptr) msvcrt._fstat64 -@ cdecl _fstat64i32(long ptr) +@ cdecl _fstat64i32(long ptr) msvcrt._fstat64i32 @ stub _ftell_nolock @ cdecl -ret64 _ftelli64(ptr) msvcrt._ftelli64 @ stub _ftelli64_nolock diff --git a/dlls/msvcr110/msvcr110.spec b/dlls/msvcr110/msvcr110.spec index 9f36dad..12fc808 100644 --- a/dlls/msvcr110/msvcr110.spec +++ b/dlls/msvcr110/msvcr110.spec @@ -1193,10 +1193,10 @@ @ cdecl _fseeki64(ptr int64 long) msvcrt._fseeki64 @ stub _fseeki64_nolock @ cdecl _fsopen(str str long) msvcrt._fsopen -@ cdecl _fstat32(long ptr) msvcr100._fstat32 +@ cdecl _fstat32(long ptr) msvcrt._fstat32 @ stub _fstat32i64 @ cdecl _fstat64(long ptr) msvcrt._fstat64 -@ cdecl _fstat64i32(long ptr) msvcr100._fstat64i32 +@ cdecl _fstat64i32(long ptr) msvcrt._fstat64i32 @ stub _ftell_nolock @ cdecl -ret64 _ftelli64(ptr) msvcrt._ftelli64 @ stub _ftelli64_nolock diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index a8490e1..47bf34f 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -507,10 +507,10 @@ @ cdecl _fseeki64(ptr int64 long) msvcrt._fseeki64 @ stub _fseeki64_nolock @ cdecl _fsopen(str str long) msvcrt._fsopen -@ cdecl _fstat32(long ptr) msvcr100._fstat32 +@ cdecl _fstat32(long ptr) msvcrt._fstat32 @ stub _fstat32i64 @ cdecl _fstat64(long ptr) msvcrt._fstat64 -@ cdecl _fstat64i32(long ptr) msvcr100._fstat64i32 +@ cdecl _fstat64i32(long ptr) msvcrt._fstat64i32 @ stub _ftell_nolock @ cdecl -ret64 _ftelli64(ptr) msvcrt._ftelli64 @ stub _ftelli64_nolock diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index 9783c2f..d76ab73 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -489,10 +489,10 @@ @ cdecl _fseeki64(ptr int64 long) msvcrt._fseeki64 @ stub _fseeki64_nolock @ cdecl _fsopen(str str long) msvcrt._fsopen -@ cdecl _fstat32(long ptr) msvcr100._fstat32 +@ cdecl _fstat32(long ptr) msvcrt._fstat32 @ stub _fstat32i64 @ cdecl _fstat64(long ptr) msvcrt._fstat64 -@ cdecl _fstat64i32(long ptr) msvcr100._fstat64i32 +@ cdecl _fstat64i32(long ptr) msvcrt._fstat64i32 @ stub _ftell_nolock @ cdecl -ret64 _ftelli64(ptr) msvcrt._ftelli64 @ stub _ftelli64_nolock diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index c8863b3..c0c7124 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -177,6 +177,36 @@ static void msvcrt_stat64_to_stati64(const struct MSVCRT__stat64 *buf64, struct buf->st_ctime = buf64->st_ctime; } +static void msvcrt_stat64_to_stat32(const struct MSVCRT__stat64 *buf64, struct MSVCRT__stat32 *buf) +{ + buf->st_dev = buf64->st_dev; + buf->st_ino = buf64->st_ino; + buf->st_mode = buf64->st_mode; + buf->st_nlink = buf64->st_nlink; + buf->st_uid = buf64->st_uid; + buf->st_gid = buf64->st_gid; + buf->st_rdev = buf64->st_rdev; + buf->st_size = buf64->st_size; + buf->st_atime = buf64->st_atime; + buf->st_mtime = buf64->st_mtime; + buf->st_ctime = buf64->st_ctime; +} + +static void msvcrt_stat64_to_stat64i32(const struct MSVCRT__stat64 *buf64, struct MSVCRT__stat64i32 *buf) +{ + buf->st_dev = buf64->st_dev; + buf->st_ino = buf64->st_ino; + buf->st_mode = buf64->st_mode; + buf->st_nlink = buf64->st_nlink; + buf->st_uid = buf64->st_uid; + buf->st_gid = buf64->st_gid; + buf->st_rdev = buf64->st_rdev; + buf->st_size = buf64->st_size; + buf->st_atime = buf64->st_atime; + buf->st_mtime = buf64->st_mtime; + buf->st_ctime = buf64->st_ctime; +} + static void time_to_filetime( MSVCRT___time64_t time, FILETIME *ft ) { /* 1601 to 1970 is 369 years plus 89 leap days */ @@ -1558,6 +1588,30 @@ int CDECL MSVCRT__fstat(int fd, struct MSVCRT__stat* buf) return ret; } +/* _fstat32 - not exported in native msvcrt */ +int CDECL _fstat32(int fd, struct MSVCRT__stat32* buf) +{ + int ret; + struct MSVCRT__stat64 buf64; + + ret = MSVCRT__fstat64(fd, &buf64); + if (!ret) + msvcrt_stat64_to_stat32(&buf64, buf); + return ret; +} + +/* _fstat64i32 - not exported in native msvcrt */ +int CDECL _fstat64i32(int fd, struct MSVCRT__stat64i32* buf) +{ + int ret; + struct MSVCRT__stat64 buf64; + + ret = MSVCRT__fstat64(fd, &buf64); + if (!ret) + msvcrt_stat64_to_stat64i32(&buf64, buf); + return ret; +} + /********************************************************************* * _futime64 (MSVCRT.@) */ diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index 43d3c45..55ebba9 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -1532,3 +1532,5 @@ @ cdecl _set_printf_count_output(long) MSVCRT__set_printf_count_output @ cdecl _getptd() @ cdecl fread_s(ptr long long long ptr) +@ cdecl _fstat32(long ptr) +@ cdecl _fstat64i32(long ptr)

1 0

Piotr Caban : msvcrt: Added _mbstok_s_l implementation.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: ff7388ca3fe27d5910c161e7074ca89dbdeff54a URL: http://source.winehq.org/git/wine.git/?a=commit;h=ff7388ca3fe27d5910c161e70… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 14:29:11 2013 +0200 msvcrt: Added _mbstok_s_l implementation. --- dlls/msvcr100/msvcr100.spec | 6 +- dlls/msvcr110/msvcr110.spec | 6 +- dlls/msvcr80/msvcr80.spec | 6 +- dlls/msvcr90/msvcr90.spec | 6 +- dlls/msvcrt/mbcs.c | 94 ++++++++++++++++++++++++++++--------------- dlls/msvcrt/msvcrt.h | 1 + dlls/msvcrt/msvcrt.spec | 6 +- 7 files changed, 77 insertions(+), 48 deletions(-) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index 52835cb..e60e0fb 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -1183,9 +1183,9 @@ @ cdecl _mbsstr(str str) msvcrt._mbsstr @ stub _mbsstr_l @ cdecl _mbstok(str str) msvcrt._mbstok -@ stub _mbstok_l -@ stub _mbstok_s -@ stub _mbstok_s_l +@ cdecl _mbstok_l(str str ptr) msvcrt._mbstok_l +@ cdecl _mbstok_s(str str ptr) msvcrt._mbstok_s +@ cdecl _mbstok_s_l(str str ptr ptr) msvcrt._mbstok_s_l @ cdecl _mbstowcs_l(ptr str long ptr) msvcrt._mbstowcs_l @ cdecl _mbstowcs_s_l(ptr ptr long str long ptr) msvcrt._mbstowcs_s_l @ cdecl _mbstrlen(str) msvcrt._mbstrlen diff --git a/dlls/msvcr110/msvcr110.spec b/dlls/msvcr110/msvcr110.spec index 171bf37..9f36dad 100644 --- a/dlls/msvcr110/msvcr110.spec +++ b/dlls/msvcr110/msvcr110.spec @@ -1545,9 +1545,9 @@ @ cdecl _mbsstr(str str) msvcrt._mbsstr @ stub _mbsstr_l @ cdecl _mbstok(str str) msvcrt._mbstok -@ stub _mbstok_l -@ stub _mbstok_s -@ stub _mbstok_s_l +@ cdecl _mbstok_l(str str ptr) msvcrt._mbstok_l +@ cdecl _mbstok_s(str str ptr) msvcrt._mbstok_s +@ cdecl _mbstok_s_l(str str ptr ptr) msvcrt._mbstok_s_l @ cdecl _mbstowcs_l(ptr str long ptr) msvcrt._mbstowcs_l @ cdecl _mbstowcs_s_l(ptr ptr long str long ptr) msvcrt._mbstowcs_s_l @ cdecl _mbstrlen(str) msvcrt._mbstrlen diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index f45095e..a8490e1 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -854,9 +854,9 @@ @ cdecl _mbsstr(str str) msvcrt._mbsstr @ stub _mbsstr_l @ cdecl _mbstok(str str) msvcrt._mbstok -@ stub _mbstok_l -@ stub _mbstok_s -@ stub _mbstok_s_l +@ cdecl _mbstok_l(str str ptr) msvcrt._mbstok_l +@ cdecl _mbstok_s(str str ptr) msvcrt._mbstok_s +@ cdecl _mbstok_s_l(str str ptr ptr) msvcrt._mbstok_s_l @ cdecl _mbstowcs_l(ptr str long ptr) msvcrt._mbstowcs_l @ cdecl _mbstowcs_s_l(ptr ptr long str long ptr) msvcrt._mbstowcs_s_l @ cdecl _mbstrlen(str) msvcrt._mbstrlen diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index 140797d..9783c2f 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -832,9 +832,9 @@ @ cdecl _mbsstr(str str) msvcrt._mbsstr @ stub _mbsstr_l @ cdecl _mbstok(str str) msvcrt._mbstok -@ stub _mbstok_l -@ stub _mbstok_s -@ stub _mbstok_s_l +@ cdecl _mbstok_l(str str ptr) msvcrt._mbstok_l +@ cdecl _mbstok_s(str str ptr) msvcrt._mbstok_s +@ cdecl _mbstok_s_l(str str ptr ptr) msvcrt._mbstok_s_l @ cdecl _mbstowcs_l(ptr str long ptr) msvcrt._mbstowcs_l @ cdecl _mbstowcs_s_l(ptr ptr long str long ptr) msvcrt._mbstowcs_s_l @ cdecl _mbstrlen(str) msvcrt._mbstrlen diff --git a/dlls/msvcrt/mbcs.c b/dlls/msvcrt/mbcs.c index 3d50d6c..b70ffa0 100644 --- a/dlls/msvcrt/mbcs.c +++ b/dlls/msvcrt/mbcs.c @@ -145,11 +145,6 @@ static inline unsigned char *u_strrchr( const unsigned char *s, unsigned char x return (unsigned char*) strrchr( (const char*)s, x ); } -static inline unsigned char *u_strtok( unsigned char *s, const unsigned char *delim ) -{ - return (unsigned char*) strtok( (char*)s, (const char*)delim ); -} - static inline unsigned char *u__strset( unsigned char *s, unsigned char c ) { return (unsigned char*) _strset( (char*)s, c); @@ -1090,38 +1085,71 @@ unsigned char* CDECL _mbsrchr(const unsigned char* s, unsigned int x) } /********************************************************************* - * _mbstok(MSVCRT.@) - * - * Find and extract tokens from strings + * _mbstok_s_l(MSVCRT.@) */ -unsigned char* CDECL _mbstok(unsigned char *str, const unsigned char *delim) +unsigned char* CDECL _mbstok_s_l(unsigned char *str, const unsigned char *delim, + unsigned char **ctx, MSVCRT__locale_t locale) { - thread_data_t *data = msvcrt_get_thread_data(); - unsigned char *ret; + MSVCRT_pthreadmbcinfo mbcinfo; + unsigned int c; - if(get_mbcinfo()->ismbcodepage) - { - unsigned int c; + if(!MSVCRT_CHECK_PMT(delim != NULL)) return NULL; + if(!MSVCRT_CHECK_PMT(ctx != NULL)) return NULL; + if(!MSVCRT_CHECK_PMT(str || *ctx)) return NULL; + + if(locale) + mbcinfo = locale->mbcinfo; + else + mbcinfo = get_mbcinfo(); - if (!str) - if (!(str = data->mbstok_next)) return NULL; + if(!mbcinfo->ismbcodepage) + return (unsigned char*)MSVCRT_strtok_s((char*)str, (const char*)delim, (char**)ctx); - while ((c = _mbsnextc(str)) && _mbschr(delim, c)) { - str += c > 255 ? 2 : 1; - } - if (!*str) return NULL; - ret = str++; - while ((c = _mbsnextc(str)) && !_mbschr(delim, c)) { - str += c > 255 ? 2 : 1; - } - if (*str) { - *str++ = 0; - if (c > 255) *str++ = 0; - } - data->mbstok_next = str; - return ret; + if(!str) + str = *ctx; + + while((c=_mbsnextc(str)) && _mbschr(delim, c)) + str += c>255 ? 2 : 1; + if(!*str) + return NULL; + + *ctx = str + (c>255 ? 2 : 1); + while((c=_mbsnextc(*ctx)) && !_mbschr(delim, c)) + *ctx += c>255 ? 2 : 1; + if (**ctx) { + *(*ctx)++ = 0; + if(c > 255) + *(*ctx)++ = 0; } - return u_strtok(str, delim); /* ASCII CP */ + + return str; +} + + +/********************************************************************* + * _mbstok_s(MSVCRT.@) + */ +unsigned char* CDECL _mbstok_s(unsigned char *str, + const unsigned char *delim, unsigned char **ctx) +{ + return _mbstok_s_l(str, delim, ctx, NULL); +} + +/********************************************************************* + * _mbstok_l(MSVCRT.@) + */ +unsigned char* CDECL _mbstok_l(unsigned char *str, + const unsigned char *delim, MSVCRT__locale_t locale) +{ + return _mbstok_s_l(str, delim, &msvcrt_get_thread_data()->mbstok_next, locale); +} + +/********************************************************************* + * _mbstok(MSVCRT.@) + */ +unsigned char* CDECL _mbstok(unsigned char *str, const unsigned char *delim) +{ + return _mbstok_s_l(str, delim, &msvcrt_get_thread_data()->mbstok_next, NULL); } /********************************************************************* diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index b51ff81..35e9c7d 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -989,6 +989,7 @@ int __cdecl MSVCRT__tolower_l(int,MSVCRT__locale_t); int __cdecl MSVCRT__strnicoll_l(const char*, const char*, MSVCRT_size_t, MSVCRT__locale_t); int __cdecl MSVCRT__strncoll_l(const char*, const char*, MSVCRT_size_t, MSVCRT__locale_t); unsigned int __cdecl _get_output_format(void); +char* __cdecl MSVCRT_strtok_s(char*, const char*, char**); /* Maybe one day we'll enable the invalid parameter handlers with the full set of information (msvcrXXd) * #define MSVCRT_INVALID_PMT(x) MSVCRT_call_invalid_parameter_handler(x, __FUNCTION__, __FILE__, __LINE__, 0) diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index 0af2423..43d3c45 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -808,9 +808,9 @@ @ cdecl _mbsstr(str str) # stub _mbsstr_l(str str ptr) @ cdecl _mbstok(str str) -# stub _mbstok_l(str str ptr) -# stub _mbstok_s(str str ptr) -# stub _mbstok_s_l(str str ptr ptr) +@ cdecl _mbstok_l(str str ptr) +@ cdecl _mbstok_s(str str ptr) +@ cdecl _mbstok_s_l(str str ptr ptr) @ cdecl _mbstowcs_l(ptr str long ptr) MSVCRT__mbstowcs_l @ cdecl _mbstowcs_s_l(ptr ptr long str long ptr) MSVCRT__mbstowcs_s_l @ cdecl _mbstrlen(str)

1 0

Piotr Caban : msvcr100: Added _vscwprintf_p implementation.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: a5f525276f6459fe5ab269a5632e7141a2c70607 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a5f525276f6459fe5ab269a56… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Apr 30 14:28:34 2013 +0200 msvcr100: Added _vscwprintf_p implementation. --- dlls/msvcr100/msvcr100.c | 8 ++++++++ dlls/msvcr100/msvcr100.spec | 2 +- dlls/msvcr110/msvcr110.spec | 2 +- dlls/msvcr80/msvcr80.spec | 2 +- dlls/msvcr90/msvcr90.spec | 2 +- include/msvcrt/stdio.h | 1 + include/msvcrt/wchar.h | 1 + 7 files changed, 14 insertions(+), 4 deletions(-) diff --git a/dlls/msvcr100/msvcr100.c b/dlls/msvcr100/msvcr100.c index d43cbbf..4449310 100644 --- a/dlls/msvcr100/msvcr100.c +++ b/dlls/msvcr100/msvcr100.c @@ -423,6 +423,14 @@ int CDECL _vswprintf_p(wchar_t *buffer, size_t length, const wchar_t *format, __ } /********************************************************************* + * _vscwprintf_p (MSVCR100.@) + */ +int CDECL _vscwprintf_p(const wchar_t *format, __ms_va_list args) +{ + return _vscwprintf_p_l(format, NULL, args); +} + +/********************************************************************* * _byteswap_ushort (MSVCR100.@) */ unsigned short CDECL _byteswap_ushort(unsigned short s) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index 197a414..52835cb 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -1442,7 +1442,7 @@ @ stub _vscprintf_p_l @ cdecl _vscwprintf(wstr ptr) msvcrt._vscwprintf @ cdecl _vscwprintf_l(wstr ptr ptr) msvcrt._vscwprintf_l -@ stub _vscwprintf_p +@ cdecl _vscwprintf_p(wstr ptr) @ cdecl _vscwprintf_p_l(wstr ptr ptr) msvcrt._vscwprintf_p_l @ cdecl _vsnprintf(ptr long str ptr) msvcrt._vsnprintf @ cdecl _vsnprintf_c(ptr long str ptr) msvcrt._vsnprintf_c diff --git a/dlls/msvcr110/msvcr110.spec b/dlls/msvcr110/msvcr110.spec index d00f1a9..171bf37 100644 --- a/dlls/msvcr110/msvcr110.spec +++ b/dlls/msvcr110/msvcr110.spec @@ -1804,7 +1804,7 @@ @ stub _vscprintf_p_l @ cdecl _vscwprintf(wstr ptr) msvcrt._vscwprintf @ cdecl _vscwprintf_l(wstr ptr ptr) msvcrt._vscwprintf_l -@ stub _vscwprintf_p +@ cdecl _vscwprintf_p(wstr ptr) msvcr100._vscwprintf_p @ cdecl _vscwprintf_p_l(wstr ptr ptr) msvcrt._vscwprintf_p_l @ cdecl _vsnprintf(ptr long str ptr) msvcrt._vsnprintf @ cdecl _vsnprintf_c(ptr long str ptr) msvcrt._vsnprintf_c diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index 860383a..f45095e 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -1120,7 +1120,7 @@ @ stub _vscprintf_p_l @ cdecl _vscwprintf(wstr ptr) msvcrt._vscwprintf @ cdecl _vscwprintf_l(wstr ptr ptr) msvcrt._vscwprintf_l -@ stub _vscwprintf_p +@ cdecl _vscwprintf_p(wstr ptr) msvcr100._vscwprintf_p @ cdecl _vscwprintf_p_l(wstr ptr ptr) msvcrt._vscwprintf_p_l @ cdecl _vsnprintf(ptr long str ptr) msvcrt._vsnprintf @ cdecl _vsnprintf_c(ptr long str ptr) msvcrt._vsnprintf_c diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index ab9cea6..140797d 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -1096,7 +1096,7 @@ @ stub _vscprintf_p_l @ cdecl _vscwprintf(wstr ptr) msvcrt._vscwprintf @ cdecl _vscwprintf_l(wstr ptr ptr) msvcrt._vscwprintf_l -@ stub _vscwprintf_p +@ cdecl _vscwprintf_p(wstr ptr) msvcr100._vscwprintf_p @ cdecl _vscwprintf_p_l(wstr ptr ptr) msvcrt._vscwprintf_p_l @ cdecl _vsnprintf(ptr long str ptr) msvcrt._vsnprintf @ cdecl _vsnprintf_c(ptr long str ptr) msvcrt._vsnprintf_c diff --git a/include/msvcrt/stdio.h b/include/msvcrt/stdio.h index a01859e..737003d 100644 --- a/include/msvcrt/stdio.h +++ b/include/msvcrt/stdio.h @@ -194,6 +194,7 @@ int __cdecl _snwprintf(wchar_t*,size_t,const wchar_t*,...); int __cdecl _snwprintf_s(wchar_t*,size_t,size_t,const wchar_t*,...); int __cdecl _scwprintf(const wchar_t*,...); int __cdecl _vscwprintf(const wchar_t*,__ms_va_list); +int __cdecl _vscwprintf_p_l(const wchar_t*,_locale_t,__ms_va_list); int __cdecl _vsnwprintf(wchar_t*,size_t,const wchar_t*,__ms_va_list); int __cdecl _vsnwprintf_s(wchar_t*,size_t,size_t,const wchar_t*,__ms_va_list); int __cdecl _vswprintf_p_l(wchar_t*,size_t,const wchar_t*,_locale_t,__ms_va_list); diff --git a/include/msvcrt/wchar.h b/include/msvcrt/wchar.h index 27f673c..d936dc3 100644 --- a/include/msvcrt/wchar.h +++ b/include/msvcrt/wchar.h @@ -320,6 +320,7 @@ int __cdecl _putws(const wchar_t*); int __cdecl _snwprintf(wchar_t*,size_t,const wchar_t*,...); int __cdecl _snwprintf_s(wchar_t*,size_t,size_t,const wchar_t*,...); int __cdecl _vscwprintf(const wchar_t*,__ms_va_list); +int __cdecl _vscwprintf_p_l(const wchar_t*,_locale_t,__ms_va_list); int __cdecl _vsnwprintf(wchar_t*,size_t,const wchar_t*,__ms_va_list); int __cdecl _vsnwprintf_s(wchar_t*,size_t,size_t,const wchar_t*,__ms_va_list); int __cdecl _vswprintf_p_l(wchar_t*,size_t,const wchar_t*,_locale_t,__ms_va_list);

1 0

Alexandre Julliard : sane.ds: Don' t unload external libraries on process shutdown.
by Alexandre Julliard 30 Apr '13

30 Apr '13

Module: wine Branch: master Commit: dc345305b0d37ce2342d7528a4cceab318e5ba4f URL: http://source.winehq.org/git/wine.git/?a=commit;h=dc345305b0d37ce2342d7528a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Apr 30 17:25:29 2013 +0200 sane.ds: Don't unload external libraries on process shutdown. --- dlls/sane.ds/sane_main.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/sane.ds/sane_main.c b/dlls/sane.ds/sane_main.c index 1c73252..c01baeb 100644 --- a/dlls/sane.ds/sane_main.c +++ b/dlls/sane.ds/sane_main.c @@ -102,14 +102,12 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) break; } case DLL_PROCESS_DETACH: + if (lpvReserved) break; #ifdef SONAME_LIBSANE TRACE("calling sane_exit()\n"); psane_exit (); - close_libsane(libsane_handle); - libsane_handle = NULL; -#endif - SANE_instance = NULL; +#endif break; }

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits April 2013