wine-commits October 2012

wine-commits@winehq.org

2 participants
960 discussions

Daniel Lehman : msvcp: Sync fpos usage and istream<>::tellg.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: a49da3ca5d8bef49dc4ff7694690ca801e7811aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=a49da3ca5d8bef49dc4ff7694… Author: Daniel Lehman <dlehman(a)esri.com> Date: Tue Oct 30 13:14:57 2012 -0700 msvcp: Sync fpos usage and istream<>::tellg. --- dlls/msvcp100/ios.c | 111 ++++++++++++++++++++++------------------------ dlls/msvcp60/ios.c | 119 ++++++++++++++++++++++++------------------------- dlls/msvcp71/ios.c | 123 +++++++++++++++++++++++++-------------------------- 3 files changed, 171 insertions(+), 182 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a49da3ca5d8bef49dc4ff…

1 0

Daniel Lehman : msvcp90: In istream<>::tellg don't use sentry.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 01c451703c3de5e029a20cc19b6fc77e382ee700 URL: http://source.winehq.org/git/wine.git/?a=commit;h=01c451703c3de5e029a20cc19… Author: Daniel Lehman <dlehman(a)esri.com> Date: Mon Oct 1 10:54:36 2012 -0700 msvcp90: In istream<>::tellg don't use sentry. --- dlls/msvcp90/ios.c | 55 +++++++++++------------- dlls/msvcp90/tests/ios.c | 104 ++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 129 insertions(+), 30 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=01c451703c3de5e029a20…

1 0

Daniel Lehman : msvcp90: Fix usage of fpos<>::off vs fpos<>::pos.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 771cc7fdbd43550980c9250b9421cfe4621f3bbf URL: http://source.winehq.org/git/wine.git/?a=commit;h=771cc7fdbd43550980c9250b9… Author: Daniel Lehman <dlehman(a)esri.com> Date: Fri Sep 28 17:17:53 2012 -0700 msvcp90: Fix usage of fpos<>::off vs fpos<>::pos. --- dlls/msvcp90/ios.c | 74 ++++++++++++++++++++++++++-------------------------- 1 files changed, 37 insertions(+), 37 deletions(-) diff --git a/dlls/msvcp90/ios.c b/dlls/msvcp90/ios.c index 9435cb2..b1acb3f 100644 --- a/dlls/msvcp90/ios.c +++ b/dlls/msvcp90/ios.c @@ -1250,8 +1250,8 @@ fpos_int* __thiscall basic_streambuf_char_seekoff(basic_streambuf_char *this, fpos_int *ret, streamoff off, int way, int mode) { TRACE("(%p %ld %d %d)\n", this, off, way, mode); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -1285,8 +1285,8 @@ fpos_int* __thiscall basic_streambuf_char_seekpos(basic_streambuf_char *this, fpos_int *ret, fpos_int pos, int mode) { TRACE("(%p %s %d)\n", this, debugstr_fpos_int(&pos), mode); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -2020,8 +2020,8 @@ fpos_int* __thiscall basic_streambuf_wchar_seekoff(basic_streambuf_wchar *this, fpos_int *ret, streamoff off, int way, int mode) { TRACE("(%p %ld %d %d)\n", this, off, way, mode); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -2061,8 +2061,8 @@ fpos_int* __thiscall basic_streambuf_wchar_seekpos(basic_streambuf_wchar *this, fpos_int *ret, fpos_int pos, int mode) { TRACE("(%p %s %d)\n", this, debugstr_fpos_int(&pos), mode); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -2804,8 +2804,8 @@ fpos_int* __thiscall basic_filebuf_char_seekoff(basic_filebuf_char *this, if(!basic_filebuf_char_is_open(this) || !basic_filebuf_char__Endwrite(this) || fseek(this->file, off, way)) { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -2830,8 +2830,8 @@ fpos_int* __thiscall basic_filebuf_char_seekpos(basic_filebuf_char *this, if(!basic_filebuf_char_is_open(this) || !basic_filebuf_char__Endwrite(this) || fseek(this->file, (LONG)pos.pos, SEEK_SET) || (pos.off && fseek(this->file, pos.off, SEEK_CUR))) { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -3421,8 +3421,8 @@ fpos_int* __thiscall basic_filebuf_wchar_seekoff(basic_filebuf_wchar *this, if(!basic_filebuf_wchar_is_open(this) || !basic_filebuf_wchar__Endwrite(this) || fseek(this->file, off, way)) { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -3449,8 +3449,8 @@ fpos_int* __thiscall basic_filebuf_wchar_seekpos(basic_filebuf_wchar *this, if(!basic_filebuf_wchar_is_open(this) || !basic_filebuf_wchar__Endwrite(this) || fseek(this->file, (LONG)pos.pos, SEEK_SET) || (pos.off && fseek(this->file, pos.off, SEEK_CUR))) { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -3854,7 +3854,7 @@ fpos_int* __thiscall basic_stringbuf_char_seekpos(basic_stringbuf_char *this, { TRACE("(%p %p %s %d)\n", this, ret, debugstr_fpos_int(&pos), mode); - if(pos.off==0 && pos.pos==-1 && pos.state==0) { + if(pos.off==-1 && pos.pos==0 && pos.state==0) { *ret = pos; return ret; } @@ -4267,7 +4267,7 @@ fpos_int* __thiscall basic_stringbuf_wchar_seekpos(basic_stringbuf_wchar *this, { TRACE("(%p %p %s %d)\n", this, ret, debugstr_fpos_int(&pos), mode); - if(pos.off==0 && pos.pos==-1 && pos.state==0) { + if(pos.off==-1 && pos.pos==0 && pos.state==0) { *ret = pos; return ret; } @@ -5610,7 +5610,7 @@ basic_ostream_char* __thiscall basic_ostream_char_seekp(basic_ostream_char *this basic_streambuf_char_pubseekoff(basic_ios_char_rdbuf_get(base), &seek, off, way, OPENMODE_out); - if(seek.off==0 && seek.pos==-1 && seek.state==0) + if(seek.off==-1 && seek.pos==0 && seek.state==0) basic_ios_char_setstate(base, IOSTATE_failbit); } return this; @@ -5630,7 +5630,7 @@ basic_ostream_char* __thiscall basic_ostream_char_seekp_fpos(basic_ostream_char basic_streambuf_char_pubseekpos(basic_ios_char_rdbuf_get(base), &seek, pos, OPENMODE_out); - if(seek.off==0 && seek.pos==-1 && seek.state==0) + if(seek.off==-1 && seek.pos==0 && seek.state==0) basic_ios_char_setstate(base, IOSTATE_failbit); } return this; @@ -5649,8 +5649,8 @@ fpos_int* __thiscall basic_ostream_char_tellp(basic_ostream_char *this, fpos_int basic_streambuf_char_pubseekoff(basic_ios_char_rdbuf_get(base), ret, 0, SEEKDIR_cur, OPENMODE_out); }else { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; } return ret; @@ -6419,7 +6419,7 @@ basic_ostream_wchar* __thiscall basic_ostream_wchar_seekp(basic_ostream_wchar *t basic_streambuf_wchar_pubseekoff(basic_ios_wchar_rdbuf_get(base), &seek, off, way, OPENMODE_out); - if(seek.off==0 && seek.pos==-1 && seek.state==0) + if(seek.off==-1 && seek.pos==0 && seek.state==0) basic_ios_wchar_setstate(base, IOSTATE_failbit); } return this; @@ -6441,7 +6441,7 @@ basic_ostream_wchar* __thiscall basic_ostream_wchar_seekp_fpos(basic_ostream_wch basic_streambuf_wchar_pubseekpos(basic_ios_wchar_rdbuf_get(base), &seek, pos, OPENMODE_out); - if(seek.off==0 && seek.pos==-1 && seek.state==0) + if(seek.off==-1 && seek.pos==0 && seek.state==0) basic_ios_wchar_setstate(base, IOSTATE_failbit); } return this; @@ -6462,8 +6462,8 @@ fpos_int* __thiscall basic_ostream_wchar_tellp(basic_ostream_wchar *this, fpos_i basic_streambuf_wchar_pubseekoff(basic_ios_wchar_rdbuf_get(base), ret, 0, SEEKDIR_cur, OPENMODE_out); }else { - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; } return ret; @@ -7746,15 +7746,15 @@ fpos_int* __thiscall basic_istream_char_tellg(basic_istream_char *this, fpos_int ret, 0, SEEKDIR_cur, OPENMODE_in); basic_istream_char_sentry_destroy(this); - if(ret->off==0 && ret->pos==-1 && ret->state==0) + if(ret->off==-1 && ret->pos==0 && ret->state==0) basic_ios_char_setstate(base, IOSTATE_failbit); return ret; } } basic_istream_char_sentry_destroy(this); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -7774,7 +7774,7 @@ basic_istream_char* __thiscall basic_istream_char_seekg(basic_istream_char *this basic_streambuf_char_pubseekoff(strbuf, &ret, off, dir, OPENMODE_in); - if(ret.off==0 && ret.pos==-1 && ret.state==0) + if(ret.off==-1 && ret.pos==0 && ret.state==0) basic_ios_char_setstate(base, IOSTATE_failbit); else basic_ios_char_clear(base, IOSTATE_goodbit); @@ -7801,7 +7801,7 @@ basic_istream_char* __thiscall basic_istream_char_seekg_fpos(basic_istream_char basic_streambuf_char_pubseekpos(strbuf, &ret, pos, OPENMODE_in); basic_istream_char_sentry_destroy(this); - if(ret.off==0 && ret.pos==-1 && ret.state==0) + if(ret.off==-1 && ret.pos==0 && ret.state==0) basic_ios_char_setstate(base, IOSTATE_failbit); else basic_ios_char_clear(base, IOSTATE_goodbit); @@ -9028,15 +9028,15 @@ fpos_int* __thiscall basic_istream_wchar_tellg(basic_istream_wchar *this, fpos_i ret, 0, SEEKDIR_cur, OPENMODE_in); basic_istream_wchar_sentry_destroy(this); - if(ret->off==0 && ret->pos==-1 && ret->state==0) + if(ret->off==-1 && ret->pos==0 && ret->state==0) basic_ios_wchar_setstate(base, IOSTATE_failbit); return ret; } } basic_istream_wchar_sentry_destroy(this); - ret->off = 0; - ret->pos = -1; + ret->off = -1; + ret->pos = 0; ret->state = 0; return ret; } @@ -9059,7 +9059,7 @@ basic_istream_wchar* __thiscall basic_istream_wchar_seekg(basic_istream_wchar *t basic_streambuf_wchar_pubseekoff(strbuf, &ret, off, dir, OPENMODE_in); basic_istream_wchar_sentry_destroy(this); - if(ret.off==0 && ret.pos==-1 && ret.state==0) + if(ret.off==-1 && ret.pos==0 && ret.state==0) basic_ios_wchar_setstate(base, IOSTATE_failbit); else basic_ios_wchar_clear(base, IOSTATE_goodbit); @@ -9088,7 +9088,7 @@ basic_istream_wchar* __thiscall basic_istream_wchar_seekg_fpos(basic_istream_wch basic_streambuf_wchar_pubseekpos(strbuf, &ret, pos, OPENMODE_in); basic_istream_wchar_sentry_destroy(this); - if(ret.off==0 && ret.pos==-1 && ret.state==0) + if(ret.off==-1 && ret.pos==0 && ret.state==0) basic_ios_wchar_setstate(base, IOSTATE_failbit); else basic_ios_wchar_clear(base, IOSTATE_goodbit); @@ -12829,7 +12829,7 @@ fpos_int* __thiscall strstreambuf_seekpos(strstreambuf *this, fpos_int *ret, fpo { TRACE("(%p %p %s %d)\n", this, ret, debugstr_fpos_int(&pos), mode); - if(pos.off==0 && pos.pos==-1 && pos.state==0) { + if(pos.off==-1 && pos.pos==0 && pos.state==0) { *ret = pos; return ret; }

1 0

Daniel Lehman : msvcp90: Update fpos<>::off instead of pos in stringbuf<>:: seekoff.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 64527285cf7fab1de5dbd2952df4eee5ecd3b02b URL: http://source.winehq.org/git/wine.git/?a=commit;h=64527285cf7fab1de5dbd2952… Author: Daniel Lehman <dlehman(a)esri.com> Date: Fri Sep 28 15:16:17 2012 -0700 msvcp90: Update fpos<>::off instead of pos in stringbuf<>::seekoff. --- dlls/msvcp90/ios.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/msvcp90/ios.c b/dlls/msvcp90/ios.c index a11f90c..9435cb2 100644 --- a/dlls/msvcp90/ios.c +++ b/dlls/msvcp90/ios.c @@ -3803,6 +3803,7 @@ fpos_int* __thiscall basic_stringbuf_char_seekoff(basic_stringbuf_char *this, this->seekhigh = cur_w; ret->off = 0; + ret->pos = 0; ret->state = 0; beg = basic_streambuf_char_eback(&this->base); @@ -3841,7 +3842,7 @@ fpos_int* __thiscall basic_stringbuf_char_seekoff(basic_stringbuf_char *this, off = -1; } - ret->pos = off; + ret->off = off; return ret; } @@ -4213,6 +4214,7 @@ fpos_int* __thiscall basic_stringbuf_wchar_seekoff(basic_stringbuf_wchar *this, this->seekhigh = cur_w; ret->off = 0; + ret->pos = 0; ret->state = 0; beg = basic_streambuf_wchar_eback(&this->base); @@ -4251,7 +4253,7 @@ fpos_int* __thiscall basic_stringbuf_wchar_seekoff(basic_stringbuf_wchar *this, off = -1; } - ret->pos = off; + ret->off = off; return ret; }

1 0

Jacek Caban : jscript: Fixed some leaks (coverity).
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 12adc677820f59a7d079eec26cf3a998a222f4a0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=12adc677820f59a7d079eec26… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 31 10:21:09 2012 +0100 jscript: Fixed some leaks (coverity). --- dlls/jscript/dispex.c | 4 +++- dlls/jscript/function.c | 4 +++- dlls/jscript/number.c | 4 +++- dlls/jscript/vbarray.c | 2 ++ 4 files changed, 11 insertions(+), 3 deletions(-) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index 211ed8a..2be474e 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -901,8 +901,10 @@ HRESULT create_dispex(script_ctx_t *ctx, const builtin_info_t *builtin_info, jsd return E_OUTOFMEMORY; hres = init_dispex(ret, ctx, builtin_info ? builtin_info : &dispex_info, prototype); - if(FAILED(hres)) + if(FAILED(hres)) { + heap_free(ret); return hres; + } *dispex = ret; return S_OK; diff --git a/dlls/jscript/function.c b/dlls/jscript/function.c index 4cac3d6..c5bcd2e 100644 --- a/dlls/jscript/function.c +++ b/dlls/jscript/function.c @@ -578,8 +578,10 @@ static HRESULT create_function(script_ctx_t *ctx, const builtin_info_t *builtin_ hres = init_dispex_from_constr(&function->dispex, ctx, builtin_info, ctx->function_constr); else hres = init_dispex_from_constr(&function->dispex, ctx, &FunctionInst_info, ctx->function_constr); - if(FAILED(hres)) + if(FAILED(hres)) { + heap_free(function); return hres; + } function->flags = flags; function->length = flags & PROPF_ARGMASK; diff --git a/dlls/jscript/number.c b/dlls/jscript/number.c index b7a1a76..ed773a3 100644 --- a/dlls/jscript/number.c +++ b/dlls/jscript/number.c @@ -609,8 +609,10 @@ static HRESULT alloc_number(script_ctx_t *ctx, jsdisp_t *object_prototype, Numbe hres = init_dispex(&number->dispex, ctx, &Number_info, object_prototype); else hres = init_dispex_from_constr(&number->dispex, ctx, &NumberInst_info, ctx->number_constr); - if(FAILED(hres)) + if(FAILED(hres)) { + heap_free(number); return hres; + } *ret = number; return S_OK; diff --git a/dlls/jscript/vbarray.c b/dlls/jscript/vbarray.c index 5878866..7b9993b 100644 --- a/dlls/jscript/vbarray.c +++ b/dlls/jscript/vbarray.c @@ -184,6 +184,8 @@ static HRESULT VBArray_toArray(script_ctx_t *ctx, vdisp_t *vthis, WORD flags, un if(r) *r = jsval_obj(array); + else + jsdisp_release(array); return S_OK; }

1 0

Frédéric Delanoy : rpcrt4: Fix resources leaks.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 9c63a30c2591a3c75b8d431bc94b208e3c15153b URL: http://source.winehq.org/git/wine.git/?a=commit;h=9c63a30c2591a3c75b8d431bc… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Oct 31 09:43:46 2012 +0100 rpcrt4: Fix resources leaks. --- dlls/rpcrt4/rpc_transport.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/rpc_transport.c b/dlls/rpcrt4/rpc_transport.c index 686ce68..aebb05e 100644 --- a/dlls/rpcrt4/rpc_transport.c +++ b/dlls/rpcrt4/rpc_transport.c @@ -1345,6 +1345,7 @@ static RPC_STATUS rpcrt4_protseq_ncacn_ip_tcp_open_endpoint(RpcServerProtseq *pr if (getsockname(sock, (struct sockaddr *)&sa, &sa_len)) { WARN("getsockname() failed: %s\n", strerror(errno)); + closesocket(sock); status = RPC_S_CANT_CREATE_ENDPOINT; continue; } @@ -1355,6 +1356,7 @@ static RPC_STATUS rpcrt4_protseq_ncacn_ip_tcp_open_endpoint(RpcServerProtseq *pr if (ret) { WARN("getnameinfo failed: %s\n", gai_strerror(ret)); + closesocket(sock); status = RPC_S_CANT_CREATE_ENDPOINT; continue; } @@ -2139,6 +2141,8 @@ static RPC_STATUS rpcrt4_http_internet_connect(RpcConnection_http *httpc) { HeapFree(GetProcessHeap(), 0, password); HeapFree(GetProcessHeap(), 0, user); + HeapFree(GetProcessHeap(), 0, proxy); + HeapFree(GetProcessHeap(), 0, servername); ERR("InternetOpenW failed with error %d\n", GetLastError()); return RPC_S_SERVER_UNAVAILABLE; } @@ -2153,6 +2157,7 @@ static RPC_STATUS rpcrt4_http_internet_connect(RpcConnection_http *httpc) { HeapFree(GetProcessHeap(), 0, password); HeapFree(GetProcessHeap(), 0, user); + HeapFree(GetProcessHeap(), 0, proxy); return RPC_S_OUT_OF_RESOURCES; } MultiByteToWideChar(CP_ACP, 0, httpc->common.NetworkAddr, -1, servername, strlen(httpc->common.NetworkAddr) + 1); @@ -2163,6 +2168,7 @@ static RPC_STATUS rpcrt4_http_internet_connect(RpcConnection_http *httpc) HeapFree(GetProcessHeap(), 0, password); HeapFree(GetProcessHeap(), 0, user); + HeapFree(GetProcessHeap(), 0, proxy); HeapFree(GetProcessHeap(), 0, servername); if (!httpc->session)

1 0

Michael Stefaniuc : ole32/tests: Fix a memleak on an error path (Smatch).
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 470ad72b05a3159ce3d21c4a2ac7e54dd59052f4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=470ad72b05a3159ce3d21c4a2… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Oct 30 23:15:44 2012 +0100 ole32/tests: Fix a memleak on an error path (Smatch). --- dlls/ole32/tests/propvariant.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/ole32/tests/propvariant.c b/dlls/ole32/tests/propvariant.c index 769d49a..4d9a81b 100644 --- a/dlls/ole32/tests/propvariant.c +++ b/dlls/ole32/tests/propvariant.c @@ -445,6 +445,7 @@ static void test_varianttoproperty(void) if (len == 0xdeadbeef) { + HeapFree(GetProcessHeap(), 0, own_propvalue); return; }

1 0

Michael Stefaniuc : user32/test: Don't over-allocate memory for DDEDATA.
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 2e31a44aca00bceb5e9a6495881adca9a8fc5a35 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2e31a44aca00bceb5e9a64958… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Oct 30 23:14:52 2012 +0100 user32/test: Don't over-allocate memory for DDEDATA. --- dlls/user32/tests/dde.c | 14 +++++--------- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index 3ad48f0..e5d5554 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -143,7 +143,7 @@ static LRESULT WINAPI dde_server_wndproc(HWND hwnd, UINT msg, WPARAM wparam, LPA else lstrcpyA(str, "requested data\r\n"); - size = sizeof(DDEDATA) + lstrlenA(str) + 1; + size = FIELD_OFFSET(DDEDATA, Value[lstrlenA(str) + 1]); hglobal = GlobalAlloc(GMEM_MOVEABLE, size); ok(hglobal != NULL, "Expected non-NULL hglobal\n"); @@ -314,8 +314,7 @@ static void test_ddeml_client(void) { str = (LPSTR)DdeAccessData(hdata, &size); ok(!lstrcmpA(str, "requested data\r\n"), "Expected 'requested data\\r\\n', got %s\n", str); - ok(size == 19 || broken(size == 28), /* sizes are rounded up on win9x */ - "Expected 19, got %d\n", size); + ok(size == 17, "Expected 17, got %d\n", size); ret = DdeUnaccessData(hdata); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); @@ -336,8 +335,7 @@ todo_wine { str = (LPSTR)DdeAccessData(hdata, &size); ok(!lstrcmpA(str, "requested data\r\n"), "Expected 'requested data\\r\\n', got %s\n", str); - ok(size == 19 || broken(size == 28), /* sizes are rounded up on win9x */ - "Expected 19, got %d\n", size); + ok(size == 17, "Expected 17, got %d\n", size); ret = DdeUnaccessData(hdata); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); @@ -357,8 +355,7 @@ todo_wine { str = (LPSTR)DdeAccessData(hdata, &size); ok(!lstrcmpA(str, "requested data\r\n"), "Expected 'requested data\\r\\n', got %s\n", str); - ok(size == 19 || broken(size == 28), /* sizes are rounded up on win9x */ - "Expected 19, got %d\n", size); + ok(size == 17, "Expected 17, got %d\n", size); ret = DdeUnaccessData(hdata); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); @@ -483,8 +480,7 @@ todo_wine { str = (LPSTR)DdeAccessData(hdata, &size); ok(!lstrcmpA(str, "command executed\r\n"), "Expected 'command executed\\r\\n', got %s\n", str); - ok(size == 21 || broken(size == 28), /* sizes are rounded up on win9x */ - "Expected 21, got %d\n", size); + ok(size == 19, "Expected 19, got %d\n", size); ret = DdeUnaccessData(hdata); ok(ret == TRUE, "Expected TRUE, got %d\n", ret);

1 0

André Hentschel : vbscript: Avoid memory leak in compile_select_statement (coverity).
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 1aa6236428ad4254d7024e023d8561b2abb47a0c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1aa6236428ad4254d7024e023… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Oct 30 20:07:10 2012 +0100 vbscript: Avoid memory leak in compile_select_statement (coverity). --- dlls/vbscript/compile.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/vbscript/compile.c b/dlls/vbscript/compile.c index 58afdf4..d324284 100644 --- a/dlls/vbscript/compile.c +++ b/dlls/vbscript/compile.c @@ -836,7 +836,7 @@ static HRESULT compile_select_statement(compile_ctx_t *ctx, select_statement_t * hres = push_instr_addr(ctx, OP_jmp, end_label); if(FAILED(hres)) - return hres; + break; } heap_free(case_labels);

1 0

André Hentschel : wininet: Fix WININET_SetAuthorization on error path (coverity).
by Alexandre Julliard 31 Oct '12

31 Oct '12

Module: wine Branch: master Commit: 5010f0933e496bb367fe2b1414fb000a46bd83d1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5010f0933e496bb367fe2b141… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Oct 30 20:07:25 2012 +0100 wininet: Fix WININET_SetAuthorization on error path (coverity). --- dlls/wininet/dialogs.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wininet/dialogs.c b/dlls/wininet/dialogs.c index a3235a4..418610d 100644 --- a/dlls/wininet/dialogs.c +++ b/dlls/wininet/dialogs.c @@ -188,7 +188,7 @@ static BOOL WININET_SetAuthorization( http_request_t *request, LPWSTR username, q = heap_strdupW(password); if( !q ) { - heap_free(username); + heap_free(p); return FALSE; }

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 October 2012