December 2006 - Wine-commits

Detlef Riekenberg : localspl/tests: Fix copy+paste typo.
by Alexandre Julliard 22 Dec '06

22 Dec '06

Module: wine Branch: master Commit: 99c716fee456d1fe95c6f9f809793355787726cd URL: http://source.winehq.org/git/wine.git/?a=commit;h=99c716fee456d1fe95c6f9f80… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Fri Dec 22 00:47:54 2006 +0100 localspl/tests: Fix copy+paste typo. --- dlls/localspl/tests/localmon.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/localspl/tests/localmon.c b/dlls/localspl/tests/localmon.c index 93b595c..4ca6644 100644 --- a/dlls/localspl/tests/localmon.c +++ b/dlls/localspl/tests/localmon.c @@ -223,7 +223,7 @@ static void test_EnumPorts(void) SetLastError(0xdeadbeef); res = pEnumPorts(NULL, level, buffer, cbBuf, &pcbNeeded, &pcReturned); ok( res, "(%d) returned %d with %d and %d, %d (expected '!= 0')\n", - level, res, GetLastError(), cbBuf, pcReturned); + level, res, GetLastError(), pcbNeeded, pcReturned); /* We can compare the returned Data with the Registry / "win.ini",[Ports] here */ pcbNeeded = 0xdeadbeef; @@ -231,7 +231,7 @@ static void test_EnumPorts(void) SetLastError(0xdeadbeef); res = pEnumPorts(NULL, level, buffer, cbBuf+1, &pcbNeeded, &pcReturned); ok( res, "(%d) returned %d with %d and %d, %d (expected '!= 0')\n", - level, res, GetLastError(), cbBuf, pcReturned); + level, res, GetLastError(), pcbNeeded, pcReturned); pcbNeeded = 0xdeadbeef; pcReturned = 0xdeadbeef; @@ -240,7 +240,7 @@ static void test_EnumPorts(void) ok( !res && (GetLastError() == ERROR_INSUFFICIENT_BUFFER), "(%d) returned %d with %d and %d, %d (expected '0' with " \ "ERROR_INSUFFICIENT_BUFFER)\n", - level, res, GetLastError(), cbBuf, pcReturned); + level, res, GetLastError(), pcbNeeded, pcReturned); if (0) { @@ -256,14 +256,14 @@ static void test_EnumPorts(void) SetLastError(0xdeadbeef); res = pEnumPorts(emptyW, level, buffer, cbBuf+1, &pcbNeeded, &pcReturned); ok( res, "(%d) returned %d with %d and %d, %d (expected '!= 0')\n", - level, res, GetLastError(), cbBuf, pcReturned); + level, res, GetLastError(), pcbNeeded, pcReturned); pcbNeeded = 0xdeadbeef; pcReturned = 0xdeadbeef; SetLastError(0xdeadbeef); res = pEnumPorts(invalid_serverW, level, buffer, cbBuf+1, &pcbNeeded, &pcReturned); ok( res, "(%d) returned %d with %d and %d, %d (expected '!= 0')\n", - level, res, GetLastError(), cbBuf, pcReturned); + level, res, GetLastError(), pcbNeeded, pcReturned); HeapFree(GetProcessHeap(), 0, buffer); }

1 0

Detlef Riekenberg : localspl/tests: Rename unclear string.
by Alexandre Julliard 22 Dec '06

22 Dec '06

Module: wine Branch: master Commit: 9721edf5c1ddee445ed153dcdb7faba747a85389 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9721edf5c1ddee445ed153dcd… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Fri Dec 22 00:47:57 2006 +0100 localspl/tests: Rename unclear string. --- dlls/localspl/tests/localmon.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/localspl/tests/localmon.c b/dlls/localspl/tests/localmon.c index 4ca6644..5aecda6 100644 --- a/dlls/localspl/tests/localmon.c +++ b/dlls/localspl/tests/localmon.c @@ -62,7 +62,6 @@ static WCHAR cmd_MonitorUIW[] = {'M','o' static WCHAR cmd_MonitorUI_lcaseW[] = {'m','o','n','i','t','o','r','u','i',0}; static WCHAR does_not_existW[] = {'d','o','e','s','_','n','o','t','_','e','x','i','s','t',0}; static WCHAR emptyW[] = {0}; -static WCHAR invalid_serverW[] = {'\\','\\','i','n','v','a','l','i','d','_','s','e','r','v','e','r',0}; static WCHAR Monitors_LocalPortW[] = { \ 'S','y','s','t','e','m','\\', 'C','u','r','r','e','n','t','C','o','n','t','r','o','l','S','e','t','\\', @@ -75,6 +74,8 @@ static WCHAR portname_com1W[] = {'C','O' static WCHAR portname_fileW[] = {'F','I','L','E',':',0}; static WCHAR portname_lpt1W[] = {'L','P','T','1',':',0}; +static WCHAR server_does_not_existW[] = {'\\','\\','d','o','e','s','_','n','o','t','_','e','x','i','s','t',0}; + /* ########################### */ static void test_AddPort(void) @@ -261,7 +262,7 @@ static void test_EnumPorts(void) pcbNeeded = 0xdeadbeef; pcReturned = 0xdeadbeef; SetLastError(0xdeadbeef); - res = pEnumPorts(invalid_serverW, level, buffer, cbBuf+1, &pcbNeeded, &pcReturned); + res = pEnumPorts(server_does_not_existW, level, buffer, cbBuf+1, &pcbNeeded, &pcReturned); ok( res, "(%d) returned %d with %d and %d, %d (expected '!= 0')\n", level, res, GetLastError(), pcbNeeded, pcReturned);

1 0

Francois Gouget : setupapi: Fix handling of 'CopyFiles=@file'.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 8cfd0251646862dd1415c9ac028958c044b38eb1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8cfd0251646862dd1415c9ac0… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Dec 21 14:48:15 2006 +0100 setupapi: Fix handling of 'CopyFiles=@file'. --- dlls/setupapi/install.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/setupapi/install.c b/dlls/setupapi/install.c index 4c60b86..46f1b94 100644 --- a/dlls/setupapi/install.c +++ b/dlls/setupapi/install.c @@ -113,7 +113,7 @@ static BOOL copy_files_callback( HINF hi struct files_callback_info *info = arg; if (field[0] == '@') /* special case: copy single file */ - SetupQueueDefaultCopyW( info->queue, info->layout, info->src_root, NULL, field, info->copy_flags ); + SetupQueueDefaultCopyW( info->queue, info->layout, info->src_root, NULL, field+1, info->copy_flags ); else SetupQueueCopySectionW( info->queue, info->src_root, info->layout, hinf, field, info->copy_flags ); return TRUE;

1 0

Huw Davies : gdi32: Extract the constituent TrueType files from Mac suitcase fonts into the user 's caches directory
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 4e6860433e72714ed1e500eb523710656e71ba02 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e6860433e72714ed1e500eb5… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Dec 21 15:21:13 2006 +0000 gdi32: Extract the constituent TrueType files from Mac suitcase fonts into the user's caches directory and use these instead of the original suitcase. --- configure | 6 +- configure.ac | 1 + dlls/gdi32/Makefile.in | 2 +- dlls/gdi32/freetype.c | 257 ++++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 264 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 6779121..3e9b56e 100755 --- a/configure +++ b/configure @@ -759,6 +759,7 @@ IOKITLIB LDEXECFLAGS DISKARBITRATIONLIB COREAUDIO +CARBONLIB CROSSTEST CROSSCC CROSSWINDRES @@ -13462,6 +13463,8 @@ fi esac if test "$ac_cv_header_Carbon_Carbon_h" = "yes" then + CARBONLIB="-framework Carbon" + QUARTZFILES='$(QUARTZFILES)' fi ;; @@ -21426,6 +21429,7 @@ IOKITLIB!$IOKITLIB$ac_delim LDEXECFLAGS!$LDEXECFLAGS$ac_delim DISKARBITRATIONLIB!$DISKARBITRATIONLIB$ac_delim COREAUDIO!$COREAUDIO$ac_delim +CARBONLIB!$CARBONLIB$ac_delim CROSSTEST!$CROSSTEST$ac_delim CROSSCC!$CROSSCC$ac_delim CROSSWINDRES!$CROSSWINDRES$ac_delim @@ -21442,7 +21446,7 @@ LIBOBJS!$LIBOBJS$ac_delim LTLIBOBJS!$LTLIBOBJS$ac_delim _ACEOF - if test `sed -n "s/.*$ac_delim\$/X/p" conf$$subs.sed | grep -c X` = 76; then + if test `sed -n "s/.*$ac_delim\$/X/p" conf$$subs.sed | grep -c X` = 77; then break elif $ac_last_try; then { { echo "$as_me:$LINENO: error: could not make $CONFIG_STATUS" >&5 diff --git a/configure.ac b/configure.ac index efbf32d..9fc757b 100644 --- a/configure.ac +++ b/configure.ac @@ -1023,6 +1023,7 @@ case $host_os in dnl Enable quartz driver on Mac OS X if test "$ac_cv_header_Carbon_Carbon_h" = "yes" then + AC_SUBST(CARBONLIB,"-framework Carbon") QUARTZFILES='$(QUARTZFILES)' fi ;; diff --git a/dlls/gdi32/Makefile.in b/dlls/gdi32/Makefile.in index df6f89b..e3b2d67 100644 --- a/dlls/gdi32/Makefile.in +++ b/dlls/gdi32/Makefile.in @@ -7,7 +7,7 @@ MODULE = gdi32.dll IMPORTLIB = libgdi32.$(IMPLIBEXT) IMPORTS = advapi32 kernel32 ntdll EXTRAINCL = @FREETYPEINCL@ -EXTRALIBS = @ICULIBS@ +EXTRALIBS = @ICULIBS@ @CARBONLIB@ SPEC_SRCS16 = \ dispdib.spec \ diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 84daf4f..b9a46bc 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -37,6 +37,44 @@ #include <stdio.h> #include <assert.h> +#ifdef HAVE_CARBON_CARBON_H +#define LoadResource __carbon_LoadResource +#define CompareString __carbon_CompareString +#define GetCurrentThread __carbon_GetCurrentThread +#define GetCurrentProcess __carbon_GetCurrentProcess +#define AnimatePalette __carbon_AnimatePalette +#define EqualRgn __carbon_EqualRgn +#define FillRgn __carbon_FillRgn +#define FrameRgn __carbon_FrameRgn +#define GetPixel __carbon_GetPixel +#define InvertRgn __carbon_InvertRgn +#define LineTo __carbon_LineTo +#define OffsetRgn __carbon_OffsetRgn +#define PaintRgn __carbon_PaintRgn +#define Polygon __carbon_Polygon +#define ResizePalette __carbon_ResizePalette +#define SetRectRgn __carbon_SetRectRgn +#include <Carbon/Carbon.h> +#undef LoadResource +#undef CompareString +#undef GetCurrentThread +#undef _CDECL +#undef DPRINTF +#undef GetCurrentProcess +#undef AnimatePalette +#undef EqualRgn +#undef FillRgn +#undef FrameRgn +#undef GetPixel +#undef InvertRgn +#undef LineTo +#undef OffsetRgn +#undef PaintRgn +#undef Polygon +#undef ResizePalette +#undef SetRectRgn +#endif /* HAVE_CARBON_CARBON_H */ + #include "windef.h" #include "winbase.h" #include "winternl.h" @@ -438,6 +476,204 @@ static const WCHAR font_mutex_nameW[] = * cga40woa.fon=cga40850.fon */ +#ifdef HAVE_CARBON_CARBON_H +static char *find_cache_dir(void) +{ + FSRef ref; + OSErr err; + static char cached_path[MAX_PATH]; + static const char *wine = "/Wine", *fonts = "/Fonts"; + + if(*cached_path) return cached_path; + + err = FSFindFolder(kUserDomain, kCachedDataFolderType, kCreateFolder, &ref); + if(err != noErr) + { + WARN("can't create cached data folder\n"); + return NULL; + } + err = FSRefMakePath(&ref, (unsigned char*)cached_path, sizeof(cached_path)); + if(err != noErr) + { + WARN("can't create cached data path\n"); + *cached_path = '\0'; + return NULL; + } + if(strlen(cached_path) + strlen(wine) + strlen(fonts) + 1 > sizeof(cached_path)) + { + ERR("Could not create full path\n"); + *cached_path = '\0'; + return NULL; + } + strcat(cached_path, wine); + + if(mkdir(cached_path, 0700) == -1 && errno != EEXIST) + { + WARN("Couldn't mkdir %s\n", cached_path); + *cached_path = '\0'; + return NULL; + } + strcat(cached_path, fonts); + if(mkdir(cached_path, 0700) == -1 && errno != EEXIST) + { + WARN("Couldn't mkdir %s\n", cached_path); + *cached_path = '\0'; + return NULL; + } + return cached_path; +} + +/****************************************************************** + * expand_mac_font + * + * Extracts individual TrueType font files from a Mac suitcase font + * and saves them into the user's caches directory (see + * find_cache_dir()). + * Returns a NULL terminated array of filenames. + * + * We do this because they are apps that try to read ttf files + * themselves and they don't like Mac suitcase files. + */ +static char **expand_mac_font(const char *path) +{ + FSRef ref; + SInt16 res_ref; + OSStatus s; + unsigned int idx; + const char *out_dir; + const char *filename; + int output_len; + struct { + char **array; + unsigned int size, max_size; + } ret; + + TRACE("path %s\n", path); + + s = FSPathMakeRef((unsigned char*)path, &ref, FALSE); + if(s != noErr) + { + WARN("failed to get ref\n"); + return NULL; + } + + s = FSOpenResourceFile(&ref, 0, NULL, fsRdPerm, &res_ref); + if(s != noErr) + { + TRACE("no data fork, so trying resource fork\n"); + res_ref = FSOpenResFile(&ref, fsRdPerm); + if(res_ref == -1) + { + TRACE("unable to open resource fork\n"); + return NULL; + } + } + + ret.size = 0; + ret.max_size = 10; + ret.array = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, ret.max_size * sizeof(*ret.array)); + if(!ret.array) + { + CloseResFile(res_ref); + return NULL; + } + + out_dir = find_cache_dir(); + + filename = strrchr(path, '/'); + if(!filename) filename = path; + else filename++; + + /* output filename has the form out_dir/filename_%04x.ttf */ + output_len = strlen(out_dir) + 1 + strlen(filename) + 5 + 5; + + UseResFile(res_ref); + idx = 1; + while(1) + { + FamRec *fam_rec; + unsigned short *num_faces_ptr, num_faces, face; + AsscEntry *assoc; + Handle fond; + + fond = Get1IndResource('FOND', idx); + if(!fond) break; + TRACE("got fond resource %d\n", idx); + HLock(fond); + + fam_rec = *(FamRec**)fond; + num_faces_ptr = (unsigned short *)(fam_rec + 1); + num_faces = GET_BE_WORD(*num_faces_ptr); + num_faces++; + assoc = (AsscEntry*)(num_faces_ptr + 1); + TRACE("num faces %04x\n", num_faces); + for(face = 0; face < num_faces; face++, assoc++) + { + Handle sfnt; + unsigned short size, font_id; + char *output; + + size = GET_BE_WORD(assoc->fontSize); + font_id = GET_BE_WORD(assoc->fontID); + if(size != 0) + { + TRACE("skipping id %04x because it's not scalable (fixed size %d)\n", font_id, size); + continue; + } + + TRACE("trying to load sfnt id %04x\n", font_id); + sfnt = GetResource('sfnt', font_id); + if(!sfnt) + { + TRACE("can't get sfnt resource %04x\n", font_id); + continue; + } + + output = HeapAlloc(GetProcessHeap(), 0, output_len); + if(output) + { + int fd; + + sprintf(output, "%s/%s_%04x.ttf", out_dir, filename, font_id); + + fd = open(output, O_CREAT | O_EXCL | O_WRONLY, 0600); + if(fd != -1 || errno == EEXIST) + { + if(fd != -1) + { + unsigned char *sfnt_data; + + HLock(sfnt); + sfnt_data = *(unsigned char**)sfnt; + write(fd, sfnt_data, GetHandleSize(sfnt)); + HUnlock(sfnt); + close(fd); + } + if(ret.size >= ret.max_size - 1) /* Always want the last element to be NULL */ + { + ret.max_size *= 2; + ret.array = HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, ret.array, ret.max_size * sizeof(*ret.array)); + } + ret.array[ret.size++] = output; + } + else + { + WARN("unable to create %s\n", output); + HeapFree(GetProcessHeap(), 0, output); + } + } + ReleaseResource(sfnt); + } + HUnlock(fond); + ReleaseResource(fond); + idx++; + } + CloseResFile(res_ref); + + return ret.array; +} + +#endif /* HAVE_CARBON_CARBON_H */ static inline BOOL is_win9x(void) { @@ -722,6 +958,27 @@ static BOOL AddFontFileToList(const char int i, bitmap_num, internal_leading; FONTSIGNATURE fs; +#ifdef HAVE_CARBON_CARBON_H + if(!fake_family) + { + char **mac_list = expand_mac_font(file); + if(mac_list) + { + BOOL had_one = FALSE; + char **cursor; + for(cursor = mac_list; *cursor; cursor++) + { + had_one = TRUE; + AddFontFileToList(*cursor, NULL, flags); + HeapFree(GetProcessHeap(), 0, *cursor); + } + HeapFree(GetProcessHeap(), 0, mac_list); + if(had_one) + return TRUE; + } + } +#endif /* HAVE_CARBON_CARBON_H */ + do { char *family_name = fake_family;

1 0

Alexandre Julliard : ntdll: Added support for physically ejecting devices on Mac OS.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 9ff161860c51d62f7f05989116cd73908f5611e9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ff161860c51d62f7f0598911… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 21 14:14:20 2006 +0100 ntdll: Added support for physically ejecting devices on Mac OS. --- dlls/ntdll/cdrom.c | 99 +++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 98 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/cdrom.c b/dlls/ntdll/cdrom.c index 65712a7..45a2b47 100644 --- a/dlls/ntdll/cdrom.c +++ b/dlls/ntdll/cdrom.c @@ -77,7 +77,9 @@ #ifdef HAVE_IOKIT_IOKITLIB_H # ifndef SENSEBUFLEN +# include <sys/disk.h> # include <IOKit/IOKitLib.h> +# include <IOKit/storage/IOMedia.h> # include <IOKit/scsi/SCSICmds_REQUEST_SENSE_Defs.h> # define SENSEBUFLEN kSenseDefaultSize # endif @@ -302,6 +304,82 @@ static int CDROM_MediaChanged(int dev) } #endif + +/****************************************************************** + * open_parent_device + * + * On Mac OS, open the device for the whole disk from a fd that points to a partition. + * This is ugly and inefficient, but we have no choice since the partition fd doesn't + * support the eject ioctl. + */ +#ifdef __APPLE__ +static int open_parent_device( int fd ) +{ + struct stat st; + int i, parent_fd = -1; + io_service_t service; + CFMutableDictionaryRef dict; + CFTypeRef val; + + if (fstat( fd, &st ) == -1) return -1; + if (!S_ISCHR( st.st_mode )) return -1; + + /* create a dictionary with the right major/minor numbers */ + + if (!(dict = IOServiceMatching( kIOMediaClass ))) return -1; + + i = major( st.st_rdev ); + val = CFNumberCreate( NULL, kCFNumberIntType, &i ); + CFDictionaryAddValue( dict, CFSTR( "BSD Major" ), val ); + CFRelease( val ); + + i = minor( st.st_rdev ); + val = CFNumberCreate( NULL, kCFNumberIntType, &i ); + CFDictionaryAddValue( dict, CFSTR( "BSD Minor" ), val ); + CFRelease( val ); + + CFDictionaryAddValue( dict, CFSTR("Removable"), kCFBooleanTrue ); + + service = IOServiceGetMatchingService( kIOMasterPortDefault, dict ); + + /* now look for the parent that has the "Whole" attribute set to TRUE */ + + while (service) + { + io_service_t parent = 0; + CFBooleanRef whole; + CFStringRef str; + int ok; + + if (!IOObjectConformsTo( service, kIOMediaClass )) + goto next; + if (!(whole = IORegistryEntryCreateCFProperty( service, CFSTR("Whole"), NULL, 0 ))) + goto next; + ok = (whole == kCFBooleanTrue); + CFRelease( whole ); + if (!ok) goto next; + + if ((str = IORegistryEntryCreateCFProperty( service, CFSTR("BSD Name"), NULL, 0 ))) + { + char name[100]; + strcpy( name, "/dev/r" ); + CFStringGetCString( str, name + 6, sizeof(name) - 6, kCFStringEncodingUTF8 ); + CFRelease( str ); + parent_fd = open( name, O_RDONLY ); + } + IOObjectRelease( service ); + break; + +next: + IORegistryEntryGetParentEntry( service, kIOServicePlane, &parent ); + IOObjectRelease( service ); + service = parent; + } + return parent_fd; +} +#endif + + /****************************************************************** * CDROM_SyncCache [internal] * @@ -628,6 +706,9 @@ static NTSTATUS CDROM_SetTray(int fd, BO return CDROM_GetStatusCode((ioctl(fd, CDIOCALLOW, NULL)) || (ioctl(fd, doEject ? CDIOCEJECT : CDIOCCLOSE, NULL)) || (ioctl(fd, CDIOCPREVENT, NULL))); +#elif defined(__APPLE__) + if (doEject) return CDROM_GetStatusCode( ioctl( fd, DKIOCEJECT, NULL ) ); + else return STATUS_NOT_SUPPORTED; #else return STATUS_NOT_SUPPORTED; #endif @@ -2059,7 +2140,23 @@ NTSTATUS CDROM_DeviceIoControl(HANDLE hD CDROM_ClearCacheEntry(dev); if (lpInBuffer != NULL || nInBufferSize != 0 || lpOutBuffer != NULL || nOutBufferSize != 0) status = STATUS_INVALID_PARAMETER; - else status = CDROM_SetTray(fd, TRUE); + else + { +#ifdef __APPLE__ + int parent_fd = open_parent_device( fd ); + if (parent_fd != -1) + { + /* This is ugly as hell, but Mac OS is unable to eject from the device fd, + * it wants an fd for the whole device, and it also requires the device fd + * to be closed first, so we have to close the handle that the caller gave us */ + NtClose( hDevice ); + if (needs_close) close( fd ); + fd = parent_fd; + needs_close = 1; + } +#endif + status = CDROM_SetTray(fd, TRUE); + } break; case IOCTL_CDROM_MEDIA_REMOVAL:

1 0

Ken Thomases : iphlpapi: Fixed a problem with enumerating network interfaces.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 6b033cc1e61a1ac50753b52c1b267f34a3a7ee54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6b033cc1e61a1ac50753b52c1… Author: Ken Thomases <ken(a)codeweavers.com> Date: Thu Dec 21 14:47:20 2006 +0100 iphlpapi: Fixed a problem with enumerating network interfaces. Keep increasing the buffer until the returned length is more than two records smaller than the buffer we provided. --- dlls/iphlpapi/ifenum.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/iphlpapi/ifenum.c b/dlls/iphlpapi/ifenum.c index 52e8413..fb3348f 100644 --- a/dlls/iphlpapi/ifenum.c +++ b/dlls/iphlpapi/ifenum.c @@ -690,7 +690,7 @@ static DWORD enumIPAddresses(PDWORD pcAd ifc->ifc_buf = HeapAlloc(GetProcessHeap(), 0, ifc->ifc_len); ioctlRet = ioctl(fd, SIOCGIFCONF, ifc); } while (ioctlRet == 0 && - ifc->ifc_len == (sizeof(struct ifreq) * guessedNumAddresses)); + ifc->ifc_len > (sizeof(struct ifreq) * (guessedNumAddresses - 2))); if (ioctlRet == 0) { ifPtr = ifc->ifc_buf;

1 0

Paul Vriens : kernel32/tests: Don't use full path for executable in tests.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 11c186fbe709573b16ca45c7701512e9c0fc31f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11c186fbe709573b16ca45c77… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Wed Dec 20 18:28:11 2006 +0100 kernel32/tests: Don't use full path for executable in tests. --- dlls/kernel32/tests/process.c | 25 +++++++++++++++++++------ 1 files changed, 19 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 0755761..52251a6 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -37,6 +37,7 @@ static LPVOID (WINAPI *pVirtualFreeEx)(H static char base[MAX_PATH]; static char selfname[MAX_PATH]; +static char* exename; static char resfile[MAX_PATH]; static int myARGC; @@ -151,14 +152,23 @@ static WCHAR* decodeW(const char* str) * generates basic information like: * base: absolute path to curr dir * selfname: the way to reinvoke ourselves + * exename: executable without the path * function-pointers, which are not implemented in all windows versions */ static int init(void) { + char *p; + myARGC = winetest_get_mainargs( &myARGV ); if (!GetCurrentDirectoryA(sizeof(base), base)) return 0; strcpy(selfname, myARGV[0]); + /* Strip the path of selfname */ + if ((p = strrchr(selfname, '\\')) != NULL) exename = p + 1; + else exename = selfname; + + if ((p = strrchr(exename, '/')) != NULL) exename = p + 1; + hkernel32 = GetModuleHandleA("kernel32"); pVirtualAllocEx = (void *) GetProcAddress(hkernel32, "VirtualAllocEx"); pVirtualFreeEx = (void *) GetProcAddress(hkernel32, "VirtualFreeEx"); @@ -758,7 +768,8 @@ static void test_CommandLine(void) /* Test for Bug1330 to show that XP doesn't change '/' to '\\' in argv[0]*/ get_file_name(resfile); - sprintf(buffer, "./%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", selfname, resfile); + /* Use exename to avoid buffer containing things like 'C:' */ + sprintf(buffer, "./%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", exename, resfile); SetLastError(0xdeadbeef); ret = CreateProcessA(NULL, buffer, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); ok(ret, "CreateProcess (%s) failed : %d\n", buffer, GetLastError()); @@ -766,13 +777,14 @@ static void test_CommandLine(void) ok(WaitForSingleObject(info.hProcess, 30000) == WAIT_OBJECT_0, "Child process termination\n"); /* child process has changed result file, so let profile functions know about it */ WritePrivateProfileStringA(NULL, NULL, NULL, resfile); - sprintf(buffer, "./%s", selfname); + sprintf(buffer, "./%s", exename); okChildString("Arguments", "argvA0", buffer); release_memory(); assert(DeleteFileA(resfile) != 0); get_file_name(resfile); - sprintf(buffer, ".\\%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", selfname, resfile); + /* Use exename to avoid buffer containing things like 'C:' */ + sprintf(buffer, ".\\%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", exename, resfile); SetLastError(0xdeadbeef); ret = CreateProcessA(NULL, buffer, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); ok(ret, "CreateProcess (%s) failed : %d\n", buffer, GetLastError()); @@ -780,7 +792,7 @@ static void test_CommandLine(void) ok(WaitForSingleObject(info.hProcess, 30000) == WAIT_OBJECT_0, "Child process termination\n"); /* child process has changed result file, so let profile functions know about it */ WritePrivateProfileStringA(NULL, NULL, NULL, resfile); - sprintf(buffer, ".\\%s", selfname); + sprintf(buffer, ".\\%s", exename); okChildString("Arguments", "argvA0", buffer); release_memory(); assert(DeleteFileA(resfile) != 0); @@ -791,7 +803,8 @@ static void test_CommandLine(void) *(lpFilePart -1 ) = 0; p = strrchr(fullpath, '\\'); assert (p); - sprintf(buffer, "..%s/%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", p, selfname, resfile); + /* Use exename to avoid buffer containing things like 'C:' */ + sprintf(buffer, "..%s/%s tests/process.c %s \"a\\\"b\\\\\" c\\\" d", p, exename, resfile); SetLastError(0xdeadbeef); ret = CreateProcessA(NULL, buffer, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); ok(ret, "CreateProcess (%s) failed : %d\n", buffer, GetLastError()); @@ -799,7 +812,7 @@ static void test_CommandLine(void) ok(WaitForSingleObject(info.hProcess, 30000) == WAIT_OBJECT_0, "Child process termination\n"); /* child process has changed result file, so let profile functions know about it */ WritePrivateProfileStringA(NULL, NULL, NULL, resfile); - sprintf(buffer, "..%s/%s", p, selfname); + sprintf(buffer, "..%s/%s", p, exename); okChildString("Arguments", "argvA0", buffer); release_memory(); assert(DeleteFileA(resfile) != 0);

1 0

Alexandre Julliard : ntdll: File positions should be aligned to sector boundaries in PE loader.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 9fed1205cae7cdb7abacfd3c86523cb22db16ef8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9fed1205cae7cdb7abacfd3c8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 21 12:34:44 2006 +0100 ntdll: File positions should be aligned to sector boundaries in PE loader. --- dlls/kernel32/tests/loader.c | 4 ---- dlls/ntdll/virtual.c | 24 +++++++++++++----------- server/mapping.c | 42 +++++++++++++++++++++--------------------- 3 files changed, 34 insertions(+), 36 deletions(-) diff --git a/dlls/kernel32/tests/loader.c b/dlls/kernel32/tests/loader.c index 8811334..4f00e4d 100644 --- a/dlls/kernel32/tests/loader.c +++ b/dlls/kernel32/tests/loader.c @@ -404,9 +404,7 @@ START_TEST(loader) ok(info.Type == SEC_IMAGE, "%d: %x != SEC_IMAGE\n", i, info.Type); if (nt_header.OptionalHeader.SectionAlignment >= si.dwPageSize) -todo_wine { ok(!memcmp((const char *)hlib + section.VirtualAddress + section.PointerToRawData, &nt_header, section.SizeOfRawData), "wrong section data\n"); -} else ok(!memcmp((const char *)hlib + section.PointerToRawData, section_data, section.SizeOfRawData), "wrong section data\n"); @@ -418,9 +416,7 @@ todo_wine { start = (const char *)hlib + section.VirtualAddress + section.PointerToRawData + section.SizeOfRawData; size = ALIGN_SIZE((ULONG_PTR)start, si.dwPageSize) - (ULONG_PTR)start; -todo_wine { ok(memcmp(start, filler, size), "%d: alignment should not be not cleared\n", i); -} } } diff --git a/dlls/ntdll/virtual.c b/dlls/ntdll/virtual.c index 152af7b..fe33566 100644 --- a/dlls/ntdll/virtual.c +++ b/dlls/ntdll/virtual.c @@ -1051,18 +1051,18 @@ static NTSTATUS map_image( HANDLE hmappi for (i = pos = 0; i < nt->FileHeader.NumberOfSections; i++, sec++) { - SIZE_T map_size, file_size, end; + static const SIZE_T sector_align = 0x1ff; + SIZE_T map_size, file_start, file_size, end; if (!sec->Misc.VirtualSize) - { - file_size = sec->SizeOfRawData; - map_size = ROUND_SIZE( 0, file_size ); - } + map_size = ROUND_SIZE( 0, sec->SizeOfRawData ); else - { map_size = ROUND_SIZE( 0, sec->Misc.VirtualSize ); - file_size = min( sec->SizeOfRawData, map_size ); - } + + /* file positions are rounded to sector boundaries regardless of OptionalHeader.FileAlignment */ + file_start = sec->PointerToRawData & ~sector_align; + file_size = (sec->SizeOfRawData + (sec->PointerToRawData & sector_align) + sector_align) & ~sector_align; + if (file_size > map_size) file_size = map_size; /* a few sanity checks */ end = sec->VirtualAddress + ROUND_SIZE( sec->VirtualAddress, map_size ); @@ -1115,9 +1115,11 @@ static NTSTATUS map_image( HANDLE hmappi /* Note: if the section is not aligned properly map_file_into_view will magically * fall back to read(), so we don't need to check anything here. */ - end = sec->PointerToRawData + file_size; - if (sec->PointerToRawData >= st.st_size || end > st.st_size || end < sec->PointerToRawData || - map_file_into_view( view, fd, sec->VirtualAddress, file_size, sec->PointerToRawData, + end = file_start + file_size; + if (sec->PointerToRawData >= st.st_size || + end > ((st.st_size + sector_align) & ~sector_align) || + end < file_start || + map_file_into_view( view, fd, sec->VirtualAddress, file_size, file_start, VPROT_COMMITTED | VPROT_READ | VPROT_WRITECOPY, removable ) != STATUS_SUCCESS) { diff --git a/server/mapping.c b/server/mapping.c index 052ea41..c7eb075 100644 --- a/server/mapping.c +++ b/server/mapping.c @@ -119,25 +119,26 @@ static struct file *get_shared_file( str return NULL; } -/* return the size of the memory mapping of a given section */ -static inline unsigned int get_section_map_size( const IMAGE_SECTION_HEADER *sec ) +/* return the size of the memory mapping and file range of a given section */ +static inline void get_section_sizes( const IMAGE_SECTION_HEADER *sec, size_t *map_size, + off_t *file_start, size_t *file_size ) { - if (!sec->Misc.VirtualSize) return ROUND_SIZE( sec->SizeOfRawData ); - else return ROUND_SIZE( sec->Misc.VirtualSize ); -} + static const unsigned int sector_align = 0x1ff; -/* return the size of the file mapping of a given section */ -static inline unsigned int get_section_filemap_size( const IMAGE_SECTION_HEADER *sec ) -{ - if (!sec->Misc.VirtualSize) return sec->SizeOfRawData; - else return min( sec->SizeOfRawData, ROUND_SIZE( sec->Misc.VirtualSize ) ); + if (!sec->Misc.VirtualSize) *map_size = ROUND_SIZE( sec->SizeOfRawData ); + else *map_size = ROUND_SIZE( sec->Misc.VirtualSize ); + + *file_start = sec->PointerToRawData & ~sector_align; + *file_size = (sec->SizeOfRawData + (sec->PointerToRawData & sector_align) + sector_align) & ~sector_align; + if (*file_size > *map_size) *file_size = *map_size; } /* allocate and fill the temp file for a shared PE image mapping */ static int build_shared_mapping( struct mapping *mapping, int fd, IMAGE_SECTION_HEADER *sec, unsigned int nb_sec ) { - unsigned int i, size, max_size, total_size; + unsigned int i; + size_t file_size, map_size, max_size, total_size; off_t shared_pos, read_pos, write_pos; char *buffer = NULL; int shared_fd; @@ -151,9 +152,9 @@ static int build_shared_mapping( struct if ((sec[i].Characteristics & IMAGE_SCN_MEM_SHARED) && (sec[i].Characteristics & IMAGE_SCN_MEM_WRITE)) { - size = get_section_filemap_size( &sec[i] ); - if (size > max_size) max_size = size; - total_size += get_section_map_size( &sec[i] ); + get_section_sizes( &sec[i], &map_size, &read_pos, &file_size ); + if (file_size > max_size) max_size = file_size; + total_size += map_size; } } if (!(mapping->shared_size = total_size)) return 1; /* nothing to do */ @@ -175,20 +176,19 @@ static int build_shared_mapping( struct { if (!(sec[i].Characteristics & IMAGE_SCN_MEM_SHARED)) continue; if (!(sec[i].Characteristics & IMAGE_SCN_MEM_WRITE)) continue; + get_section_sizes( &sec[i], &map_size, &read_pos, &file_size ); write_pos = shared_pos; - shared_pos += get_section_map_size( &sec[i] ); - read_pos = sec[i].PointerToRawData; - size = get_section_filemap_size( &sec[i] ); - if (!read_pos || !size) continue; - toread = size; + shared_pos += map_size; + if (!sec[i].PointerToRawData || !file_size) continue; + toread = file_size; while (toread) { - long res = pread( fd, buffer + sec[i].SizeOfRawData - toread, toread, read_pos ); + long res = pread( fd, buffer + file_size - toread, toread, read_pos ); if (res <= 0) goto error; toread -= res; read_pos += res; } - if (pwrite( shared_fd, buffer, size, write_pos ) != size) goto error; + if (pwrite( shared_fd, buffer, file_size, write_pos ) != file_size) goto error; } free( buffer ); return 1;

1 0

Ken Thomases : winecoreaudio: Merge wodReset and wodHelper_Reset.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: acf349991626dc58b6d7005e79cef91ca069b571 URL: http://source.winehq.org/git/wine.git/?a=commit;h=acf349991626dc58b6d7005e7… Author: Ken Thomases <ken(a)codeweavers.com> Date: Thu Dec 21 03:49:58 2006 -0600 winecoreaudio: Merge wodReset and wodHelper_Reset. --- dlls/winmm/winecoreaudio/audio.c | 104 ++++++++++++-------------------------- 1 files changed, 32 insertions(+), 72 deletions(-) diff --git a/dlls/winmm/winecoreaudio/audio.c b/dlls/winmm/winecoreaudio/audio.c index b0d6c2c..a28057d 100644 --- a/dlls/winmm/winecoreaudio/audio.c +++ b/dlls/winmm/winecoreaudio/audio.c @@ -970,75 +970,6 @@ static void wodHelper_NotifyCompletions( } } -/************************************************************************** -* wodHelper_Reset [internal] -* -* Resets current output stream. -*/ -static DWORD wodHelper_Reset(WINE_WAVEOUT* wwo, BOOL reset) -{ - OSStatus status; - - FIXME("\n"); - - /* updates current notify list */ - /* if resetting, remove all wave headers and notify client that all headers were completed */ - wodHelper_NotifyCompletions(wwo, reset); - - pthread_mutex_lock(&wwo->lock); - - if (reset) - { - wwo->lpPlayPtr = wwo->lpQueuePtr = wwo->lpLoopPtr = NULL; - wwo->state = WINE_WS_STOPPED; - wwo->dwPlayedTotal = wwo->dwWrittenTotal = 0; - - wwo->dwPartialOffset = 0; /* Clear partial wavehdr */ - } - else - { - if (wwo->lpLoopPtr) - { - /* complicated case, not handled yet (could imply modifying the loop counter) */ - FIXME("Pausing while in loop isn't correctly handled yet, except strange results\n"); - wwo->lpPlayPtr = wwo->lpLoopPtr; - wwo->dwPartialOffset = 0; - wwo->dwWrittenTotal = wwo->dwPlayedTotal; /* this is wrong !!! */ - } else - { - LPWAVEHDR ptr; - DWORD sz = wwo->dwPartialOffset; - - /* reset all the data as if we had written only up to lpPlayedTotal bytes */ - /* compute the max size playable from lpQueuePtr */ - for (ptr = wwo->lpQueuePtr; ptr && ptr != wwo->lpPlayPtr; ptr = ptr->lpNext) - { - sz += ptr->dwBufferLength; - } - - /* because the reset lpPlayPtr will be lpQueuePtr */ - if (wwo->dwWrittenTotal > wwo->dwPlayedTotal + sz) ERR("doh\n"); - wwo->dwPartialOffset = sz - (wwo->dwWrittenTotal - wwo->dwPlayedTotal); - wwo->dwWrittenTotal = wwo->dwPlayedTotal; - wwo->lpPlayPtr = wwo->lpQueuePtr; - } - - wwo->state = WINE_WS_PAUSED; - } - - pthread_mutex_unlock(&wwo->lock); - - status = AudioOutputUnitStart(wwo->audioUnit); - - if (status) { - ERR( "AudioOutputUnitStart return %c%c%c%c\n", - (char) (status >> 24), (char) (status >> 16), (char) (status >> 8), (char) status); - return MMSYSERR_ERROR; /* FIXME return an error based on the OSStatus */ - } - - return MMSYSERR_NOERROR; -} - /************************************************************************** * wodWrite [internal] @@ -1183,15 +1114,44 @@ static DWORD wodRestart(WORD wDevID) */ static DWORD wodReset(WORD wDevID) { + WINE_WAVEOUT* wwo; + OSStatus status; + TRACE("(%u);\n", wDevID); - + if (wDevID >= MAX_WAVEOUTDRV) { WARN("bad device ID !\n"); return MMSYSERR_BADDEVICEID; } - - return wodHelper_Reset(&WOutDev[wDevID], TRUE); + + wwo = &WOutDev[wDevID]; + + FIXME("\n"); + + /* updates current notify list */ + /* if resetting, remove all wave headers and notify client that all headers were completed */ + wodHelper_NotifyCompletions(wwo, TRUE); + + pthread_mutex_lock(&wwo->lock); + + wwo->lpPlayPtr = wwo->lpQueuePtr = wwo->lpLoopPtr = NULL; + wwo->state = WINE_WS_STOPPED; + wwo->dwPlayedTotal = wwo->dwWrittenTotal = 0; + + wwo->dwPartialOffset = 0; /* Clear partial wavehdr */ + + pthread_mutex_unlock(&wwo->lock); + + status = AudioOutputUnitStart(wwo->audioUnit); + + if (status) { + ERR( "AudioOutputUnitStart return %c%c%c%c\n", + (char) (status >> 24), (char) (status >> 16), (char) (status >> 8), (char) status); + return MMSYSERR_ERROR; /* FIXME return an error based on the OSStatus */ + } + + return MMSYSERR_NOERROR; } /**************************************************************************

1 0

Ken Thomases : winecoreaudio: Move a memset out of a mutex-guarded section.
by Alexandre Julliard 21 Dec '06

21 Dec '06

Module: wine Branch: master Commit: 234558c2c7bcc815c43791acf6fdf4ad7262376d URL: http://source.winehq.org/git/wine.git/?a=commit;h=234558c2c7bcc815c43791acf… Author: Ken Thomases <ken(a)codeweavers.com> Date: Thu Dec 21 03:50:09 2006 -0600 winecoreaudio: Move a memset out of a mutex-guarded section. --- dlls/winmm/winecoreaudio/audio.c | 61 ++++++++++++++++++------------------- 1 files changed, 30 insertions(+), 31 deletions(-) diff --git a/dlls/winmm/winecoreaudio/audio.c b/dlls/winmm/winecoreaudio/audio.c index a28057d..e5b9e41 100644 --- a/dlls/winmm/winecoreaudio/audio.c +++ b/dlls/winmm/winecoreaudio/audio.c @@ -1365,47 +1365,46 @@ OSStatus CoreAudio_woAudioUnitIOProc(voi pthread_mutex_lock(&wwo->lock); - while (dataNeeded > 0) + while (dataNeeded > 0 && wwo->state == WINE_WS_PLAYING && wwo->lpPlayPtr) { - if (wwo->state == WINE_WS_PLAYING && wwo->lpPlayPtr) - { - unsigned int available = wwo->lpPlayPtr->dwBufferLength - wwo->dwPartialOffset; - unsigned int toCopy; - - if (available >= dataNeeded) - toCopy = dataNeeded; - else - toCopy = available; + unsigned int available = wwo->lpPlayPtr->dwBufferLength - wwo->dwPartialOffset; + unsigned int toCopy; - if (toCopy > 0) - { - memcpy((char*)ioData->mBuffers[0].mData + dataProvided, - wwo->lpPlayPtr->lpData + wwo->dwPartialOffset, toCopy); - wwo->dwPartialOffset += toCopy; - wwo->dwPlayedTotal += toCopy; - dataProvided += toCopy; - dataNeeded -= toCopy; - available -= toCopy; - } + if (available >= dataNeeded) + toCopy = dataNeeded; + else + toCopy = available; - if (available == 0) - { - wodHelper_PlayPtrNext(wwo); - needNotify = 1; - } + if (toCopy > 0) + { + memcpy((char*)ioData->mBuffers[0].mData + dataProvided, + wwo->lpPlayPtr->lpData + wwo->dwPartialOffset, toCopy); + wwo->dwPartialOffset += toCopy; + wwo->dwPlayedTotal += toCopy; + dataProvided += toCopy; + dataNeeded -= toCopy; + available -= toCopy; } - else + + if (available == 0) { - if (!dataProvided) - *ioActionFlags |= kAudioUnitRenderAction_OutputIsSilence; - memset((char*)ioData->mBuffers[0].mData + dataProvided, 0, dataNeeded); - dataProvided += dataNeeded; - dataNeeded = 0; + wodHelper_PlayPtrNext(wwo); + needNotify = 1; } } pthread_mutex_unlock(&wwo->lock); + /* We can't provide any more wave data. Fill the rest with silence. */ + if (dataNeeded > 0) + { + if (!dataProvided) + *ioActionFlags |= kAudioUnitRenderAction_OutputIsSilence; + memset((char*)ioData->mBuffers[0].mData + dataProvided, 0, dataNeeded); + dataProvided += dataNeeded; + dataNeeded = 0; + } + /* We only fill buffer 0. Set any others that might be requested to 0. */ for (buffer = 1; buffer < ioData->mNumberBuffers; buffer++) {

1 0