January 2010 - Wine-commits

Paul Vriens : msvidc32: Fix size parameter for memset (Coccinelle).
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: b543c40de4d09d2ea196c48ed1d8ee2a0e5aa6c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b543c40de4d09d2ea196c48ed… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon Jan 11 08:54:26 2010 +0100 msvidc32: Fix size parameter for memset (Coccinelle). --- dlls/msvidc32/msvideo1.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/msvidc32/msvideo1.c b/dlls/msvidc32/msvideo1.c index 3052964..19784eb 100644 --- a/dlls/msvidc32/msvideo1.c +++ b/dlls/msvidc32/msvideo1.c @@ -508,7 +508,7 @@ LRESULT WINAPI CRAM_DriverProc( DWORD_PTR dwDriverId, HDRVR hdrvr, UINT msg, info = HeapAlloc( GetProcessHeap(), 0, sizeof (Msvideo1Context) ); if( info ) { - memset( info, 0, sizeof info ); + memset( info, 0, sizeof *info ); info->dwMagic = CRAM_MAGIC; } r = (LRESULT) info;

1 0

Paul Vriens : d3d9/tests: Correct the size of a memcmp.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: e0405286bc9f3de2a1e78c880f31ced5c50ec758 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e0405286bc9f3de2a1e78c880… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon Jan 11 09:47:19 2010 +0100 d3d9/tests: Correct the size of a memcmp. --- dlls/d3d9/tests/vertexdeclaration.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/d3d9/tests/vertexdeclaration.c b/dlls/d3d9/tests/vertexdeclaration.c index 72b8f9f..9d507a1 100644 --- a/dlls/d3d9/tests/vertexdeclaration.c +++ b/dlls/d3d9/tests/vertexdeclaration.c @@ -214,7 +214,7 @@ static void test_get_declaration(IDirect3DVertexDeclaration9 *decl_ptr, D3DVERTE ok(hret == D3D_OK && num_elements == expected_num_elements, "GetDeclaration returned: hret 0x%x, num_elements %d. " "Expected hret 0x%x, num_elements %d.\n", hret, num_elements, D3D_OK, expected_num_elements); - i = memcmp(decl, vertex_decl, sizeof(*vertex_decl)); + i = memcmp(decl, vertex_decl, num_elements * sizeof(*vertex_decl)); ok (!i, "Original and returned vertexdeclarations are not the same\n"); ZeroMemory(decl, sizeof(D3DVERTEXELEMENT9) * expected_num_elements); @@ -223,7 +223,7 @@ static void test_get_declaration(IDirect3DVertexDeclaration9 *decl_ptr, D3DVERTE ok(hret == D3D_OK && num_elements == expected_num_elements, "GetDeclaration returned: hret 0x%x, num_elements %d. " "Expected hret 0x%x, num_elements %d.\n", hret, num_elements, D3D_OK, expected_num_elements); - i = memcmp(decl, vertex_decl, sizeof(*vertex_decl)); + i = memcmp(decl, vertex_decl, num_elements * sizeof(*vertex_decl)); ok (!i, "Original and returned vertexdeclarations are not the same\n"); HeapFree(GetProcessHeap(), 0, decl);

1 0

Jason Edmeades : msvcrt: Add support for vsprintf_s.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: a2d4e5cb5da8d6ecef4427b860fcde1b8f47efad URL: http://source.winehq.org/git/wine.git/?a=commit;h=a2d4e5cb5da8d6ecef4427b86… Author: Jason Edmeades <jason.edmeades(a)googlemail.com> Date: Sun Jan 10 13:37:31 2010 -0800 msvcrt: Add support for vsprintf_s. --- dlls/msvcrt/msvcrt.spec | 1 + dlls/msvcrt/wcs.c | 8 ++++++++ 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index c6c749d..819ccb3 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -791,6 +791,7 @@ @ cdecl vfwprintf(ptr wstr ptr) MSVCRT_vfwprintf @ cdecl vprintf(str ptr) MSVCRT_vprintf @ cdecl vsprintf(ptr str ptr) MSVCRT_vsprintf +@ cdecl vsprintf_s(ptr long str ptr) MSVCRT_vsprintf_s @ cdecl vswprintf(ptr wstr ptr) MSVCRT_vswprintf @ cdecl vswprintf_s(ptr long wstr ptr) MSVCRT_vswprintf_s @ cdecl vwprintf(wstr ptr) MSVCRT_vwprintf diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index 6e08822..88fae62 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -830,6 +830,14 @@ int CDECL MSVCRT_vsprintf( char *str, const char *format, __ms_va_list valist) } /********************************************************************* + * vsprintf_s (MSVCRT.@) + */ +int CDECL MSVCRT_vsprintf_s( char *str, MSVCRT_size_t num, const char *format, __ms_va_list valist) +{ + return MSVCRT_vsnprintf(str, num, format, valist); +} + +/********************************************************************* * _vscprintf (MSVCRT.@) */ int CDECL _vscprintf( const char *format, __ms_va_list valist )

1 0

Jason Edmeades : msvcrt: Add support for sprintf_s.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: 97194ec07094957241a36cb15e97e6dfc3e38d84 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97194ec07094957241a36cb15… Author: Jason Edmeades <jason.edmeades(a)googlemail.com> Date: Sun Jan 10 13:40:40 2010 -0800 msvcrt: Add support for sprintf_s. --- dlls/msvcrt/msvcrt.spec | 1 + dlls/msvcrt/wcs.c | 14 ++++++++++++++ 2 files changed, 15 insertions(+), 0 deletions(-) diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index 819ccb3..721bdbc 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -747,6 +747,7 @@ @ cdecl sin(double) MSVCRT_sin @ cdecl sinh(double) MSVCRT_sinh @ varargs sprintf(ptr str) MSVCRT_sprintf +@ varargs sprintf_s(ptr long str) MSVCRT_sprintf_s @ cdecl sqrt(double) MSVCRT_sqrt @ cdecl srand(long) MSVCRT_srand @ varargs sscanf(str str) MSVCRT_sscanf diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index 88fae62..2a1dd25 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -902,6 +902,20 @@ int CDECL MSVCRT_sprintf( char *str, const char *format, ... ) } /********************************************************************* + * sprintf_s (MSVCRT.@) + */ +int CDECL MSVCRT_sprintf_s( char *str, MSVCRT_size_t num, const char *format, ... ) +{ + __ms_va_list ap; + int r; + + __ms_va_start( ap, format ); + r = MSVCRT_vsnprintf( str, num, format, ap ); + __ms_va_end( ap ); + return r; +} + +/********************************************************************* * swprintf (MSVCRT.@) */ int CDECL MSVCRT_swprintf( MSVCRT_wchar_t *str, const MSVCRT_wchar_t *format, ... )

1 0

Detlef Riekenberg : commdlg: Fix PrintDlg parameter checks.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: 93475667a080bf5ea6fd0a4123c52b499ab32067 URL: http://source.winehq.org/git/wine.git/?a=commit;h=93475667a080bf5ea6fd0a412… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Sun Jan 10 20:51:21 2010 +0100 commdlg: Fix PrintDlg parameter checks. --- dlls/commdlg.dll16/printdlg.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/dlls/commdlg.dll16/printdlg.c b/dlls/commdlg.dll16/printdlg.c index 64bb395..ab6be7c 100644 --- a/dlls/commdlg.dll16/printdlg.c +++ b/dlls/commdlg.dll16/printdlg.c @@ -107,6 +107,8 @@ BOOL16 WINAPI PrintDlg16( LPPRINTDLG16 lppd ) PRINTDLGA pd32; BOOL ret; + if (!lppd) return PrintDlgA(NULL); /* generate failure with CDERR_INITIALIZATION */ + pd32.lStructSize = sizeof(pd32); pd32.Flags = lppd->Flags & ~(PD_ENABLEPRINTTEMPLATE | PD_ENABLEPRINTTEMPLATEHANDLE | PD_ENABLESETUPTEMPLATE | PD_ENABLESETUPTEMPLATEHANDLE | @@ -128,6 +130,9 @@ BOOL16 WINAPI PrintDlg16( LPPRINTDLG16 lppd ) if (lppd->Flags & PD_ENABLESETUPHOOK) FIXME( "custom setup hook %p no longer supported\n", lppd->lpfnSetupHook ); + /* Generate failure with CDERR_STRUCTSIZE, when needed */ + if (lppd->lStructSize != sizeof(PRINTDLG16)) pd32.lStructSize--; + if ((ret = PrintDlgA( &pd32 ))) { lppd->hDC = HDC_16( pd32.hDC );

1 0

Juan Lang : secur32: Report an error if libgnutls isn't found.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: 7c3b9a1551e1876d42b8fa24a1a31b8696d593a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c3b9a1551e1876d42b8fa24a… Author: Juan Lang <juan.lang(a)gmail.com> Date: Mon Jan 11 10:43:34 2010 -0800 secur32: Report an error if libgnutls isn't found. --- dlls/secur32/schannel.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/secur32/schannel.c b/dlls/secur32/schannel.c index 27cb1b7..6205f1d 100644 --- a/dlls/secur32/schannel.c +++ b/dlls/secur32/schannel.c @@ -1451,7 +1451,11 @@ void SECUR32_deinitSchannelSP(void) #else /* SONAME_LIBGNUTLS */ -void SECUR32_initSchannelSP(void) {} +void SECUR32_initSchannelSP(void) +{ + ERR("libgnutls not found, SSL connections will fail\n"); +} + void SECUR32_deinitSchannelSP(void) {} #endif /* SONAME_LIBGNUTLS */

1 0

Detlef Riekenberg : avifil32/tests: Add initial tests.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: a60204461afc907f5377d416dbbe8ca5387dc055 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a60204461afc907f5377d416d… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Sun Jan 10 17:37:36 2010 +0100 avifil32/tests: Add initial tests. --- configure | 9 ++++ configure.ac | 1 + dlls/avifil32/tests/Makefile.in | 13 +++++ dlls/avifil32/tests/api.c | 97 +++++++++++++++++++++++++++++++++++++++ 4 files changed, 120 insertions(+), 0 deletions(-) diff --git a/configure b/configure index a97fe17..b92e426 100755 --- a/configure +++ b/configure @@ -13817,6 +13817,14 @@ dlls/avifil32/Makefile: dlls/avifil32/Makefile.in dlls/Makedll.rules" ac_config_files="$ac_config_files dlls/avifil32/Makefile" ALL_MAKEFILES="$ALL_MAKEFILES \\ + dlls/avifil32/tests/Makefile" +test "x$enable_tests" != xno && ALL_TEST_DIRS="$ALL_TEST_DIRS \\ + avifil32/tests" +ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS +dlls/avifil32/tests/Makefile: dlls/avifil32/tests/Makefile.in dlls/Maketest.rules" +ac_config_files="$ac_config_files dlls/avifil32/tests/Makefile" + +ALL_MAKEFILES="$ALL_MAKEFILES \\ dlls/avifile.dll16/Makefile" test "x$enable_win16" != xno && ALL_DLL_DIRS="$ALL_DLL_DIRS \\ avifile.dll16" @@ -18761,6 +18769,7 @@ do "dlls/authz/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/authz/Makefile" ;; "dlls/avicap32/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/avicap32/Makefile" ;; "dlls/avifil32/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/avifil32/Makefile" ;; + "dlls/avifil32/tests/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/avifil32/tests/Makefile" ;; "dlls/avifile.dll16/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/avifile.dll16/Makefile" ;; "dlls/avrt/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/avrt/Makefile" ;; "dlls/bcrypt/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/bcrypt/Makefile" ;; diff --git a/configure.ac b/configure.ac index 6cae9ba..943938b 100644 --- a/configure.ac +++ b/configure.ac @@ -2138,6 +2138,7 @@ WINE_CONFIG_MAKEFILE([dlls/atl/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DI WINE_CONFIG_MAKEFILE([dlls/authz/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/avicap32/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/avifil32/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) +WINE_CONFIG_MAKEFILE([dlls/avifil32/tests/Makefile],[dlls/Maketest.rules],[dlls],[ALL_TEST_DIRS],[enable_tests]) WINE_CONFIG_MAKEFILE([dlls/avifile.dll16/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS],[enable_win16]) WINE_CONFIG_MAKEFILE([dlls/avrt/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/bcrypt/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) diff --git a/dlls/avifil32/tests/Makefile.in b/dlls/avifil32/tests/Makefile.in new file mode 100644 index 0000000..c43ac34 --- /dev/null +++ b/dlls/avifil32/tests/Makefile.in @@ -0,0 +1,13 @@ +TOPSRCDIR = @top_srcdir@ +TOPOBJDIR = ../../.. +SRCDIR = @srcdir@ +VPATH = @srcdir@ +TESTDLL = avifil32.dll +IMPORTS = avifil32 kernel32 + +CTESTS = \ + api.c + +(a)MAKE_TEST_RULES@ + +(a)DEPENDENCIES@ # everything below this line is overwritten by make depend diff --git a/dlls/avifil32/tests/api.c b/dlls/avifil32/tests/api.c new file mode 100644 index 0000000..55b194f --- /dev/null +++ b/dlls/avifil32/tests/api.c @@ -0,0 +1,97 @@ +/* + * Unit test suite for AVI Functions + * + * Copyright 2008 Detlef Riekenberg + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + * + */ + +#include <stdarg.h> + +#include "windef.h" +#include "winbase.h" +#include "winerror.h" +#include "wingdi.h" +#include "vfw.h" +#include "wine/test.h" + +/* ########################### */ + +static const CHAR winetest0[] = "winetest0"; +static const CHAR winetest1[] = "winetest1"; + +/* ########################### */ + +static void test_AVISaveOptions(void) +{ + AVICOMPRESSOPTIONS options[2]; + LPAVICOMPRESSOPTIONS poptions[2]; + PAVISTREAM streams[2] = {NULL, NULL}; + HRESULT hres; + DWORD res; + LONG lres; + + poptions[0] = &options[0]; + poptions[1] = &options[1]; + ZeroMemory(options, sizeof(options)); + + SetLastError(0xdeadbeef); + hres = CreateEditableStream(&streams[0], NULL); + ok(hres == AVIERR_OK, "0: got 0x%x and %p (expected AVIERR_OK)\n", hres, streams[0]); + + SetLastError(0xdeadbeef); + hres = CreateEditableStream(&streams[1], NULL); + ok(hres == AVIERR_OK, "1: got 0x%x and %p (expected AVIERR_OK)\n", hres, streams[1]); + + SetLastError(0xdeadbeef); + hres = EditStreamSetNameA(streams[0], winetest0); + todo_wine ok(hres == AVIERR_OK, "0: got 0x%x (expected AVIERR_OK)\n", hres); + + SetLastError(0xdeadbeef); + hres = EditStreamSetNameA(streams[1], winetest1); + todo_wine ok(hres == AVIERR_OK, "1: got 0x%x (expected AVIERR_OK)\n", hres); + + if (winetest_interactive) { + SetLastError(0xdeadbeef); + res = AVISaveOptions(0, ICMF_CHOOSE_DATARATE |ICMF_CHOOSE_KEYFRAME | ICMF_CHOOSE_ALLCOMPRESSORS, + 2, streams, poptions); + trace("got %u with 0x%x/%u\n", res, GetLastError(), GetLastError()); + } + + SetLastError(0xdeadbeef); + lres = AVISaveOptionsFree(2, poptions); + ok(lres == AVIERR_OK, "got 0x%x with 0x%x/%u\n", lres, GetLastError(), GetLastError()); + + SetLastError(0xdeadbeef); + res = AVIStreamRelease(streams[0]); + ok(res == 0, "0: got refcount %u (expected 0)\n", res); + + SetLastError(0xdeadbeef); + res = AVIStreamRelease(streams[1]); + ok(res == 0, "1: got refcount %u (expected 0)\n", res); + +} + +/* ########################### */ + +START_TEST(api) +{ + + AVIFileInit(); + test_AVISaveOptions(); + AVIFileExit(); + +}

1 0

Julius Schwartzenberg : avifil32/tests: Initial test data and simple test for avifil32.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: e2944fc2eeca14bd4a546265d9dfd434ae79ca22 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e2944fc2eeca14bd4a546265d… Author: Julius Schwartzenberg <julius.schwartzenberg(a)gmail.com> Date: Sun Jan 10 17:37:37 2010 +0100 avifil32/tests: Initial test data and simple test for avifil32. --- dlls/avifil32/tests/api.c | 270 +++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 270 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e2944fc2eeca14bd4a546…

1 0

Julius Schwartzenberg : avifil32/tests: Test corrupt avi list magic.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: ef19d85229c746f6fc7df3fc2ccd7bf0df89bcec URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef19d85229c746f6fc7df3fc2… Author: Julius Schwartzenberg <julius.schwartzenberg(a)gmail.com> Date: Sun Jan 10 01:54:19 2010 +0100 avifil32/tests: Test corrupt avi list magic. --- dlls/avifil32/tests/api.c | 22 ++++++++++++++++++++++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git a/dlls/avifil32/tests/api.c b/dlls/avifil32/tests/api.c index 13c06db..7cc0c1b 100644 --- a/dlls/avifil32/tests/api.c +++ b/dlls/avifil32/tests/api.c @@ -354,6 +354,27 @@ static void test_default_data(void) ok(DeleteFile(filename) !=0, "Deleting file %s failed", filename); } +static void test_amh_corruption(void) +{ + COMMON_AVI_HEADERS cah; + char filename[MAX_PATH]; + PAVIFILE pFile; + int res; + + GetTempPath(MAX_PATH, filename); + strcpy(filename+strlen(filename), testfilename); + + /* Make sure only AVI files with the proper headers will be loaded */ + init_test_struct(&cah); + cah.fh[3] = mmioFOURCC('A', 'V', 'i', ' '); + + create_avi_file(&cah, filename); + res = AVIFileOpen(&pFile, filename, OF_SHARE_DENY_WRITE, 0L); + ok(res != 0, "Able to open file: error=%u\n", res); + + ok(DeleteFile(filename) !=0, "Deleting file %s failed\n", filename); +} + /* ########################### */ START_TEST(api) @@ -362,6 +383,7 @@ START_TEST(api) AVIFileInit(); test_AVISaveOptions(); test_default_data(); + test_amh_corruption(); AVIFileExit(); }

1 0

Julius Schwartzenberg : avifil32/tests: Test how avifile deals with audio stream and wave headers.
by Alexandre Julliard 11 Jan '10

11 Jan '10

Module: wine Branch: master Commit: 9bcb143a4094ede8656bd6b37a318cf9dbf73f73 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9bcb143a4094ede8656bd6b37… Author: Julius Schwartzenberg <julius.schwartzenberg(a)gmail.com> Date: Sun Jan 10 01:57:33 2010 +0100 avifil32/tests: Test how avifile deals with audio stream and wave headers. --- dlls/avifil32/tests/api.c | 72 +++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 72 insertions(+), 0 deletions(-) diff --git a/dlls/avifil32/tests/api.c b/dlls/avifil32/tests/api.c index 7cc0c1b..28bae04 100644 --- a/dlls/avifil32/tests/api.c +++ b/dlls/avifil32/tests/api.c @@ -375,6 +375,76 @@ static void test_amh_corruption(void) ok(DeleteFile(filename) !=0, "Deleting file %s failed\n", filename); } +static void test_ash1_corruption(void) +{ + COMMON_AVI_HEADERS cah; + char filename[MAX_PATH]; + PAVIFILE pFile; + int res; + PAVISTREAM pStream1; + AVISTREAMINFO asi1; + + GetTempPath(MAX_PATH, filename); + strcpy(filename+strlen(filename), testfilename); + + /* Corrupt the sample size in the audio stream header */ + init_test_struct(&cah); + cah.ash1.dwSampleSize = 0xdeadbeef; + + create_avi_file(&cah, filename); + + res = AVIFileOpen(&pFile, filename, OF_SHARE_DENY_WRITE, 0L); + ok(res == 0, "Unable to open file: error=%u\n", res); + + res = AVIFileGetStream(pFile, &pStream1, 0, 1); + ok(res == 0, "Unable to open audio stream: error=%u\n", res); + + res = AVIStreamInfo(pStream1, &asi1, sizeof(AVISTREAMINFO)); + ok(res == 0, "Unable to read stream info: error=%u\n", res); + + /* The result will still be 2, because the value is dynamically replaced with the nBlockAlign + value from the stream format header. The next test will prove this */ + todo_wine{ ok(asi1.dwSampleSize == 2, "got %u (expected 2)\n", asi1.dwSampleSize); } + + AVIStreamRelease(pStream1); + AVIFileRelease(pFile); + ok(DeleteFile(filename) !=0, "Deleting file %s failed", filename); +} + +static void test_ash1_corruption2(void) +{ + COMMON_AVI_HEADERS cah; + char filename[MAX_PATH]; + PAVIFILE pFile; + int res; + PAVISTREAM pStream1; + AVISTREAMINFO asi1; + + GetTempPath(MAX_PATH, filename); + strcpy(filename+strlen(filename), testfilename); + + /* Corrupt the block alignment in the audio format header */ + init_test_struct(&cah); + cah.pcmwf.wf.nBlockAlign = 0xdead; + + create_avi_file(&cah, filename); + + res = AVIFileOpen(&pFile, filename, OF_SHARE_DENY_WRITE, 0L); + ok(res == 0, "Unable to open file: error=%u\n", res); + + res = AVIFileGetStream(pFile, &pStream1, 0, 1); + ok(res == 0, "Unable to open audio stream: error=%u\n", res); + + ok(AVIStreamInfo(pStream1, &asi1, sizeof(AVISTREAMINFO)) == 0, "Unable to read stream info\n"); + + /* The result will also be the corrupt value, as explained above. */ + todo_wine{ ok(asi1.dwSampleSize == 0xdead, "got 0x%x (expected 0xdead)\n", asi1.dwSampleSize); } + + AVIStreamRelease(pStream1); + AVIFileRelease(pFile); + ok(DeleteFile(filename) !=0, "Deleting file %s failed", filename); +} + /* ########################### */ START_TEST(api) @@ -384,6 +454,8 @@ START_TEST(api) test_AVISaveOptions(); test_default_data(); test_amh_corruption(); + test_ash1_corruption(); + test_ash1_corruption2(); AVIFileExit(); }

1 0