wine-commits October 2006

wine-commits@winehq.org

3 participants
1032 discussions

James Hawkins : msi: Add tests for installing from continuous cabinets.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 1f3f88bf67d32983ff00d96bf58bf91c6e72ae02 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1f3f88bf67d32983ff00d96bf… Author: James Hawkins <truiken(a)gmail.com> Date: Mon Oct 23 18:15:28 2006 -0700 msi: Add tests for installing from continuous cabinets. --- dlls/msi/tests/install.c | 126 ++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 117 insertions(+), 9 deletions(-) diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index 2d845c4..aed5a78 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -147,6 +147,36 @@ static const CHAR service_control_dat[] "ServiceControl\tServiceControl\n" "ServiceControl\tTestService\t8\t\t0\tservice_comp"; +/* tables for test_continuouscabs */ +static const CHAR cc_component_dat[] = "Component\tComponentId\tDirectory_\tAttributes\tCondition\tKeyPath\n" + "s72\tS38\ts72\ti2\tS255\tS72\n" + "Component\tComponent\n" + "maximus\t\tMSITESTDIR\t0\t1\tmaximus\n" + "augustus\t\tMSITESTDIR\t0\t1\taugustus"; + +static const CHAR cc_feature_dat[] = "Feature\tFeature_Parent\tTitle\tDescription\tDisplay\tLevel\tDirectory_\tAttributes\n" + "s38\tS38\tL64\tL255\tI2\ti2\tS72\ti2\n" + "Feature\tFeature\n" + "feature\t\t\t\t2\t1\tTARGETDIR\t0"; + +static const CHAR cc_feature_comp_dat[] = "Feature_\tComponent_\n" + "s38\ts72\n" + "FeatureComponents\tFeature_\tComponent_\n" + "feature\tmaximus\n" + "feature\taugustus"; + +static const CHAR cc_file_dat[] = "File\tComponent_\tFileName\tFileSize\tVersion\tLanguage\tAttributes\tSequence\n" + "s72\ts72\tl255\ti4\tS72\tS20\tI2\ti2\n" + "File\tFile\n" + "maximus\tmaximus\tmaximus\t500\t\t\t16384\t1\n" + "augustus\taugustus\taugustus\t50000\t\t\t16384\t1"; + +static const CHAR cc_media_dat[] = "DiskId\tLastSequence\tDiskPrompt\tCabinet\tVolumeLabel\tSource\n" + "i2\ti4\tL64\tS255\tS32\tS72\n" + "Media\tDiskId\n" + "1\t10\t\ttest1.cab\tDISK1\t\n" + "2\t2\t\ttest2.cab\tDISK2\t\n"; + typedef struct _msi_table { const CHAR *filename; @@ -171,10 +201,22 @@ static const msi_table tables[] = ADD_TABLE(service_control) }; +static const msi_table cc_tables[] = +{ + ADD_TABLE(cc_component), + ADD_TABLE(directory), + ADD_TABLE(cc_feature), + ADD_TABLE(cc_feature_comp), + ADD_TABLE(cc_file), + ADD_TABLE(install_exec_seq), + ADD_TABLE(cc_media), + ADD_TABLE(property), +}; + /* cabinet definitions */ /* make the max size large so there is only one cab file */ -#define MEDIA_SIZE 999999999 +#define MEDIA_SIZE 0x7FFFFFFF #define FOLDER_THRESHOLD 900000 /* the FCI callbacks */ @@ -191,6 +233,7 @@ static void mem_free(void *memory) static BOOL get_next_cabinet(PCCAB pccab, ULONG cbPrevCab, void *pv) { + sprintf(pccab->szCab, pv, pccab->iCab); return TRUE; } @@ -336,11 +379,10 @@ static INT_PTR get_open_info(char *pszNa return (INT_PTR)handle; } -static void add_file(HFCI hfci, const char *file, TCOMP compress) +static BOOL add_file(HFCI hfci, const char *file, TCOMP compress) { char path[MAX_PATH]; char filename[MAX_PATH]; - BOOL res; lstrcpyA(path, CURR_DIR); lstrcatA(path, "\\"); @@ -348,9 +390,8 @@ static void add_file(HFCI hfci, const ch lstrcpyA(filename, file); - res = FCIAddFile(hfci, path, filename, FALSE, get_next_cabinet, progress, - get_open_info, compress); - ok(res, "Expected FCIAddFile to succeed\n"); + return FCIAddFile(hfci, path, filename, FALSE, get_next_cabinet, + progress, get_open_info, compress); } static void set_cab_parameters(PCCAB pCabParams, const CHAR *name, DWORD max_size) @@ -360,6 +401,7 @@ static void set_cab_parameters(PCCAB pCa pCabParams->cb = max_size; pCabParams->cbFolderThresh = FOLDER_THRESHOLD; pCabParams->setID = 0xbeef; + pCabParams->iCab = 1; lstrcpyA(pCabParams->szCabPath, CURR_DIR); lstrcatA(pCabParams->szCabPath, "\\"); lstrcpyA(pCabParams->szCab, name); @@ -384,7 +426,8 @@ static void create_cab_file(const CHAR * ptr = files; while (*ptr) { - add_file(hfci, ptr, tcompTYPE_MSZIP); + res = add_file(hfci, ptr, tcompTYPE_MSZIP); + ok(res, "Failed to add file: %s\n", ptr); ptr += lstrlen(ptr) + 1; } @@ -432,8 +475,6 @@ static void create_file(const CHAR *name static void create_test_files(void) { - get_program_files_dir(PROG_FILES_DIR); - CreateDirectoryA("msitest", NULL); create_file("msitest\\one.txt", 100); CreateDirectoryA("msitest\\first", NULL); @@ -716,10 +757,76 @@ static void test_packagecoltypes(void) DeleteFile(msifile); } +static void create_cc_test_files(void) +{ + CCAB cabParams; + HFCI hfci; + ERF erf; + BOOL res; + + create_file("maximus", 500); + create_file("augustus", 50000); + + set_cab_parameters(&cabParams, "test1.cab", 200); + + hfci = FCICreate(&erf, file_placed, mem_alloc, mem_free, fci_open, + fci_read, fci_write, fci_close, fci_seek, fci_delete, + get_temp_file, &cabParams, (void*)"test%d.cab"); + ok(hfci != NULL, "Failed to create an FCI context\n"); + + /* spews out hundreds of cab files. re-enable when cabinet.dll is fixed */ +#if 0 + res = add_file(hfci, "maximus", tcompTYPE_MSZIP); + ok(res, "Failed to add file maximus\n"); + + res = add_file(hfci, "augustus", tcompTYPE_MSZIP); + todo_wine + { + ok(res, "Failed to add file augustus\n"); + } +#endif + + res = FCIFlushCabinet(hfci, FALSE, get_next_cabinet, progress); + ok(res, "Failed to flush the cabinet\n"); + + res = FCIDestroy(hfci); + ok(res, "Failed to destroy the cabinet\n"); + + DeleteFile("maximus"); + DeleteFile("augustus"); +} + +static void test_continuouscabs(void) +{ + UINT r; + + create_cc_test_files(); + create_database(msifile, cc_tables, sizeof(cc_tables) / sizeof(msi_table)); + + r = MsiInstallProductA(msifile, NULL); + todo_wine + { + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r); + } + + todo_wine + { + ok(delete_pf("msitest\\maximus", TRUE), "File not installed\n"); + ok(delete_pf("msitest\\augustus", TRUE), "File not installed\n"); + } + ok(delete_pf("msitest", FALSE), "File not installed\n"); + + DeleteFile("test1.cab"); + DeleteFile("test2.cab"); + DeleteFile(msifile); +} + START_TEST(install) { DWORD len; + get_program_files_dir(PROG_FILES_DIR); + GetCurrentDirectoryA(MAX_PATH, CURR_DIR); len = lstrlenA(CURR_DIR); @@ -729,4 +836,5 @@ START_TEST(install) test_MsiInstallProduct(); test_MsiSetComponentState(); test_packagecoltypes(); + test_continuouscabs(); }

1 0

James Hawkins : msi: Allow more customization of install test files.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 3fc13d4b051bd7189e8d8af85996dad755759566 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3fc13d4b051bd7189e8d8af85… Author: James Hawkins <truiken(a)gmail.com> Date: Mon Oct 23 18:11:24 2006 -0700 msi: Allow more customization of install test files. --- dlls/msi/tests/install.c | 54 ++++++++++++++++++++++++++++----------------- 1 files changed, 33 insertions(+), 21 deletions(-) diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index bd22916..2d845c4 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -336,25 +336,28 @@ static INT_PTR get_open_info(char *pszNa return (INT_PTR)handle; } -static void add_file(HFCI hfci, char *file) +static void add_file(HFCI hfci, const char *file, TCOMP compress) { char path[MAX_PATH]; + char filename[MAX_PATH]; BOOL res; lstrcpyA(path, CURR_DIR); lstrcatA(path, "\\"); lstrcatA(path, file); - res = FCIAddFile(hfci, path, file, FALSE, get_next_cabinet, progress, - get_open_info, tcompTYPE_MSZIP); + lstrcpyA(filename, file); + + res = FCIAddFile(hfci, path, filename, FALSE, get_next_cabinet, progress, + get_open_info, compress); ok(res, "Expected FCIAddFile to succeed\n"); } -static void set_cab_parameters(PCCAB pCabParams, const CHAR *name) +static void set_cab_parameters(PCCAB pCabParams, const CHAR *name, DWORD max_size) { ZeroMemory(pCabParams, sizeof(CCAB)); - pCabParams->cb = MEDIA_SIZE; + pCabParams->cb = max_size; pCabParams->cbFolderThresh = FOLDER_THRESHOLD; pCabParams->setID = 0xbeef; lstrcpyA(pCabParams->szCabPath, CURR_DIR); @@ -362,16 +365,15 @@ static void set_cab_parameters(PCCAB pCa lstrcpyA(pCabParams->szCab, name); } -static void create_cab_file(const CHAR *name) +static void create_cab_file(const CHAR *name, DWORD max_size, const CHAR *files) { CCAB cabParams; + LPCSTR ptr; HFCI hfci; ERF erf; - static CHAR four_txt[] = "four.txt", - five_txt[] = "five.txt"; BOOL res; - set_cab_parameters(&cabParams, name); + set_cab_parameters(&cabParams, name, max_size); hfci = FCICreate(&erf, file_placed, mem_alloc, mem_free, fci_open, fci_read, fci_write, fci_close, fci_seek, fci_delete, @@ -379,8 +381,12 @@ static void create_cab_file(const CHAR * ok(hfci != NULL, "Failed to create an FCI context\n"); - add_file(hfci, four_txt); - add_file(hfci, five_txt); + ptr = files; + while (*ptr) + { + add_file(hfci, ptr, tcompTYPE_MSZIP); + ptr += lstrlen(ptr) + 1; + } res = FCIFlushCabinet(hfci, FALSE, get_next_cabinet, progress); ok(res, "Failed to flush the cabinet\n"); @@ -406,15 +412,21 @@ static BOOL get_program_files_dir(LPSTR return TRUE; } -static void create_file(const CHAR *name) +static void create_file(const CHAR *name, DWORD size) { HANDLE file; - DWORD written; + DWORD written, left; file = CreateFileA(name, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, 0, NULL); ok(file != INVALID_HANDLE_VALUE, "Failure to open file %s\n", name); WriteFile(file, name, strlen(name), &written, NULL); WriteFile(file, "\n", strlen("\n"), &written, NULL); + + left = size - lstrlen(name) - 1; + + SetFilePointer(file, left, NULL, FILE_CURRENT); + SetEndOfFile(file); + CloseHandle(file); } @@ -423,18 +435,18 @@ static void create_test_files(void) get_program_files_dir(PROG_FILES_DIR); CreateDirectoryA("msitest", NULL); - create_file("msitest\\one.txt"); + create_file("msitest\\one.txt", 100); CreateDirectoryA("msitest\\first", NULL); - create_file("msitest\\first\\two.txt"); + create_file("msitest\\first\\two.txt", 100); CreateDirectoryA("msitest\\second", NULL); - create_file("msitest\\second\\three.txt"); + create_file("msitest\\second\\three.txt", 100); - create_file("four.txt"); - create_file("five.txt"); - create_cab_file("msitest.cab"); + create_file("four.txt", 100); + create_file("five.txt", 100); + create_cab_file("msitest.cab", MEDIA_SIZE, "four.txt\0five.txt\0"); - create_file("msitest\\filename"); - create_file("msitest\\service.exe"); + create_file("msitest\\filename", 100); + create_file("msitest\\service.exe", 100); DeleteFileA("four.txt"); DeleteFileA("five.txt");

1 0

James Hawkins : msi: Remove unused function pointer and definitions.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: a3a3d6c2547b36892f7dbc9ffe7a2806b87edb0e URL: http://source.winehq.org/git/wine.git/?a=commit;h=a3a3d6c2547b36892f7dbc9ff… Author: James Hawkins <truiken(a)gmail.com> Date: Mon Oct 23 18:10:14 2006 -0700 msi: Remove unused function pointer and definitions. --- dlls/msi/tests/install.c | 45 --------------------------------------------- 1 files changed, 0 insertions(+), 45 deletions(-) diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index 327b938..bd22916 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -177,35 +177,6 @@ static const msi_table tables[] = #define MEDIA_SIZE 999999999 #define FOLDER_THRESHOLD 900000 -/* The following defintions were copied from dlls/cabinet/cabinet.h - * because they are undocumented in windows. - */ - -/* EXTRACTdest flags */ -#define EXTRACT_FILLFILELIST 0x00000001 -#define EXTRACT_EXTRACTFILES 0x00000002 - -struct ExtractFileList { - LPSTR filename; - struct ExtractFileList *next; - BOOL unknown; /* always 1L */ -}; - -/* the first parameter of the function extract */ -typedef struct { - long result1; /* 0x000 */ - long unknown1[3]; /* 0x004 */ - struct ExtractFileList *filelist; /* 0x010 */ - long filecount; /* 0x014 */ - long flags; /* 0x018 */ - char directory[0x104]; /* 0x01c */ - char lastfile[0x20c]; /* 0x120 */ -} EXTRACTDEST; - -/* cabinet function pointers */ -HMODULE hCabinet; -static HRESULT (WINAPI *pExtract)(EXTRACTDEST*, LPCSTR); - /* the FCI callbacks */ static void *mem_alloc(ULONG cb) @@ -418,19 +389,6 @@ static void create_cab_file(const CHAR * ok(res, "Failed to destroy the cabinet\n"); } -static BOOL init_function_pointers(void) -{ - hCabinet = LoadLibraryA("cabinet.dll"); - if (!hCabinet) - return FALSE; - - pExtract = (void *)GetProcAddress(hCabinet, "Extract"); - if (!pExtract) - return FALSE; - - return TRUE; -} - static BOOL get_program_files_dir(LPSTR buf) { HKEY hkey; @@ -750,9 +708,6 @@ START_TEST(install) { DWORD len; - if (!init_function_pointers()) - return; - GetCurrentDirectoryA(MAX_PATH, CURR_DIR); len = lstrlenA(CURR_DIR);

1 0

James Hawkins : msi: Remove two unnecessary install tables.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: a93763ae21ae73dd9bf252d769a8ce31ccd02833 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a93763ae21ae73dd9bf252d76… Author: James Hawkins <truiken(a)gmail.com> Date: Mon Oct 23 18:07:42 2006 -0700 msi: Remove two unnecessary install tables. --- dlls/msi/tests/install.c | 31 ------------------------------- 1 files changed, 0 insertions(+), 31 deletions(-) diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index 83e68df..327b938 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -34,35 +34,6 @@ CHAR PROG_FILES_DIR[MAX_PATH]; /* msi database data */ -static const CHAR admin_exec_seq_dat[] = "Action\tCondition\tSequence\n" - "s72\tS255\tI2\n" - "AdminExecuteSequence\tAction\n" - "CostFinalize\t\t1000\n" - "CostInitialize\t\t800\n" - "FileCost\t\t900\n" - "InstallAdminPackage\t\t3900\n" - "InstallFiles\t\t4000\n" - "InstallFinalize\t\t6600\n" - "InstallInitialize\t\t1500\n" - "InstallValidate\t\t1400"; - -static const CHAR advt_exec_seq_dat[] = "Action\tCondition\tSequence\n" - "s72\tS255\tI2\n" - "AdvtExecuteSequence\tAction\n" - "CostFinalize\t\t1000\n" - "CostInitialize\t\t800\n" - "CreateShortcuts\t\t4500\n" - "InstallFinalize\t\t6600\n" - "InstallInitialize\t\t1500\n" - "InstallValidate\t\t1400\n" - "PublishComponents\t\t6200\n" - "PublishFeatures\t\t6300\n" - "PublishProduct\t\t6400\n" - "RegisterClassInfo\t\t4600\n" - "RegisterExtensionInfo\t\t4700\n" - "RegisterMIMEInfo\t\t4900\n" - "RegisterProgIdInfo\t\t4800"; - static const CHAR component_dat[] = "Component\tComponentId\tDirectory_\tAttributes\tCondition\tKeyPath\n" "s72\tS38\ts72\ti2\tS255\tS72\n" "Component\tComponent\n" @@ -187,8 +158,6 @@ #define ADD_TABLE(x) {#x".idt", x##_dat, static const msi_table tables[] = { - ADD_TABLE(admin_exec_seq), - ADD_TABLE(advt_exec_seq), ADD_TABLE(component), ADD_TABLE(directory), ADD_TABLE(feature),

1 0

James Hawkins : msi: Add support for localizable strings in MsiDatabaseImport.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 5b19cc79bd4c35f1710280e569ec2847597b679c URL: http://source.winehq.org/git/wine.git/?a=commit;h=5b19cc79bd4c35f1710280e56… Author: James Hawkins <truiken(a)gmail.com> Date: Mon Oct 23 18:07:04 2006 -0700 msi: Add support for localizable strings in MsiDatabaseImport. --- dlls/msi/database.c | 15 ++++++++++++--- dlls/msi/tests/install.c | 5 +---- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index e496acf..8abed5c 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -351,7 +351,7 @@ static LPWSTR msi_build_createsql_column LPCWSTR type; DWORD sql_size = 1, i, len; WCHAR expanded[128], *ptr; - WCHAR size[10], comma[2], extra[10]; + WCHAR size[10], comma[2], extra[30]; static const WCHAR column_fmt[] = {'`','%','s','`',' ','%','s','%','s','%','s','%','s',' ',0}; static const WCHAR size_fmt[] = {'(','%','s',')',0}; @@ -359,6 +359,7 @@ static LPWSTR msi_build_createsql_column static const WCHAR type_int[] = {'I','N','T',0}; static const WCHAR type_long[] = {'L','O','N','G',0}; static const WCHAR type_notnull[] = {' ','N','O','T',' ','N','U','L','L',0}; + static const WCHAR localizable[] = {' ','L','O','C','A','L','I','Z','A','B','L','E',0}; columns = msi_alloc_zero(sql_size * sizeof(WCHAR)); if (!columns) @@ -376,12 +377,20 @@ static LPWSTR msi_build_createsql_column ptr = &types[i][1]; len = atolW(ptr); + extra[0] = '\0'; switch (types[i][0]) { - case 'l': case 's': + case 'l': lstrcpyW(extra, type_notnull); - case 'L': case 'S': + case 'L': + lstrcatW(extra, localizable); + type = type_char; + sprintfW(size, size_fmt, ptr); + break; + case 's': + lstrcpyW(extra, type_notnull); + case 'S': type = type_char; sprintfW(size, size_fmt, ptr); break; diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index f5a2eeb..83e68df 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -766,13 +766,10 @@ static void test_packagecoltypes(void) ok(count == 6, "Expected 6, got %d\n", count); ok(check_record(rec, 1, "i2"), "wrong column label\n"); ok(check_record(rec, 2, "i4"), "wrong column label\n"); + ok(check_record(rec, 3, "L64"), "wrong column label\n"); ok(check_record(rec, 4, "S255"), "wrong column label\n"); ok(check_record(rec, 5, "S32"), "wrong column label\n"); ok(check_record(rec, 6, "S72"), "wrong column label\n"); - todo_wine - { - ok(check_record(rec, 3, "L64"), "wrong column label\n"); - } MsiCloseHandle(rec); MsiCloseHandle(view);

1 0

Jacek Caban : urlmon: Added video/mpeg mime filter.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 53f68ae3e663abec500a3df6e69d28495822da93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=53f68ae3e663abec500a3df6e… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Oct 23 23:40:58 2006 +0200 urlmon: Added video/mpeg mime filter. --- dlls/urlmon/tests/misc.c | 13 ++++++++++++- dlls/urlmon/urlmon_main.c | 9 +++++++++ 2 files changed, 21 insertions(+), 1 deletions(-) diff --git a/dlls/urlmon/tests/misc.c b/dlls/urlmon/tests/misc.c index 5cfe701..0fa5f32 100644 --- a/dlls/urlmon/tests/misc.c +++ b/dlls/urlmon/tests/misc.c @@ -337,6 +337,7 @@ static const WCHAR mimeImageBmp[] = {'i' static const WCHAR mimeImageXPng[] = {'i','m','a','g','e','/','x','-','p','n','g',0}; static const WCHAR mimeImageTiff[] = {'i','m','a','g','e','/','t','i','f','f',0}; static const WCHAR mimeVideoAvi[] = {'v','i','d','e','o','/','a','v','i',0}; +static const WCHAR mimeVideoMpeg[] = {'v','i','d','e','o','/','m','p','e','g',0}; static const struct { LPCWSTR url; @@ -400,6 +401,11 @@ static BYTE data46[] = {'R','I','F','F', static BYTE data47[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x21,0xff}; static BYTE data48[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x20,'<','h','t','m','l','>'}; static BYTE data49[] = {'R','I','F','F',0x0f,0x0f,0xf0,0xf0,'A','V','I',0x20,0xf0,0x00}; +static BYTE data50[] = {0x00,0x00,0x01,0xb3,0xff}; +static BYTE data51[] = {0x00,0x00,0x01,0xba,0xff}; +static BYTE data52[] = {0x00,0x00,0x01,0xb8,0xff}; +static BYTE data53[] = {0x00,0x00,0x01,0xba}; +static BYTE data54[] = {0x00,0x00,0x01,0xba,'<','h','t','m','l','>'}; static const struct { BYTE *data; @@ -454,7 +460,12 @@ static const struct { {data46, sizeof(data46), mimeTextPlain}, {data47, sizeof(data47), mimeTextPlain}, {data48, sizeof(data48), mimeTextHtml}, - {data49, sizeof(data49), mimeVideoAvi} + {data49, sizeof(data49), mimeVideoAvi}, + {data50, sizeof(data50), mimeVideoMpeg}, + {data51, sizeof(data51), mimeVideoMpeg}, + {data52, sizeof(data52), mimeAppOctetStream}, + {data53, sizeof(data53), mimeAppOctetStream}, + {data54, sizeof(data54), mimeTextHtml} }; static void test_FindMimeFromData(void) diff --git a/dlls/urlmon/urlmon_main.c b/dlls/urlmon/urlmon_main.c index ce5fd5b..d0d8f2c 100644 --- a/dlls/urlmon/urlmon_main.c +++ b/dlls/urlmon/urlmon_main.c @@ -462,6 +462,13 @@ static BOOL video_avi_filter(const BYTE && b[8] == 'A' && b[9] == 'V' && b[10] == 'I' && b[11] == 0x20; } +static BOOL video_mpeg_filter(const BYTE const *b, DWORD size) +{ + return size > 4 + && !b[0] && !b[1] && b[2] == 0x01 + && (b[3] == 0xb3 || b[3] == 0xba); +} + static BOOL text_plain_filter(const BYTE const *b, DWORD size) { const BYTE *ptr; @@ -521,6 +528,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC static const WCHAR wszImageXPng[] = {'i','m','a','g','e','/','x','-','p','n','g',0}; static const WCHAR wszImageBmp[] = {'i','m','a','g','e','/','b','m','p',0}; static const WCHAR wszVideoAvi[] = {'v','i','d','e','o','/','a','v','i',0}; + static const WCHAR wszVideoMpeg[] = {'v','i','d','e','o','/','m','p','e','g',0}; static const WCHAR wszTextPlain[] = {'t','e','x','t','/','p','l','a','i','n','\0'}; static const WCHAR wszAppOctetStream[] = {'a','p','p','l','i','c','a','t','i','o','n','/', 'o','c','t','e','t','-','s','t','r','e','a','m','\0'}; @@ -536,6 +544,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC {wszImageXPng, image_xpng_filter}, {wszImageBmp, image_bmp_filter}, {wszVideoAvi, video_avi_filter}, + {wszVideoMpeg, video_mpeg_filter}, {wszTextPlain, text_plain_filter}, {wszAppOctetStream, application_octet_stream_filter} };

1 0

Jacek Caban : urlmon: Code clean up.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 3515dbc37ce2cf806c810b60c68955ecc600fa46 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3515dbc37ce2cf806c810b60c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Oct 23 23:39:42 2006 +0200 urlmon: Code clean up. --- dlls/urlmon/urlmon_main.c | 50 ++++++++++++++++++++++---------------------- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/dlls/urlmon/urlmon_main.c b/dlls/urlmon/urlmon_main.c index 916747b..ce5fd5b 100644 --- a/dlls/urlmon/urlmon_main.c +++ b/dlls/urlmon/urlmon_main.c @@ -402,9 +402,8 @@ void WINAPI ReleaseBindInfo(BINDINFO* pb * * Determines the Multipurpose Internet Mail Extensions (MIME) type from the data provided. */ -static BOOL text_html_filter(LPVOID buf, DWORD size) +static BOOL text_html_filter(const BYTE const *b, DWORD size) { - const char *b = buf; int i; if(size < 5) @@ -422,10 +421,8 @@ static BOOL text_html_filter(LPVOID buf, return FALSE; } -static BOOL image_gif_filter(LPVOID buf, DWORD size) +static BOOL image_gif_filter(const BYTE const *b, DWORD size) { - const BYTE const *b = buf; - return size >= 6 && (b[0] == 'G' || b[0] == 'g') && (b[1] == 'I' || b[1] == 'i') @@ -435,44 +432,41 @@ static BOOL image_gif_filter(LPVOID buf, && (b[5] == 'A' || b[5] == 'a'); } -static BOOL image_pjpeg_filter(LPVOID buf, DWORD size) +static BOOL image_pjpeg_filter(const BYTE const *b, DWORD size) { - return size > 2 && *(BYTE*)buf == 0xff && *((BYTE*)buf+1) == 0xd8; + return size > 2 && b[0] == 0xff && b[1] == 0xd8; } -static BOOL image_tiff_filter(LPVOID buf, DWORD size) +static BOOL image_tiff_filter(const BYTE const *b, DWORD size) { - return size > 2 && *(WORD*)buf == 0x4d4d; + return size > 2 && b[0] == 0x4d && b[1] == 0x4d; } -static BOOL image_xpng_filter(LPVOID buf, DWORD size) +static BOOL image_xpng_filter(const BYTE const *b, DWORD size) { static const BYTE xpng_header[] = {0x89,'P','N','G',0x0d,0x0a,0x1a,0x0a}; - return size > sizeof(xpng_header) && !memcmp(buf, xpng_header, sizeof(xpng_header)); + return size > sizeof(xpng_header) && !memcmp(b, xpng_header, sizeof(xpng_header)); } -static BOOL image_bmp_filter(LPVOID buf, DWORD size) +static BOOL image_bmp_filter(const BYTE const *b, DWORD size) { return size >= 14 - && *(BYTE*)buf == 0x42 - && *((BYTE*)buf+1) == 0x4d - && *(DWORD*)((BYTE*)buf+6) == 0; + && b[0] == 0x42 && b[1] == 0x4d + && *(DWORD*)(b+6) == 0; } -static BOOL video_avi_filter(LPVOID buf, DWORD size) +static BOOL video_avi_filter(const BYTE const *b, DWORD size) { - const BYTE const *b = buf; - return size > 12 && b[0] == 'R' && b[1] == 'I' && b[2] == 'F' && b[3] == 'F' && b[8] == 'A' && b[9] == 'V' && b[10] == 'I' && b[11] == 0x20; } -static BOOL text_plain_filter(LPVOID buf, DWORD size) +static BOOL text_plain_filter(const BYTE const *b, DWORD size) { - UCHAR *ptr; + const BYTE *ptr; - for(ptr = buf; ptr < (UCHAR*)buf+size-1; ptr++) { + for(ptr = b; ptr < b+size-1; ptr++) { if(*ptr < 0x20 && *ptr != '\n' && *ptr != '\r' && *ptr != '\t') return FALSE; } @@ -480,7 +474,7 @@ static BOOL text_plain_filter(LPVOID buf return TRUE; } -static BOOL application_octet_stream_filter(LPVOID buf, DWORD size) +static BOOL application_octet_stream_filter(const BYTE const *b, DWORD size) { return TRUE; } @@ -515,6 +509,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC } if(pBuffer) { + const BYTE const *buf = pBuffer; DWORD len; LPCWSTR ret = NULL; int i; @@ -532,7 +527,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC static const struct { LPCWSTR mime; - BOOL (*filter)(LPVOID,DWORD); + BOOL (*filter)(const BYTE const*,DWORD); } mime_filters[] = { {wszTextHtml, text_html_filter}, {wszImageGif, image_gif_filter}, @@ -555,7 +550,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC } if(i == sizeof(mime_filters)/sizeof(*mime_filters) - || mime_filters[i].filter(pBuffer, cbSize)) { + || mime_filters[i].filter(buf, cbSize)) { len = strlenW(pwzMimeProposed)+1; *ppwzMimeOut = CoTaskMemAlloc(len*sizeof(WCHAR)); memcpy(*ppwzMimeOut, pwzMimeProposed, len*sizeof(WCHAR)); @@ -565,11 +560,16 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC i=0; while(!ret) { - if(mime_filters[i].filter(pBuffer, cbSize)) + if(mime_filters[i].filter(buf, cbSize)) ret = mime_filters[i].mime; i++; } + TRACE("found %s for data\n" + "%02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x\n", + debugstr_w(ret), buf[0],buf[1],buf[2],buf[3], buf[4],buf[5],buf[6],buf[7], + buf[8],buf[9],buf[10],buf[11], buf[12],buf[13],buf[14],buf[15]); + if(pwzMimeProposed) { if(i == sizeof(mime_filters)/sizeof(*mime_filters)) ret = pwzMimeProposed;

1 0

Jacek Caban : urlmon: Added video/avi mime filter implementation.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 84253b011d77a243123000e99bcfe36d9be4308a URL: http://source.winehq.org/git/wine.git/?a=commit;h=84253b011d77a243123000e99… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Oct 23 23:38:31 2006 +0200 urlmon: Added video/avi mime filter implementation. --- dlls/urlmon/tests/misc.c | 15 ++++++++++++++- dlls/urlmon/urlmon_main.c | 13 ++++++++++++- 2 files changed, 26 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/tests/misc.c b/dlls/urlmon/tests/misc.c index 17657e7..5cfe701 100644 --- a/dlls/urlmon/tests/misc.c +++ b/dlls/urlmon/tests/misc.c @@ -336,6 +336,7 @@ static const WCHAR mimeImageGif[] = {'i' static const WCHAR mimeImageBmp[] = {'i','m','a','g','e','/','b','m','p',0}; static const WCHAR mimeImageXPng[] = {'i','m','a','g','e','/','x','-','p','n','g',0}; static const WCHAR mimeImageTiff[] = {'i','m','a','g','e','/','t','i','f','f',0}; +static const WCHAR mimeVideoAvi[] = {'v','i','d','e','o','/','a','v','i',0}; static const struct { LPCWSTR url; @@ -393,6 +394,12 @@ static BYTE data40[] = {0x4d,0x4d,0x00,0 static BYTE data41[] = {0x4d,0x4d,0xff}; static BYTE data42[] = {0x4d,0x4d}; static BYTE data43[] = {0x00,0x4d,0x4d,0x00}; +static BYTE data44[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x20,0xff}; +static BYTE data45[] = {'R','I','F','f',0xff,0xff,0xff,0xff,'A','V','I',0x20,0xff}; +static BYTE data46[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x20}; +static BYTE data47[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x21,0xff}; +static BYTE data48[] = {'R','I','F','F',0xff,0xff,0xff,0xff,'A','V','I',0x20,'<','h','t','m','l','>'}; +static BYTE data49[] = {'R','I','F','F',0x0f,0x0f,0xf0,0xf0,'A','V','I',0x20,0xf0,0x00}; static const struct { BYTE *data; @@ -441,7 +448,13 @@ static const struct { {data40, sizeof(data40), mimeTextHtml}, {data41, sizeof(data41), mimeImageTiff}, {data42, sizeof(data42), mimeTextPlain}, - {data43, sizeof(data43), mimeAppOctetStream} + {data43, sizeof(data43), mimeAppOctetStream}, + {data44, sizeof(data44), mimeVideoAvi}, + {data45, sizeof(data45), mimeTextPlain}, + {data46, sizeof(data46), mimeTextPlain}, + {data47, sizeof(data47), mimeTextPlain}, + {data48, sizeof(data48), mimeTextHtml}, + {data49, sizeof(data49), mimeVideoAvi} }; static void test_FindMimeFromData(void) diff --git a/dlls/urlmon/urlmon_main.c b/dlls/urlmon/urlmon_main.c index 044c2f8..916747b 100644 --- a/dlls/urlmon/urlmon_main.c +++ b/dlls/urlmon/urlmon_main.c @@ -459,6 +459,15 @@ static BOOL image_bmp_filter(LPVOID buf, && *(DWORD*)((BYTE*)buf+6) == 0; } +static BOOL video_avi_filter(LPVOID buf, DWORD size) +{ + const BYTE const *b = buf; + + return size > 12 + && b[0] == 'R' && b[1] == 'I' && b[2] == 'F' && b[3] == 'F' + && b[8] == 'A' && b[9] == 'V' && b[10] == 'I' && b[11] == 0x20; +} + static BOOL text_plain_filter(LPVOID buf, DWORD size) { UCHAR *ptr; @@ -516,6 +525,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC static const WCHAR wszImageTiff[] = {'i','m','a','g','e','/','t','i','f','f',0}; static const WCHAR wszImageXPng[] = {'i','m','a','g','e','/','x','-','p','n','g',0}; static const WCHAR wszImageBmp[] = {'i','m','a','g','e','/','b','m','p',0}; + static const WCHAR wszVideoAvi[] = {'v','i','d','e','o','/','a','v','i',0}; static const WCHAR wszTextPlain[] = {'t','e','x','t','/','p','l','a','i','n','\0'}; static const WCHAR wszAppOctetStream[] = {'a','p','p','l','i','c','a','t','i','o','n','/', 'o','c','t','e','t','-','s','t','r','e','a','m','\0'}; @@ -530,6 +540,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC {wszImageTiff, image_tiff_filter}, {wszImageXPng, image_xpng_filter}, {wszImageBmp, image_bmp_filter}, + {wszVideoAvi, video_avi_filter}, {wszTextPlain, text_plain_filter}, {wszAppOctetStream, application_octet_stream_filter} }; @@ -564,7 +575,7 @@ HRESULT WINAPI FindMimeFromData(LPBC pBC ret = pwzMimeProposed; /* text/html is a special case */ - if(!strcmpW(pwzMimeProposed, wszTextHtml) && !strcmpW(ret, wszTextPlain)) + if(!strcmpW(pwzMimeProposed, wszTextHtml) && !strcmpW(ret, wszTextPlain)) ret = wszTextHtml; }

1 0

Andrew Talbot : user: Remove superfluous shadow variable.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: 83b836b3f9d684d9f9b4bb33395d66a2c900b02d URL: http://source.winehq.org/git/wine.git/?a=commit;h=83b836b3f9d684d9f9b4bb333… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Mon Oct 23 22:53:41 2006 +0100 user: Remove superfluous shadow variable. --- dlls/user/dde_client.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/user/dde_client.c b/dlls/user/dde_client.c index 977e428..ecd3b3b 100644 --- a/dlls/user/dde_client.c +++ b/dlls/user/dde_client.c @@ -1035,7 +1035,6 @@ static HDDEDATA WDML_SyncWaitTransaction while (PeekMessageW(&msg, 0, WM_DDE_FIRST, WM_DDE_LAST, PM_REMOVE)) { - WDML_CONV *pConv; HDDEDATA hdd; EnterCriticalSection(&WDML_CritSect);

1 0

Andrew Talbot : crypt32: Cast-qual warnings fix.
by Alexandre Julliard 24 Oct '06

24 Oct '06

Module: wine Branch: master Commit: c70841fb36bd3373e8a6518bdeb7e96f1d905883 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c70841fb36bd3373e8a6518bd… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Mon Oct 23 21:15:58 2006 +0100 crypt32: Cast-qual warnings fix. --- dlls/crypt32/cert.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/crypt32/cert.c b/dlls/crypt32/cert.c index fd39ef3..b39c3ca 100644 --- a/dlls/crypt32/cert.c +++ b/dlls/crypt32/cert.c @@ -364,7 +364,7 @@ BOOL WINAPI CertGetCertificateContextPro * - rgProvParam[0]... */ static void CRYPT_CopyKeyProvInfo(PCRYPT_KEY_PROV_INFO to, - PCRYPT_KEY_PROV_INFO from) + const CRYPT_KEY_PROV_INFO *from) { DWORD i; LPBYTE nextData = (LPBYTE)to + sizeof(CRYPT_KEY_PROV_INFO); @@ -403,7 +403,7 @@ static void CRYPT_CopyKeyProvInfo(PCRYPT } static BOOL CertContext_SetKeyProvInfoProperty(PCONTEXT_PROPERTY_LIST properties, - PCRYPT_KEY_PROV_INFO info) + const CRYPT_KEY_PROV_INFO *info) { BOOL ret; LPBYTE buf = NULL; @@ -468,7 +468,7 @@ static BOOL WINAPI CertContext_SetProper { if (pvData) { - PCRYPT_DATA_BLOB blob = (PCRYPT_DATA_BLOB)pvData; + const CRYPT_DATA_BLOB *blob = (const CRYPT_DATA_BLOB *)pvData; ret = ContextPropertyList_SetProperty(properties, dwPropId, blob->pbData, blob->cbData); @@ -483,7 +483,7 @@ static BOOL WINAPI CertContext_SetProper case CERT_DATE_STAMP_PROP_ID: if (pvData) ret = ContextPropertyList_SetProperty(properties, dwPropId, - (LPBYTE)pvData, sizeof(FILETIME)); + (const BYTE *)pvData, sizeof(FILETIME)); else { ContextPropertyList_RemoveProperty(properties, dwPropId); @@ -494,7 +494,7 @@ static BOOL WINAPI CertContext_SetProper { if (pvData) { - PCERT_KEY_CONTEXT keyContext = (PCERT_KEY_CONTEXT)pvData; + const CERT_KEY_CONTEXT *keyContext = (const CERT_KEY_CONTEXT *)pvData; ret = ContextPropertyList_SetProperty(properties, dwPropId, (const BYTE *)keyContext, keyContext->cbSize); @@ -509,7 +509,7 @@ static BOOL WINAPI CertContext_SetProper case CERT_KEY_PROV_INFO_PROP_ID: if (pvData) ret = CertContext_SetKeyProvInfoProperty(properties, - (PCRYPT_KEY_PROV_INFO)pvData); + (const CRYPT_KEY_PROV_INFO *)pvData); else { ContextPropertyList_RemoveProperty(properties, dwPropId); @@ -528,7 +528,7 @@ static BOOL WINAPI CertContext_SetProper if (!(dwFlags & CERT_STORE_NO_CRYPT_RELEASE_FLAG)) CryptReleaseContext(keyContext.hCryptProv, 0); if (pvData) - keyContext.hCryptProv = *(HCRYPTPROV *)pvData; + keyContext.hCryptProv = *(const HCRYPTPROV *)pvData; else keyContext.hCryptProv = 0; ret = CertContext_SetProperty(context, CERT_KEY_CONTEXT_PROP_ID,

1 0

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 October 2006