wine-commits March 2013

wine-commits@winehq.org

2 participants
531 discussions

Piotr Caban : wininet: Improved urlcache debug traces.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: b3c96b61bb790e28f5b71d038dbce58525cccd45 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b3c96b61bb790e28f5b71d038… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Mar 6 12:48:15 2013 +0100 wininet: Improved urlcache debug traces. --- dlls/wininet/urlcache.c | 38 +++++++++++++++++++++++--------------- 1 files changed, 23 insertions(+), 15 deletions(-) diff --git a/dlls/wininet/urlcache.c b/dlls/wininet/urlcache.c index 578473d..0c9a4e2 100644 --- a/dlls/wininet/urlcache.c +++ b/dlls/wininet/urlcache.c @@ -1803,15 +1803,16 @@ BOOL WINAPI GetUrlCacheEntryInfoExA( if (pEntry->dwSignature != URL_SIGNATURE) { URLCacheContainer_UnlockIndex(pContainer, pHeader); - FIXME("Trying to retrieve entry of unknown format %s\n", debugstr_an((LPCSTR)&pEntry->dwSignature, sizeof(DWORD))); + FIXME("Trying to retrieve entry of unknown format %s\n", + debugstr_an((LPCSTR)&pEntry->dwSignature, sizeof(DWORD))); SetLastError(ERROR_FILE_NOT_FOUND); return FALSE; } pUrlEntry = (const URL_CACHEFILE_ENTRY *)pEntry; TRACE("Found URL: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetUrl)); - if (pUrlEntry->dwOffsetHeaderInfo) - TRACE("Header info: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo)); + TRACE("Header info: %s\n", debugstr_an((LPCSTR)pUrlEntry + + pUrlEntry->dwOffsetHeaderInfo, pUrlEntry->dwHeaderInfoSize)); if((dwFlags & GET_INSTALLED_ENTRY) && !(pUrlEntry->CacheEntryType & INSTALLED_CACHE_ENTRY)) { @@ -1838,7 +1839,8 @@ BOOL WINAPI GetUrlCacheEntryInfoExA( SetLastError(error); return FALSE; } - TRACE("Local File Name: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); + if(pUrlEntry->dwOffsetLocalName) + TRACE("Local File Name: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); } URLCacheContainer_UnlockIndex(pContainer, pHeader); @@ -1944,14 +1946,16 @@ BOOL WINAPI GetUrlCacheEntryInfoExW( if (pEntry->dwSignature != URL_SIGNATURE) { URLCacheContainer_UnlockIndex(pContainer, pHeader); - FIXME("Trying to retrieve entry of unknown format %s\n", debugstr_an((LPCSTR)&pEntry->dwSignature, sizeof(DWORD))); + FIXME("Trying to retrieve entry of unknown format %s\n", + debugstr_an((LPCSTR)&pEntry->dwSignature, sizeof(DWORD))); SetLastError(ERROR_FILE_NOT_FOUND); return FALSE; } pUrlEntry = (const URL_CACHEFILE_ENTRY *)pEntry; TRACE("Found URL: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetUrl)); - TRACE("Header info: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo)); + TRACE("Header info: %s\n", debugstr_an((LPCSTR)pUrlEntry + + pUrlEntry->dwOffsetHeaderInfo, pUrlEntry->dwHeaderInfoSize)); if (lpdwCacheEntryInfoBufSize) { @@ -1971,7 +1975,8 @@ BOOL WINAPI GetUrlCacheEntryInfoExW( SetLastError(error); return FALSE; } - TRACE("Local File Name: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); + if(pUrlEntry->dwOffsetLocalName) + TRACE("Local File Name: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); } URLCacheContainer_UnlockIndex(pContainer, pHeader); @@ -2169,8 +2174,9 @@ BOOL WINAPI RetrieveUrlCacheEntryFileA( return FALSE; } - TRACE("Found URL: %s\n", (LPSTR)pUrlEntry + pUrlEntry->dwOffsetUrl); - TRACE("Header info: %s\n", (LPBYTE)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo); + TRACE("Found URL: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetUrl)); + TRACE("Header info: %s\n", debugstr_an((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo, + pUrlEntry->dwHeaderInfoSize)); error = URLCache_CopyEntry(pContainer, pHeader, lpCacheEntryInfo, lpdwCacheEntryInfoBufferSize, pUrlEntry, @@ -2266,8 +2272,9 @@ BOOL WINAPI RetrieveUrlCacheEntryFileW( return FALSE; } - TRACE("Found URL: %s\n", (LPSTR)pUrlEntry + pUrlEntry->dwOffsetUrl); - TRACE("Header info: %s\n", (LPBYTE)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo); + TRACE("Found URL: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetUrl)); + TRACE("Header info: %s\n", debugstr_an((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo, + pUrlEntry->dwHeaderInfoSize)); error = URLCache_CopyEntry( pContainer, @@ -2593,7 +2600,7 @@ BOOL WINAPI FreeUrlCacheSpaceW(LPCWSTR cache_path, DWORD size, DWORD filter) continue; if(urlcache_rate_entry(url_entry, &cur_time) <= delete_factor) { - TRACE("deleting file: %s\n", (char*)url_entry+url_entry->dwOffsetLocalName); + TRACE("deleting file: %s\n", debugstr_a((char*)url_entry+url_entry->dwOffsetLocalName)); DeleteUrlCacheEntryInternal(container, header, hash_entry); if(header->CacheUsage.QuadPart+header->ExemptUsage.QuadPart <= desired_size) @@ -3978,7 +3985,8 @@ static BOOL FindNextUrlCacheEntryInternal( TRACE("Found URL: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetUrl)); TRACE("Header info: %s\n", - debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo)); + debugstr_an((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetHeaderInfo, + pUrlEntry->dwHeaderInfoSize)); error = URLCache_CopyEntry( pContainer, @@ -3993,8 +4001,8 @@ static BOOL FindNextUrlCacheEntryInternal( SetLastError(error); return FALSE; } - TRACE("Local File Name: %s\n", - debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); + if(pUrlEntry->dwOffsetLocalName) + TRACE("Local File Name: %s\n", debugstr_a((LPCSTR)pUrlEntry + pUrlEntry->dwOffsetLocalName)); /* increment the current index so that next time the function * is called the next entry is returned */

1 0

Jacek Caban : jscript: Reimplemented do_attribute_tag_format without direct access to object' s string buffer.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 6eaef96dad12fb4aaec5044421941f3cd7ddb44a URL: http://source.winehq.org/git/wine.git/?a=commit;h=6eaef96dad12fb4aaec504442… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 6 12:39:38 2013 +0100 jscript: Reimplemented do_attribute_tag_format without direct access to object's string buffer. --- dlls/jscript/string.c | 39 +++++++++++++++++++++++++++++++-------- 1 files changed, 31 insertions(+), 8 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index 6235b51..8f38621 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -186,14 +186,11 @@ static HRESULT do_attributeless_tag_format(script_ctx_t *ctx, vdisp_t *jsthis, j } static HRESULT do_attribute_tag_format(script_ctx_t *ctx, vdisp_t *jsthis, unsigned argc, jsval_t *argv, jsval_t *r, - const WCHAR *tagname, const WCHAR *attr) + const WCHAR *tagname, const WCHAR *attrname) { - static const WCHAR tagfmtW[] - = {'<','%','s',' ','%','s','=','\"','%','s','\"','>','%','s','<','/','%','s','>',0}; static const WCHAR undefinedW[] = {'u','n','d','e','f','i','n','e','d',0}; jsstr_t *str, *attr_value = NULL; - const WCHAR *attr_str; unsigned attr_len; HRESULT hres; @@ -207,19 +204,45 @@ static HRESULT do_attribute_tag_format(script_ctx_t *ctx, vdisp_t *jsthis, unsig jsstr_release(str); return hres; } - attr_str = attr_value->str; attr_len = jsstr_length(attr_value); }else { - attr_str = undefinedW; attr_len = sizeof(undefinedW)/sizeof(WCHAR)-1; } if(r) { + unsigned attrname_len = strlenW(attrname); + unsigned tagname_len = strlenW(tagname); jsstr_t *ret; - ret = jsstr_alloc_buf(2*strlenW(tagname) + strlenW(attr) + attr_len + jsstr_length(str) + 9); + ret = jsstr_alloc_buf(2*tagname_len + attrname_len + attr_len + jsstr_length(str) + 9); if(ret) { - sprintfW(ret->str, tagfmtW, tagname, attr, attr_str, str->str, tagname); + WCHAR *ptr = ret->str; + + *ptr++ = '<'; + memcpy(ptr, tagname, tagname_len*sizeof(WCHAR)); + ptr += tagname_len; + *ptr++ = ' '; + memcpy(ptr, attrname, attrname_len*sizeof(WCHAR)); + ptr += attrname_len; + *ptr++ = '='; + *ptr++ = '"'; + + if(attr_value) + jsstr_flush(attr_value, ptr); + else + memcpy(ptr, undefinedW, attr_len*sizeof(WCHAR)); + ptr += attr_len; + + *ptr++ = '"'; + *ptr++ = '>'; + ptr += jsstr_flush(str, ptr); + + *ptr++ = '<'; + *ptr++ = '/'; + memcpy(ptr, tagname, tagname_len*sizeof(WCHAR)); + ptr += tagname_len; + *ptr = '>'; + *r = jsval_string(ret); }else { hres = E_OUTOFMEMORY;

1 0

Jacek Caban : jscript: Reimplemented do_attributeless_tag_format without direct access to object' s string buffer.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 25dbf32b5d60937afafe6d2f7be7a910b6bcefc3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=25dbf32b5d60937afafe6d2f7… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 6 12:39:27 2013 +0100 jscript: Reimplemented do_attributeless_tag_format without direct access to object's string buffer. --- dlls/jscript/string.c | 39 ++++++++++++++++++++++++++++----------- 1 files changed, 28 insertions(+), 11 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index 1d4a188..6235b51 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -144,27 +144,44 @@ static HRESULT String_valueOf(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, un static HRESULT do_attributeless_tag_format(script_ctx_t *ctx, vdisp_t *jsthis, jsval_t *r, const WCHAR *tagname) { - jsstr_t *str; + unsigned tagname_len; + jsstr_t *str, *ret; + WCHAR *ptr; HRESULT hres; - static const WCHAR tagfmt[] = {'<','%','s','>','%','s','<','/','%','s','>',0}; - hres = get_string_val(ctx, jsthis, &str); if(FAILED(hres)) return hres; - if(r) { - jsstr_t *ret = jsstr_alloc_buf(jsstr_length(str) + 2*strlenW(tagname) + 5); - if(!ret) { - jsstr_release(str); - return E_OUTOFMEMORY; - } + if(!r) { + jsstr_release(str); + return S_OK; + } - sprintfW(ret->str, tagfmt, tagname, str->str, tagname); - *r = jsval_string(ret); + tagname_len = strlenW(tagname); + + ret = jsstr_alloc_buf(jsstr_length(str) + 2*tagname_len + 5); + if(!ret) { + jsstr_release(str); + return E_OUTOFMEMORY; } + ptr = ret->str; + *ptr++ = '<'; + memcpy(ptr, tagname, tagname_len*sizeof(WCHAR)); + ptr += tagname_len; + *ptr++ = '>'; + + ptr += jsstr_flush(str, ptr); jsstr_release(str); + + *ptr++ = '<'; + *ptr++ = '/'; + memcpy(ptr, tagname, tagname_len*sizeof(WCHAR)); + ptr += tagname_len; + *ptr = '>'; + + *r = jsval_string(ret); return S_OK; }

1 0

Alexandre Julliard : vbscript: Return after assert(0) to silence warnings.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 72226ea8df753b4432b3a5f4b0d8350b51162624 URL: http://source.winehq.org/git/wine.git/?a=commit;h=72226ea8df753b4432b3a5f4b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 14:09:47 2013 +0100 vbscript: Return after assert(0) to silence warnings. --- dlls/vbscript/compile.c | 1 + dlls/vbscript/global.c | 1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/vbscript/compile.c b/dlls/vbscript/compile.c index 58494b9..fb1974b 100644 --- a/dlls/vbscript/compile.c +++ b/dlls/vbscript/compile.c @@ -1400,6 +1400,7 @@ static HRESULT create_class_funcprop(compile_ctx_t *ctx, function_decl_t *func_d break; default: assert(0); + return E_FAIL; } assert(!desc->entries[invoke_type]); diff --git a/dlls/vbscript/global.c b/dlls/vbscript/global.c index 9ff1f76..9c9cd41 100644 --- a/dlls/vbscript/global.c +++ b/dlls/vbscript/global.c @@ -1012,6 +1012,7 @@ static HRESULT Global_InStr(vbdisp_t *This, VARIANT *args, unsigned args_cnt, VA return E_NOTIMPL; default: assert(0); + return E_FAIL; } if(startv) {

1 0

Alexandre Julliard : wininet: Always pass a ULONG variable to ioctlsocket.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 456773fdc067ab8673a330c7ae17aecc2815690a URL: http://source.winehq.org/git/wine.git/?a=commit;h=456773fdc067ab8673a330c7a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 13:32:49 2013 +0100 wininet: Always pass a ULONG variable to ioctlsocket. --- dlls/wininet/netconnection.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c index 0d74adf..3564282 100644 --- a/dlls/wininet/netconnection.c +++ b/dlls/wininet/netconnection.c @@ -607,7 +607,8 @@ static DWORD init_openssl(void) static DWORD create_netconn_socket(server_t *server, netconn_t *netconn, DWORD timeout) { - int result, flag; + int result; + ULONG flag; assert(server->addr_len); result = netconn->socket = socket(server->addr.ss_family, SOCK_STREAM, 0); @@ -633,7 +634,7 @@ static DWORD create_netconn_socket(server_t *server, netconn_t *netconn, DWORD t { int err; socklen_t len = sizeof(err); - if (!getsockopt(netconn->socket, SOL_SOCKET, SO_ERROR, &err, &len) && !err) + if (!getsockopt(netconn->socket, SOL_SOCKET, SO_ERROR, (void *)&err, &len) && !err) result = 0; } } @@ -1003,7 +1004,7 @@ BOOL NETCON_query_data_available(netconn_t *connection, DWORD *available) if(!connection->secure) { #ifdef FIONREAD - int unread; + ULONG unread; int retval = ioctlsocket(connection->socket, FIONREAD, &unread); if (!retval) {

1 0

Alexandre Julliard : winhttp: Always pass a ULONG variable to ioctlsocket.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 847a29375c42433dbd70701d59d250f1a390ce0b URL: http://source.winehq.org/git/wine.git/?a=commit;h=847a29375c42433dbd70701d5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 13:32:44 2013 +0100 winhttp: Always pass a ULONG variable to ioctlsocket. --- dlls/winhttp/net.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/winhttp/net.c b/dlls/winhttp/net.c index 5ce7683..2177416 100644 --- a/dlls/winhttp/net.c +++ b/dlls/winhttp/net.c @@ -331,7 +331,8 @@ BOOL netconn_close( netconn_t *conn ) BOOL netconn_connect( netconn_t *conn, const struct sockaddr *sockaddr, unsigned int addr_len, int timeout ) { BOOL ret = FALSE; - int res = 0, state; + int res = 0; + ULONG state; if (timeout > 0) { @@ -734,7 +735,8 @@ BOOL netconn_recv( netconn_t *conn, void *buf, size_t len, int flags, int *recvd BOOL netconn_query_data_available( netconn_t *conn, DWORD *available ) { #ifdef FIONREAD - int ret, unread; + int ret; + ULONG unread; #endif *available = 0; if (!netconn_connected( conn )) return FALSE;

1 0

Alexandre Julliard : winedbg: Use socklen_t where appropriate.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 51947acd5abb07209a8929ab639c1016a733e828 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51947acd5abb07209a8929ab6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 14:09:24 2013 +0100 winedbg: Use socklen_t where appropriate. --- programs/winedbg/gdbproxy.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index 580fb88..66ff6a8 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -476,6 +476,7 @@ static inline DWORD64 cpu_register(CONTEXT* ctx, unsigned idx) default: fprintf(stderr, "got unexpected size: %u\n", (unsigned)cpu_register_map[idx].ctx_length); assert(0); + return 0; } } @@ -2363,7 +2364,7 @@ static BOOL gdb_startup(struct gdb_context* gdbctx, DEBUG_EVENT* de, unsigned fl { int sock; struct sockaddr_in s_addrs; - unsigned int s_len = sizeof(s_addrs); + socklen_t s_len = sizeof(s_addrs); struct pollfd pollfd; IMAGEHLP_MODULE64 imh_mod; BOOL ret = FALSE;

1 0

Alexandre Julliard : ws2_32: Use socklen_t where appropriate.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: cfed5aa16c97871ad8454856877caabebfc6e174 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cfed5aa16c97871ad84548568… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 13:26:59 2013 +0100 ws2_32: Use socklen_t where appropriate. --- dlls/ws2_32/socket.c | 35 ++++++++++++++++++----------------- 1 files changed, 18 insertions(+), 17 deletions(-) diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c index 6fd1e34..05c7aac 100644 --- a/dlls/ws2_32/socket.c +++ b/dlls/ws2_32/socket.c @@ -985,9 +985,8 @@ static char *strdup_lower(const char *str) static inline int sock_error_p(int s) { - unsigned int optval, optlen; - - optlen = sizeof(optval); + unsigned int optval; + socklen_t optlen = sizeof(optval); getsockopt(s, SOL_SOCKET, SO_ERROR, (void *) &optval, &optlen); if (optval) WARN("\t[%i] error: %d\n", s, optval); return optval != 0; @@ -999,7 +998,7 @@ static inline int sock_error_p(int s) static inline int get_rcvsnd_timeo( int fd, int optname) { struct timeval tv; - unsigned int len = sizeof(tv); + socklen_t len = sizeof(tv); int ret = getsockopt(fd, SOL_SOCKET, optname, &tv, &len); if( ret >= 0) ret = tv.tv_sec * 1000 + tv.tv_usec / 1000; @@ -1777,7 +1776,7 @@ static int WS2_send( int fd, struct ws2_async *wsa ) { struct sockaddr_ipx* uipx = (struct sockaddr_ipx*)hdr.msg_name; int val=0; - unsigned int len=sizeof(int); + socklen_t len = sizeof(int); /* The packet type is stored at the ipx socket level; At least the linux kernel seems * to do something with it in case hdr.msg_name is NULL. Nonetheless can we use it to store @@ -2148,7 +2147,8 @@ static int WINAPI WS2_WSARecvMsg( SOCKET s, LPWSAMSG msg, LPDWORD lpNumberOfByte static BOOL interface_bind( SOCKET s, int fd, struct sockaddr *addr ) { struct sockaddr_in *in_sock = (struct sockaddr_in *) addr; - unsigned int sock_type = 0, optlen = sizeof(sock_type); + unsigned int sock_type = 0; + socklen_t optlen = sizeof(sock_type); in_addr_t bind_addr = in_sock->sin_addr.s_addr; PIP_ADAPTER_INFO adapters = NULL, adapter; BOOL ret = FALSE; @@ -2523,7 +2523,7 @@ int WINAPI WS_getpeername(SOCKET s, struct WS_sockaddr *name, int *namelen) if (fd != -1) { union generic_unix_sockaddr uaddr; - unsigned int uaddrlen = sizeof(uaddr); + socklen_t uaddrlen = sizeof(uaddr); if (getpeername(fd, &uaddr.addr, &uaddrlen) == 0) { @@ -2565,7 +2565,7 @@ int WINAPI WS_getsockname(SOCKET s, struct WS_sockaddr *name, int *namelen) if (fd != -1) { union generic_unix_sockaddr uaddr; - unsigned int uaddrlen = sizeof(uaddr); + socklen_t uaddrlen = sizeof(uaddr); if (getsockname(fd, &uaddr.addr, &uaddrlen) != 0) { @@ -2622,7 +2622,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, if ( (fd = get_sock_fd( s, 0, NULL )) == -1) return SOCKET_ERROR; convert_sockopt(&level, &optname); - if (getsockopt(fd, level, optname, optval, (unsigned int *)optlen) != 0 ) + if (getsockopt(fd, level, optname, optval, (socklen_t *)optlen) != 0 ) { SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); ret = SOCKET_ERROR; @@ -2633,7 +2633,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, case WS_SO_DONTLINGER: { struct linger lingval; - unsigned int len = sizeof(struct linger); + socklen_t len = sizeof(struct linger); if (!optlen || *optlen < sizeof(BOOL)|| !optval) { @@ -2695,7 +2695,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, { struct linger lingval; int so_type; - unsigned int len = sizeof(struct linger), slen = sizeof(int); + socklen_t len = sizeof(struct linger), slen = sizeof(int); /* struct linger and LINGER have different sizes */ if (!optlen || *optlen < sizeof(LINGER) || !optval) @@ -2759,7 +2759,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, #if defined(SO_RCVTIMEO) || defined(SO_SNDTIMEO) { struct timeval tv; - unsigned int len = sizeof(struct timeval); + socklen_t len = sizeof(struct timeval); if (!optlen || *optlen < sizeof(int)|| !optval) { @@ -2802,7 +2802,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, case IPX_PTYPE: if ((fd = get_sock_fd( s, 0, NULL )) == -1) return SOCKET_ERROR; #ifdef SOL_IPX - if(getsockopt(fd, SOL_IPX, IPX_TYPE, optval, (unsigned int*)optlen) == -1) + if(getsockopt(fd, SOL_IPX, IPX_TYPE, optval, (socklen_t *)optlen) == -1) { ret = SOCKET_ERROR; } @@ -2931,7 +2931,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, if ( (fd = get_sock_fd( s, 0, NULL )) == -1) return SOCKET_ERROR; convert_sockopt(&level, &optname); - if (getsockopt(fd, level, optname, optval, (unsigned int *)optlen) != 0 ) + if (getsockopt(fd, level, optname, optval, (socklen_t *)optlen) != 0 ) { SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); ret = SOCKET_ERROR; @@ -2965,7 +2965,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, if ( (fd = get_sock_fd( s, 0, NULL )) == -1) return SOCKET_ERROR; convert_sockopt(&level, &optname); - if (getsockopt(fd, level, optname, optval, (unsigned int *)optlen) != 0 ) + if (getsockopt(fd, level, optname, optval, (socklen_t *)optlen) != 0 ) { SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); ret = SOCKET_ERROR; @@ -3000,7 +3000,7 @@ INT WINAPI WS_getsockopt(SOCKET s, INT level, if ( (fd = get_sock_fd( s, 0, NULL )) == -1) return SOCKET_ERROR; convert_sockopt(&level, &optname); - if (getsockopt(fd, level, optname, optval, (unsigned int *)optlen) != 0 ) + if (getsockopt(fd, level, optname, optval, (socklen_t *)optlen) != 0 ) { SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); ret = SOCKET_ERROR; @@ -3245,7 +3245,8 @@ INT WINAPI WSAIoctl(SOCKET s, DWORD code, LPVOID in_buff, DWORD in_size, LPVOID case WS_SIOCATMARK: { - unsigned int oob = 0, oobsize = sizeof(int), atmark = 0; + unsigned int oob = 0, atmark = 0; + socklen_t oobsize = sizeof(int); if (out_size != sizeof(WS_u_long) || IS_INTRESOURCE(out_buff)) { WSASetLastError(WSAEFAULT);

1 0

Alexandre Julliard : server: Use socklen_t where appropriate.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 9fdf7bc7715b9b00846d8f0b451533906a131eaa URL: http://source.winehq.org/git/wine.git/?a=commit;h=9fdf7bc7715b9b00846d8f0b4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 13:40:38 2013 +0100 server: Use socklen_t where appropriate. --- server/request.c | 2 +- server/sock.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/server/request.c b/server/request.c index eeb3476..e075212 100644 --- a/server/request.c +++ b/server/request.c @@ -537,7 +537,7 @@ static void master_socket_poll_event( struct fd *fd, int event ) else if (event & POLLIN) { struct sockaddr_un dummy; - unsigned int len = sizeof(dummy); + socklen_t len = sizeof(dummy); int client = accept( get_unix_fd( master_socket->fd ), (struct sockaddr *) &dummy, &len ); if (client == -1) return; fcntl( client, F_SETFL, O_NONBLOCK ); diff --git a/server/sock.c b/server/sock.c index 7dcc101..ba18a66 100644 --- a/server/sock.c +++ b/server/sock.c @@ -285,9 +285,9 @@ static void sock_wake_up( struct sock *sock ) static inline int sock_error( struct fd *fd ) { - unsigned int optval = 0, optlen; + unsigned int optval = 0; + socklen_t optlen = sizeof(optval); - optlen = sizeof(optval); getsockopt( get_unix_fd(fd), SOL_SOCKET, SO_ERROR, (void *) &optval, &optlen); return optval; } @@ -667,7 +667,7 @@ static int accept_new_fd( struct sock *sock ) */ int acceptfd; struct sockaddr saddr; - unsigned int slen = sizeof(saddr); + socklen_t slen = sizeof(saddr); acceptfd = accept( get_unix_fd(sock->fd), &saddr, &slen); if (acceptfd == -1) {

1 0

Ken Thomases : winemac: Implement WGL_EXT_swap_control.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 3f807e7a1a59aa62c626142f7f98e40d11008056 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3f807e7a1a59aa62c626142f7… Author: Ken Thomases <ken(a)codeweavers.com> Date: Wed Mar 6 04:59:35 2013 -0600 winemac: Implement WGL_EXT_swap_control. --- dlls/winemac.drv/opengl.c | 65 +++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 65 insertions(+), 0 deletions(-) diff --git a/dlls/winemac.drv/opengl.c b/dlls/winemac.drv/opengl.c index c62db1a..0f62e28 100644 --- a/dlls/winemac.drv/opengl.c +++ b/dlls/winemac.drv/opengl.c @@ -2327,6 +2327,31 @@ static BOOL macdrv_wglGetPixelFormatAttribfvARB(HDC hdc, int iPixelFormat, int i } +/********************************************************************** + * macdrv_wglGetSwapIntervalEXT + * + * WGL_EXT_swap_control: wglGetSwapIntervalEXT + */ +static int macdrv_wglGetSwapIntervalEXT(void) +{ + struct wgl_context *context = NtCurrentTeb()->glContext; + long value; + CGLError err; + + TRACE("\n"); + + err = CGLGetParameter(context->cglcontext, kCGLCPSwapInterval, (GLint*)&value); + if (err != kCGLNoError) + { + WARN("CGLGetParameter(kCGLCPSwapInterval) failed; error %d %s\n", + err, CGLErrorString(err)); + value = 1; + } + + return value; +} + + /*********************************************************************** * macdrv_wglMakeContextCurrentARB * @@ -2686,6 +2711,42 @@ static BOOL macdrv_wglSetPixelFormatWINE(HDC hdc, int fmt) } +/********************************************************************** + * macdrv_wglSwapIntervalEXT + * + * WGL_EXT_swap_control: wglSwapIntervalEXT + */ +static BOOL macdrv_wglSwapIntervalEXT(int interval) +{ + struct wgl_context *context = NtCurrentTeb()->glContext; + long value; + CGLError err; + + TRACE("interval %d\n", interval); + + if (interval < 0) + { + SetLastError(ERROR_INVALID_DATA); + return FALSE; + } + + if (interval > 1) + interval = 1; + + value = interval; + err = CGLSetParameter(context->cglcontext, kCGLCPSwapInterval, (GLint*)&value); + if (err != kCGLNoError) + { + WARN("CGLSetParameter(kCGLCPSwapInterval) failed; error %d %s\n", + err, CGLErrorString(err)); + SetLastError(ERROR_GEN_FAILURE); + return FALSE; + } + + return TRUE; +} + + static void register_extension(const char *ext) { if (gl_info.wglExtensions[0]) @@ -2754,6 +2815,10 @@ static void load_extensions(void) register_extension("WGL_EXT_extensions_string"); opengl_funcs.ext.p_wglGetExtensionsStringEXT = macdrv_wglGetExtensionsStringEXT; + register_extension("WGL_EXT_swap_control"); + opengl_funcs.ext.p_wglSwapIntervalEXT = macdrv_wglSwapIntervalEXT; + opengl_funcs.ext.p_wglGetSwapIntervalEXT = macdrv_wglGetSwapIntervalEXT; + /* Presumably identical to [W]GL_ARB_framebuffer_sRGB, above, but clients may check for either, so register them separately. */ if (gluCheckExtension((GLubyte*)"GL_EXT_framebuffer_sRGB", (GLubyte*)gl_info.glExtensions))

1 0

← Newer
1
...
39
40
41
42
43
44
45
...
54
Older →

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 March 2013