Re: [PATCH v2 0/5] MR1987: Draft: webservices/tests: Move firewall code to a common header.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about include/wine/test_fw.h:
> -
>
> +enum winetest_firewall_op
> +{
> - WINETEST_FW_APP_ADD,
> - WINETEST_FW_APP_REMOVE
>
> +};
> -
>
> +static BOOL winetest_set_firewall( const WCHAR *name, enum winetest_firewall_op op )
> +{
> - HRESULT hr, init;
> - INetFwMgr *mgr = NULL;
> - INetFwPolicy *policy = NULL;
> - INetFwProfile *profile = NULL;
> - INetFwAuthorizedApplication *app = NULL;
> - INetFwAuthorizedApplications *apps = NULL;
> - BSTR str, image = SysAllocStringLen( NULL, MAX_PATH );

There is an [anecdotal evidence][1] that claims that you cannot use `SysAllocStringLen` without having COM initialized first; I'm not sure if I can trust it, so take it with a grain of salt.

[1]: https://stackoverflow.com/questions/40330569/sysallocstring-sysfreestring-co...