Hans Leidekker (@hans) commented about dlls/winhttp/session.c:
*buflen = sizeof(flags); return TRUE; } + case WINHTTP_OPTION_SECURITY_INFO: + { + WINHTTP_SECURITY_INFO *info = (WINHTTP_SECURITY_INFO *)buffer; + SECURITY_STATUS res; + + if (!validate_buffer( buffer, buflen, sizeof(WINHTTP_SECURITY_INFO) )) return FALSE; + + memset(info, 0 , sizeof(WINHTTP_SECURITY_INFO)); + if (!request->netconn->secure) return TRUE; + res = QueryContextAttributesW(&request->netconn->ssl_ctx, SECPKG_ATTR_CONNECTION_INFO, (void*)&info->ConnectionInfo); + if(res != SEC_E_OK) + WARN( "QueryContextAttributesW failed: %#lx\n", res );
Please conform to surrounding style. ``` res = QueryContextAttributesW( &request->netconn->ssl_ctx, SECPKG_ATTR_CONNECTION_INFO, (void *)&info->ConnectionInfo ); if (res != SEC_E_OK) WARN( "QueryContextAttributesW failed: %#lx\n", res ); ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10390#note_132947