February 2016 - Wine-Devel - list.winehq.org

Re: [PATCH 2/3] msvcp110: Add tr2_sys__Symlink implementation and test.

by Piotr Caban

Hi, On 02/05/16 10:30, YongHao Hu wrote: > On 16/1/12 下午9:28, Piotr Caban wrote: >> On 01/11/16 14:56, YongHao Hu wrote: >>> + delete_flag = 1; >>> + for(i=0; i<sizeof(tests)/sizeof(tests[0]); i++) { >>> + errno = 0xdeadbeef; >>> + SetLastError(0xdeadbeef); >>> + ret = p_tr2_sys__Symlink(tests[i].existing_path, >>> tests[i].new_path); >>> + if(ret==ERROR_PRIVILEGE_NOT_HELD || >>> ret==ERROR_INVALID_FUNCTION) { >>> + delete_flag = 0; >>> + tests[i].last_error = ret; >>> + win_skip("Privilege not held or symbolic link not >>> supported, skipping symbolic link tests.\n"); >> I think it would be nicer if the tests are really skipped in this >> case. Please detect such case and return from the function instead of >> adding workarounds in the code for it. >> > Could you explain a bit more details？ You're trying to create the symlinks in loop. Because of this if symbolic links are not supported you're printing this information many times. I think you should try to create one symlink and if it doesn't work just skip all of the tests. Thanks, Piotr

Feb. 8, 2016

Re: [PATCH 2/7] reg: Split reg_printfW into separate functions

by Alexandre Julliard

Hugh McMaster <hugh.mcmaster(a)outlook.com> writes: > +static void __cdecl output_array(const WCHAR *fmt, ...) > +{ > + __ms_va_list va_args; > + > + __ms_va_start(va_args, fmt); > + output_formatstring(fmt, va_args); > + __ms_va_end(va_args); > } That's a strange name, why "array"? -- Alexandre Julliard julliard(a)winehq.org

Feb. 8, 2016

Re: shell32/tests: Allow the short and long forms when checking a ShellExecute() path.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20248 Your paranoid android. === wvistau64 (32 bit shlexec) === shlexec.c:1691: Test failed: WaitForSingleObject returned 258 === w1064 (32 bit shlexec) === shlexec.c:2458: Test failed: failed: rc=29 err=2 shlexec.c:2456: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2458: Test failed: failed: rc=29 err=32 shlexec.c:2458: Test failed: failed: rc=29 err=2 shlexec.c:2456: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2458: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2458: Test failed: failed: rc=29 err=2 shlexec.c:2458: Test failed: failed: rc=29 err=2 shlexec.c:2456: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2458: Test failed: failed: rc=29 err=32 shlexec.c:2470: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wtA1B2.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wtA1B2.tmp\test file.sde")]' shlexec.c:2458: Test failed: failed: rc=29 err=2 shlexec.c:2458: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Fix the ShellExecute() environment variable inheritance checks.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20247 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2453: Test failed: failed: rc=29 err=2 shlexec.c:2451: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2453: Test failed: failed: rc=29 err=32 shlexec.c:2453: Test failed: failed: rc=29 err=2 shlexec.c:2451: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2453: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2453: Test failed: failed: rc=29 err=2 shlexec.c:2453: Test failed: failed: rc=29 err=2 shlexec.c:2451: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2453: Test failed: failed: rc=29 err=32 shlexec.c:2465: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt87E1.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt87E1.tmp\test file.sde")]' shlexec.c:2453: Test failed: failed: rc=29 err=2 shlexec.c:2453: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Move a test_argify() check so it is run even if we could not create associations.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20246 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 shlexec.c:2454: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt5289.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt5289.tmp\test file.sde")]' shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2442: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Skip a few ShellExecute() tests when the process has insufficient privileges.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20245 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2448: Test failed: failed: rc=29 err=2 shlexec.c:2446: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2448: Test failed: failed: rc=29 err=32 shlexec.c:2448: Test failed: failed: rc=29 err=2 shlexec.c:2446: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2448: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2448: Test failed: failed: rc=29 err=2 shlexec.c:2448: Test failed: failed: rc=29 err=2 shlexec.c:2446: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2448: Test failed: failed: rc=29 err=32 shlexec.c:2460: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt477D.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt477D.tmp\test file.sde")]' shlexec.c:2448: Test failed: failed: rc=29 err=2 shlexec.c:2448: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Remove support for a todo_wine flag in ShellExecute()'s test_lnks() loop.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20244 Your paranoid android. === wvistau64 (32 bit shlexec) === shlexec.c:1683: Test failed: WaitForSingleObject returned 258 === w1064 (32 bit shlexec) === shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 shlexec.c:2447: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt9F9F.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt9F9F.tmp\test file.sde")]' shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2435: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Fix a ShellExecute() race condition.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20243 Your paranoid android. === wvistau64 (32 bit shlexec) === shlexec.c:1675: Test failed: WaitForSingleObject returned 258 shlexec.c:1704: Test failed: argvA3 expected 'Open', but key not found or empty shlexec.c:1712: Test failed: argvA4 expected 'C:\Users\winetest\AppData\Local\Temp\wtDD72.tmp\test file.noassoc.shlexec', but key not found or empty shlexec.c:1704: Test failed: argvA3 expected 'QuotedLowerL', got 'Open' shlexec.c:1712: Test failed: argvA4 expected 'C:\Users\winetest\AppData\Local\Temp\wtDD72.tmp\test file.shlexec', got 'C:\Users\winetest\AppData\Local\Temp\wtDD72.tmp\test file.noassoc.shlexec' shlexec.c:1704: Test failed: argvA3 expected 'QuotedUpperL', got 'QuotedLowerL' === w1064 (32 bit shlexec) === shlexec.c:2443: Test failed: failed: rc=29 err=2 shlexec.c:2441: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2443: Test failed: failed: rc=29 err=32 shlexec.c:2443: Test failed: failed: rc=29 err=2 shlexec.c:2441: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2443: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2443: Test failed: failed: rc=29 err=2 shlexec.c:2443: Test failed: failed: rc=29 err=2 shlexec.c:2441: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2443: Test failed: failed: rc=29 err=32 shlexec.c:2455: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt48B4.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt48B4.tmp\test file.sde")]' shlexec.c:2443: Test failed: failed: rc=29 err=2 shlexec.c:2443: Test failed: failed: rc=29 err=2

Feb. 8, 2016

Re: shell32/tests: Fix test_dde_default_app() on Windows XP SP1.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=20242 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32 shlexec.c:2442: Test failed: failed: rc=29 err=2 shlexec.c:2440: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2442: Test failed: failed: rc=29 err=32

Feb. 8, 2016

Re: [v2] shell32: fix two bugs in IQueryAssocations

by Theodore Dubois

On Feb 7, 2016, at 4:38 PM, Nikolay Sivov <bunglehead(a)gmail.com> wrote: > > On 08.02.2016 3:07, Theodore Dubois wrote: >> >>> On Feb 7, 2016, at 4:25 AM, Nikolay Sivov <bunglehead(a)gmail.com> wrote: >>> >>> Why do you need this? >> >> So I can assert in Init. > > That's not necessary. If you don’t like it, I’ll take it out in the next version of the patch. > >> >>> >>>> This->hkeyProgID = This->hkeySource; >>> >>> I know it's not something you introduced, but it's dirty and should be >>> fixed, because it leads to double free on a same handle. >> >> I’ll fix that in the next version. >> >>> Basically you remove everything, and tests don't show that you should do >>> that. For example looking at Win7 registry I see HKCR\.exe with default >>> value of "exefile", then default value for HKCR\exefile is Application >>> (and also there is a FriendlyTypeName key that's most likely localized). >>> Current code will return "Application" and that seems like a better >>> candidate for a friendly name. >> >> In that case, hkeyProgID would refer to HKCR\exefile, and then I return the >> default value from that key which is “Application”. It used to start with hkeySource >> (would point to HKCR\.exe) and do some digging, but my code does that > work in Init. > >> + if (*pszAssoc == '.') >> + { >> + /* for a file extension, the progid is the default value */ >> + hr = ASSOC_GetValue(This->hkeySource, NULL, (void**)&progId, NULL); >> + if (FAILED(hr)) >> + return hr; >> + } > > What happens if default value is not set? It would return HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND). Windows reverts to all the defaults, such as capitalizing the file extension and adding “File” when you ask for the doc name. So, probably should not fail here, and the doc name code should be fixed so it does that when hkeyProgID is null. > >> + else /* if (*pszAssoc == '{') */ >> + { >> + assert(*pszAssoc == '{'); >> + >> + HKEY progIdKey; >> + /* for a clsid, the progid is the default value of the ProgID subkey */ >> + ret = RegOpenKeyExW(This->hkeySource, >> + szProgID, >> + 0, >> + KEY_READ, >> + &progIdKey); >> + if (ret != ERROR_SUCCESS) >> + return HRESULT_FROM_WIN32(ret); >> + hr = ASSOC_GetValue(progIdKey, NULL, (void**)&progId, NULL); >> + if (FAILED(hr)) >> + return hr; >> + RegCloseKey(progIdKey); >> + } > > Now Init() will fail on missing progid subkey, and it didn't before. > Also this introduces leaks. Again, instead of failing, I really should just revert to default values in GetString. And yes, I forgot to free progId. I’ll fix that. ~Theodore

Feb. 8, 2016