wine-commits June 2023

wine-commits@winehq.org

1 participants
633 discussions

Alex Henrie : wininet: Fix memory leaks on error paths in INTERNET_LoadProxySettings (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: faa68dabc3d0c6de929f13e8344da1abc32be1f9 URL: https://gitlab.winehq.org/wine/wine/-/commit/faa68dabc3d0c6de929f13e8344da1… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Tue Jun 6 23:25:02 2023 -0600 wininet: Fix memory leaks on error paths in INTERNET_LoadProxySettings (scan-build). --- dlls/wininet/internet.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index a2d9f00ce6f..5af3aac71ea 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -637,6 +637,7 @@ static LONG INTERNET_LoadProxySettings( proxyinfo_t *lpwpi ) if (!(szProxy = malloc( len ))) { RegCloseKey( key ); + FreeProxyInfo( lpwpi ); return ERROR_OUTOFMEMORY; } RegQueryValueExW( key, L"ProxyOverride", NULL, &type, (BYTE*)szProxy, &len ); @@ -661,6 +662,7 @@ static LONG INTERNET_LoadProxySettings( proxyinfo_t *lpwpi ) if (!(envproxyW = malloc( wcslen(envproxy) * sizeof(WCHAR) ))) { RegCloseKey( key ); + FreeProxyInfo( lpwpi ); return ERROR_OUTOFMEMORY; } lstrcpyW( envproxyW, envproxy );

1 0

Alex Henrie : vbscript: Fix memory leak on error path in RegExp2_Execute (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: 100131fa7e2932120e143c686316f49583921307 URL: https://gitlab.winehq.org/wine/wine/-/commit/100131fa7e2932120e143c686316f4… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Tue Jun 6 23:44:59 2023 -0600 vbscript: Fix memory leak on error path in RegExp2_Execute (scan-build). --- dlls/vbscript/vbregexp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/vbscript/vbregexp.c b/dlls/vbscript/vbregexp.c index 6023b25c929..641798f97dd 100644 --- a/dlls/vbscript/vbregexp.c +++ b/dlls/vbscript/vbregexp.c @@ -1320,9 +1320,9 @@ static HRESULT WINAPI RegExp2_Execute(IRegExp2 *iface, break; hres = add_match(match_collection, add); + IMatch2_Release(add); if(FAILED(hres)) break; - IMatch2_Release(add); if(!(This->flags & REG_GLOB)) break;

1 0

Alex Henrie : jscript: Fix memory leak on error path in regexp_string_match (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: b9aaaacc16360180af0b77c1104d6c9f75b21c5d URL: https://gitlab.winehq.org/wine/wine/-/commit/b9aaaacc16360180af0b77c1104d6c… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Tue Jun 6 23:35:29 2023 -0600 jscript: Fix memory leak on error path in regexp_string_match (scan-build). --- dlls/jscript/jsregexp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/jscript/jsregexp.c b/dlls/jscript/jsregexp.c index 7da365f4e6c..7b97a2951df 100644 --- a/dlls/jscript/jsregexp.c +++ b/dlls/jscript/jsregexp.c @@ -766,8 +766,10 @@ HRESULT regexp_string_match(script_ctx_t *ctx, jsdisp_t *re, jsstr_t *jsstr, jsv } hres = create_array(ctx, match_cnt, &array); - if(FAILED(hres)) + if(FAILED(hres)) { + free(match_result); return hres; + } for(i=0; i < match_cnt; i++) { jsstr_t *tmp_str;

1 0

Alex Henrie : wldap32: Fix memory leak on error path in ldap_start_tls_sW (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: 35dc94bc012e1fee34a6514d1b7d0149d02db0b5 URL: https://gitlab.winehq.org/wine/wine/-/commit/35dc94bc012e1fee34a6514d1b7d01… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jun 7 00:14:40 2023 -0600 wldap32: Fix memory leak on error path in ldap_start_tls_sW (scan-build). --- dlls/wldap32/init.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c index 519466da1ee..c5b11a4de28 100644 --- a/dlls/wldap32/init.c +++ b/dlls/wldap32/init.c @@ -450,14 +450,12 @@ ULONG CDECL ldap_start_tls_sW( LDAP *ld, ULONG *retval, LDAPMessage **result, LD } if (!ld) return ~0u; + if (CONNECTED(ld)) return WLDAP32_LDAP_LOCAL_ERROR; if (serverctrls && !(serverctrlsU = controlarrayWtoU( serverctrls ))) goto exit; if (clientctrls && !(clientctrlsU = controlarrayWtoU( clientctrls ))) goto exit; - else - { - if (CONNECTED(ld)) return WLDAP32_LDAP_LOCAL_ERROR; - ret = map_error( ldap_start_tls_s( CTX(ld), serverctrlsU, clientctrlsU ) ); - } + + ret = map_error( ldap_start_tls_s( CTX(ld), serverctrlsU, clientctrlsU ) ); exit: controlarrayfreeU( serverctrlsU );

1 0

Alex Henrie : hhctrl: Fix memory leak on error path in resolve_filename (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: 7b69313bbb7727cadf5be31ccf9a77759300b43c URL: https://gitlab.winehq.org/wine/wine/-/commit/7b69313bbb7727cadf5be31ccf9a77… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jun 7 00:02:24 2023 -0600 hhctrl: Fix memory leak on error path in resolve_filename (scan-build). --- dlls/hhctrl.ocx/hhctrl.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/dlls/hhctrl.ocx/hhctrl.c b/dlls/hhctrl.ocx/hhctrl.c index 829c1c5305b..3de9a741422 100644 --- a/dlls/hhctrl.ocx/hhctrl.c +++ b/dlls/hhctrl.ocx/hhctrl.c @@ -148,7 +148,14 @@ static BOOL resolve_filename(const WCHAR *env_filename, WCHAR *fullname, DWORD b free(filename); - return (GetFileAttributesW(fullname) != INVALID_FILE_ATTRIBUTES); + if (GetFileAttributesW(fullname) == INVALID_FILE_ATTRIBUTES) + { + if (window) free(*window); + if (index) free(*index); + return FALSE; + } + + return TRUE; } /******************************************************************

1 0

Alex Henrie : dnsapi: Fix memory leak on error path in do_query_netbios (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: 2cabb3f79d43694f9cbc9dfde2769ea9eb24f655 URL: https://gitlab.winehq.org/wine/wine/-/commit/2cabb3f79d43694f9cbc9dfde2769e… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Tue Jun 6 23:12:33 2023 -0600 dnsapi: Fix memory leak on error path in do_query_netbios (scan-build). --- dlls/dnsapi/query.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/dnsapi/query.c b/dlls/dnsapi/query.c index 90a3a3e9950..1802e93dd75 100644 --- a/dlls/dnsapi/query.c +++ b/dlls/dnsapi/query.c @@ -77,6 +77,7 @@ static DNS_STATUS do_query_netbios( PCSTR name, DNS_RECORDA **recp ) if (!record->pName) { status = ERROR_NOT_ENOUGH_MEMORY; + free( record ); goto exit; }

1 0

Alex Henrie : kerberos: Fix memory leaks in (wow64_)query_ticket_cache (scan-build).
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: wine Branch: master Commit: 025e9a0183079d7d30551973d5311c0f1968451d URL: https://gitlab.winehq.org/wine/wine/-/commit/025e9a0183079d7d30551973d5311c… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Mon Jun 5 23:26:24 2023 -0600 kerberos: Fix memory leaks in (wow64_)query_ticket_cache (scan-build). --- dlls/kerberos/unixlib.c | 27 +++++++++++++++------------ 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/dlls/kerberos/unixlib.c b/dlls/kerberos/unixlib.c index a918959dbe5..45cf494aa70 100644 --- a/dlls/kerberos/unixlib.c +++ b/dlls/kerberos/unixlib.c @@ -339,22 +339,30 @@ done: return status; } +static void free_tickets_in_list( struct ticket_list *list ) +{ + ULONG i; + + for (i = 0; i < list->count; i++) + { + free( list->tickets[i].RealmName.Buffer ); + free( list->tickets[i].ServerName.Buffer ); + } + + free( list->tickets ); +} + static NTSTATUS query_ticket_cache( void *args ) { struct query_ticket_cache_params *params = args; struct ticket_list list = { 0 }; NTSTATUS status; - ULONG i; status = kerberos_fill_ticket_list( &list ); if (status == STATUS_SUCCESS) status = copy_tickets_to_client( &list, params->resp, params->out_size ); - for (i = 0; i < list.count; i++) - { - free( list.tickets[i].RealmName.Buffer ); - free( list.tickets[i].ServerName.Buffer ); - } + free_tickets_in_list( &list ); return status; } @@ -1237,17 +1245,12 @@ static NTSTATUS wow64_query_ticket_cache( void *args ) } const *params32 = args; struct ticket_list list = { 0 }; NTSTATUS status; - ULONG i; status = kerberos_fill_ticket_list( &list ); if (status == STATUS_SUCCESS) status = copy_tickets_to_client32( &list, ULongToPtr(params32->resp), ULongToPtr(params32->out_size) ); - for (i = 0; i < list.count; i++) - { - free( list.tickets[i].RealmName.Buffer ); - free( list.tickets[i].ServerName.Buffer ); - } + free_tickets_in_list( &list ); return status; }

1 0

Francisco Casas : vkd3d-shader/hlsl: Support non-constant vector indexing.
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: vkd3d Branch: master Commit: ebf7573571d4bfcd3f38846886106f204fd7f0e8 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/ebf7573571d4bfcd3f38846886106… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Mon May 8 18:25:18 2023 -0400 vkd3d-shader/hlsl: Support non-constant vector indexing. Non-constant vector indexing is not solved with relative addressing in the register indexes because this indexation cannot be at the level of register-components. Mathematical operations must be used instead. --- libs/vkd3d-shader/hlsl.c | 13 ++++ libs/vkd3d-shader/hlsl.h | 2 + libs/vkd3d-shader/hlsl_codegen.c | 88 +++++++++++++++++++++++++- tests/array-index-expr.shader_test | 36 +++++------ tests/expr-indexing.shader_test | 22 +++---- tests/hlsl-matrix-indexing.shader_test | 6 +- tests/hlsl-vector-indexing-uniform.shader_test | 4 +- 7 files changed, 134 insertions(+), 37 deletions(-)

1 0

Francisco Casas : vkd3d-shader/hlsl: Lower dot for non-float types.
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: vkd3d Branch: master Commit: 6cfa8cf85904a6c329608073823af7d7a7d99be6 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/6cfa8cf85904a6c329608073823af… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Wed May 24 01:01:38 2023 -0400 vkd3d-shader/hlsl: Lower dot for non-float types. --- libs/vkd3d-shader/hlsl_codegen.c | 50 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 50 insertions(+) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 3d41524f..9774ca08 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -2389,6 +2389,55 @@ static bool lower_int_abs(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, void return true; } +static bool lower_int_dot(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, void *context) +{ + struct hlsl_ir_node *arg1, *arg2, *mult, *comps[4] = {0}, *res; + struct hlsl_type *type = instr->data_type; + struct hlsl_ir_expr *expr; + unsigned int i, dimx; + + if (instr->type != HLSL_IR_EXPR) + return false; + expr = hlsl_ir_expr(instr); + + if (expr->op != HLSL_OP2_DOT) + return false; + + if (type->base_type == HLSL_TYPE_INT || type->base_type == HLSL_TYPE_UINT) + { + arg1 = expr->operands[0].node; + arg2 = expr->operands[1].node; + assert(arg1->data_type->dimx == arg2->data_type->dimx); + dimx = arg1->data_type->dimx; + + if (!(mult = hlsl_new_binary_expr(ctx, HLSL_OP2_MUL, arg1, arg2))) + return false; + list_add_before(&instr->entry, &mult->entry); + + for (i = 0; i < dimx; ++i) + { + unsigned int s = hlsl_swizzle_from_writemask(1 << i); + + if (!(comps[i] = hlsl_new_swizzle(ctx, s, 1, mult, &instr->loc))) + return false; + list_add_before(&instr->entry, &comps[i]->entry); + } + + res = comps[0]; + for (i = 1; i < dimx; ++i) + { + if (!(res = hlsl_new_binary_expr(ctx, HLSL_OP2_ADD, res, comps[i]))) + return false; + list_add_before(&instr->entry, &res->entry); + } + + hlsl_replace_node(instr, res); + return true; + } + + return false; +} + static bool lower_float_modulus(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, void *context) { struct hlsl_ir_node *arg1, *arg2, *mul1, *neg1, *ge, *neg2, *div, *mul2, *frc, *cond; @@ -3936,6 +3985,7 @@ int hlsl_emit_bytecode(struct hlsl_ctx *ctx, struct hlsl_ir_function_decl *entry hlsl_transform_ir(ctx, lower_narrowing_casts, body, NULL); hlsl_transform_ir(ctx, lower_casts_to_bool, body, NULL); + hlsl_transform_ir(ctx, lower_int_dot, body, NULL); hlsl_transform_ir(ctx, lower_int_division, body, NULL); hlsl_transform_ir(ctx, lower_int_modulus, body, NULL); hlsl_transform_ir(ctx, lower_int_abs, body, NULL);

1 0

Francisco Casas : vkd3d-shader/hlsl: Introduce transform_derefs().
by Alexandre Julliard 07 Jun '23

07 Jun '23

Module: vkd3d Branch: master Commit: 441902bb85dac81fb1d83facc7367830ce804956 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/441902bb85dac81fb1d83facc7367… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Thu May 4 15:06:58 2023 -0400 vkd3d-shader/hlsl: Introduce transform_derefs(). --- libs/vkd3d-shader/hlsl_codegen.c | 77 +++++++++++++++++++++++----------------- 1 file changed, 44 insertions(+), 33 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 2b6c595a..3d41524f 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -123,15 +123,14 @@ static struct hlsl_ir_node *new_offset_instr_from_deref(struct hlsl_ctx *ctx, st } /* TODO: remove when no longer needed, only used for transform_deref_paths_into_offsets() */ -static void replace_deref_path_with_offset(struct hlsl_ctx *ctx, struct hlsl_deref *deref, +static bool replace_deref_path_with_offset(struct hlsl_ctx *ctx, struct hlsl_deref *deref, struct hlsl_ir_node *instr) { const struct hlsl_type *type; struct hlsl_ir_node *offset; struct hlsl_block block; - if (!deref->var) - return; + assert(deref->var); /* register offsets shouldn't be used before this point is reached. */ assert(!deref->offset.node); @@ -143,45 +142,19 @@ static void replace_deref_path_with_offset(struct hlsl_ctx *ctx, struct hlsl_der if (type->class == HLSL_CLASS_STRUCT || type->class == HLSL_CLASS_ARRAY) { hlsl_cleanup_deref(deref); - return; + return true; } deref->offset_regset = hlsl_type_get_regset(type); if (!(offset = new_offset_instr_from_deref(ctx, &block, deref, &instr->loc))) - return; + return false; list_move_before(&instr->entry, &block.instrs); hlsl_cleanup_deref(deref); hlsl_src_from_node(&deref->offset, offset); -} - -/* TODO: remove when no longer needed. */ -static bool transform_deref_paths_into_offsets(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, void *context) -{ - switch(instr->type) - { - case HLSL_IR_LOAD: - replace_deref_path_with_offset(ctx, &hlsl_ir_load(instr)->src, instr); - return true; - case HLSL_IR_STORE: - replace_deref_path_with_offset(ctx, &hlsl_ir_store(instr)->lhs, instr); - return true; - - case HLSL_IR_RESOURCE_LOAD: - replace_deref_path_with_offset(ctx, &hlsl_ir_resource_load(instr)->resource, instr); - replace_deref_path_with_offset(ctx, &hlsl_ir_resource_load(instr)->sampler, instr); - return true; - - case HLSL_IR_RESOURCE_STORE: - replace_deref_path_with_offset(ctx, &hlsl_ir_resource_store(instr)->resource, instr); - return true; - - default: - return false; - } - return false; + return true; } /* Split uniforms into two variables representing the constant and temp @@ -600,6 +573,44 @@ bool hlsl_transform_ir(struct hlsl_ctx *ctx, bool (*func)(struct hlsl_ctx *ctx, return progress; } +static bool transform_instr_derefs(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, void *context) +{ + bool res; + bool (*func)(struct hlsl_ctx *ctx, struct hlsl_deref *, struct hlsl_ir_node *) = context; + + switch(instr->type) + { + case HLSL_IR_LOAD: + res = func(ctx, &hlsl_ir_load(instr)->src, instr); + return res; + + case HLSL_IR_STORE: + res = func(ctx, &hlsl_ir_store(instr)->lhs, instr); + return res; + + case HLSL_IR_RESOURCE_LOAD: + res = func(ctx, &hlsl_ir_resource_load(instr)->resource, instr); + if (hlsl_ir_resource_load(instr)->sampler.var) + res |= func(ctx, &hlsl_ir_resource_load(instr)->sampler, instr); + return res; + + case HLSL_IR_RESOURCE_STORE: + res = func(ctx, &hlsl_ir_resource_store(instr)->resource, instr); + return res; + + default: + return false; + } + return false; +} + +static bool transform_derefs(struct hlsl_ctx *ctx, + bool (*func)(struct hlsl_ctx *ctx, struct hlsl_deref *, struct hlsl_ir_node *), + struct hlsl_block *block) +{ + return hlsl_transform_ir(ctx, transform_instr_derefs, block, func); +} + struct recursive_call_ctx { const struct hlsl_ir_function_decl **backtrace; @@ -3956,7 +3967,7 @@ int hlsl_emit_bytecode(struct hlsl_ctx *ctx, struct hlsl_ir_function_decl *entry hlsl_transform_ir(ctx, track_object_components_usage, body, NULL); /* TODO: move forward, remove when no longer needed */ - hlsl_transform_ir(ctx, transform_deref_paths_into_offsets, body, NULL); + transform_derefs(ctx, replace_deref_path_with_offset, body); while (hlsl_transform_ir(ctx, hlsl_fold_constant_exprs, body, NULL)); do

1 0

← Newer
1
...
49
50
51
52
53
54
55
...
64
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 June 2023