- wine-devel - winehq.org

Re: [2/4] server: Add access mapping for a named pipe device object, adjust its default access rights. Resend.
by Alexandre Julliard 17 Apr '12

17 Apr '12

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > @@ -238,7 +238,7 @@ static const struct object_ops named_pipe_device_ops = > no_satisfied, /* satisfied */ > no_signal, /* signal */ > named_pipe_device_get_fd, /* get_fd */ > - no_map_access, /* map_access */ > + named_pipe_map_access, /* map_access */ > default_get_sd, /* get_sd */ > default_set_sd, /* set_sd */ > named_pipe_device_lookup_name, /* lookup_name */ > @@ -270,10 +270,10 @@ static void named_pipe_dump( struct object *obj, int verbose ) > > static unsigned int named_pipe_map_access( struct object *obj, unsigned int access ) > { > - if (access & GENERIC_READ) access |= STANDARD_RIGHTS_READ; > + if (access & GENERIC_READ) access |= STANDARD_RIGHTS_READ | FILE_READ_ATTRIBUTES; > if (access & GENERIC_WRITE) access |= STANDARD_RIGHTS_WRITE | FILE_CREATE_PIPE_INSTANCE; > if (access & GENERIC_EXECUTE) access |= STANDARD_RIGHTS_EXECUTE; > - if (access & GENERIC_ALL) access |= STANDARD_RIGHTS_ALL; > + if (access & GENERIC_ALL) access |= STANDARD_RIGHTS_ALL | FILE_READ_ATTRIBUTES | FILE_CREATE_PIPE_INSTANCE; > return access & ~(GENERIC_READ | GENERIC_WRITE | GENERIC_EXECUTE | GENERIC_ALL); Why do you need this for the device? -- Alexandre Julliard julliard(a)winehq.org

2 3

Re: [PATCH 2/3] Added support for shadowing of element pseudo-variables
by Marvin 17 Apr '12

17 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17915 Your paranoid android. === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [4/4] server: Propagate an error code returned by token_access_check() to the client. Resend.
by Marvin 17 Apr '12

17 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17912 Your paranoid android. === W7PROX64 (64 bit sync) === sync.c:923: Test failed: DeleteTimerQueueTimer sync.c:924: Test failed: ChangeTimerQueueTimer

2 1

Re: kernel32: Add more memory access checks.
by Marvin 17 Apr '12

17 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17913 Your paranoid android. === TEST64_W7SP1 (64 bit virtual) === virtual.c:854: Test failed: got size 0x100000 virtual.c:935: Test failed: got size 0x100000

1 0

Re: [3/4] server: Perform an access check for kernel objects without a security descriptor using access rights of the owner'
by Marvin 17 Apr '12

17 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17911 Your paranoid android. === WVISTAADM (32 bit sync) === sync.c:923: Test failed: DeleteTimerQueueTimer

2 1

[shlwapi] Add NULL checks to StrCpyW and StrCatW
by carlo.bramix＠libero.it 16 Apr '12

16 Apr '12

Hello friends, according to these results, it looks like the test I added in the patch crashes win2k SP4 but it works on all others. And it does not seem to be a thing that it can be resolved with dynamic library loading. So, what should I do? Do I need to comment them like it has been done (for example) into test_StrChrA()? Do I need to remove interely the test and revive previous patch? Please excuse my inexperience... Sincerely, Carlo Bramini. >----Messaggio originale---- >Da: testbot(a)testbot.winehq.org >Data: 16/04/2012 22.48 >A: <carlo.bramix(a)libero.it> >Ogg: TestBot job 17900 results: [shlwapi] Add NULL checks to StrCpyW and StrCatW > >VM Status Number of test failures >WINEBUILD completed >WNT4WSSP6 completed 0 >W2KPROSP4 completed 1 >WXPPROSP3 completed 0 >W2K3R2SESP2 completed 0 >WVISTAADM completed 0 >W2K8SE completed 0 >W7PRO completed 0 >W7PROX64 completed 0 >TEST64_W7SP1 completed 0 >W7PROX64 completed 0 >TEST64_W7SP1 completed 0 > >=== WINEBUILD (build) === >No build failures found > >=== WNT4WSSP6 (32 bit string) === >No test failures found > >=== W2KPROSP4 (32 bit string) === > >shlwapi: >string: unhandled exception c0000005 at 77C7F8A8 > >=== WXPPROSP3 (32 bit string) === >No test failures found > >=== W2K3R2SESP2 (32 bit string) === >No test failures found > >=== WVISTAADM (32 bit string) === >No test failures found > >=== W2K8SE (32 bit string) === >No test failures found > >=== W7PRO (32 bit string) === >No test failures found > >=== W7PROX64 (32 bit string) === >No test failures found > >=== TEST64_W7SP1 (32 bit string) === >No test failures found > >=== W7PROX64 (64 bit string) === >No test failures found > >=== TEST64_W7SP1 (64 bit string) === >No test failures found

1 0

Re: [shlwapi] Add NULL checks to StrCpyW and StrCatW
by Marvin 16 Apr '12

16 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17900 Your paranoid android. === W2KPROSP4 (32 bit string) === string: unhandled exception c0000005 at 77C7F8A8

1 0

Re: [PATCH] winealsa.drv: Init *num to 0 (Coverity)
by Andrew Eikum 16 Apr '12

16 Apr '12

On Mon, Apr 16, 2012 at 09:09:58AM +0200, Marcus Meissner wrote: > Also initialize a "may be uninitialized" value the compiler sees. > Not sure what this means. *num is initialized at the beginning of alsa_enum_devices, which is always called from GetEndpointIDs. Seems like a Coverity oversight. I'd either leave it as-is, or also remove the *num=0 from alsa_enum_devices. Andrew > Ciao, Marcus > --- > dlls/winealsa.drv/mmdevdrv.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c > index e3c1d15..d84bbb2 100644 > --- a/dlls/winealsa.drv/mmdevdrv.c > +++ b/dlls/winealsa.drv/mmdevdrv.c > @@ -345,7 +345,7 @@ static WCHAR *construct_device_id(EDataFlow flow, const WCHAR *chunk1, const cha > { > WCHAR *ret; > const WCHAR *prefix; > - DWORD len_wchars = 0, chunk1_len, copied = 0, prefix_len; > + DWORD len_wchars = 0, chunk1_len = 0, copied = 0, prefix_len; > > static const WCHAR dashW[] = {' ','-',' ',0}; > static const size_t dashW_len = (sizeof(dashW) / sizeof(*dashW)) - 1; > @@ -583,6 +583,7 @@ HRESULT WINAPI AUDDRV_GetEndpointIDs(EDataFlow flow, WCHAR ***ids, GUID **guids, > > TRACE("%d %p %p %p %p\n", flow, ids, guids, num, def_index); > > + *num = 0; > *ids = NULL; > *guids = NULL; > > -- > 1.7.3.4 > > >

2 2

Re: msvcrt: Added more length modifiers in scanf function
by Marvin 16 Apr '12

16 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17893 Your paranoid android. === WNT4WSSP6 (32 bit scanf) === scanf.c:144: Test failed: sprintf retuned 10 scanf.c:145: Test failed: got 1942892530, expected 12345678901234 === W2KPROSP4 (32 bit scanf) === scanf.c:144: Test failed: sprintf retuned 10 scanf.c:145: Test failed: got 1942892530, expected 12345678901234 === WXPPROSP3 (32 bit scanf) === scanf.c:144: Test failed: sprintf retuned 10 scanf.c:145: Test failed: got 1942892530, expected 12345678901234 === W2K3R2SESP2 (32 bit scanf) === scanf.c:144: Test failed: sprintf retuned 10 scanf.c:145: Test failed: got 1942892530, expected 12345678901234

1 0

Re: ws2_32: Fix hostent memory allocation
by Alexandre Julliard 16 Apr '12

16 Apr '12

Bruno Jesus <00cpxxx(a)gmail.com> writes: > @@ -5546,24 +5546,70 @@ INT WINAPI WSAUnhookBlockingHook(void) > * pointers (via a template of some kind). > */ > > -static int list_size(char** l, int item_size) > +/*********************************************************************** > + * list_size (INTERNAL) > + * > + * Calculate the size of data and number of items from a list based on > + * a fixed item size or strings by using strlen. The source list must be > + * NULL terminated. > + * > + * PARAMS > + * l [I] Pointer to array of source items. > + * item_size [I] Fixed item size or zero if it's a string list. > + * size_sum [O] Pointer to where the sum of item bytes will be > + * stored. May be NULL if not required by the caller. > + * item_count [O] Pointer to where the number of items bytes will be > + * stored. May be NULL if not required by the caller. > + * > + * NOTES > + * When item_size is zero the byte sum will also count the NULL > + * terminator for each string. > + */ > +static void list_size(char** l, int item_size, int *size_sum, int *item_count) > { > - int i,j = 0; > - if(l) > - { for(i=0;l[i];i++) > - j += (item_size) ? item_size : strlen(l[i]) + 1; > - j += (i + 1) * sizeof(char*); } > - return j; > + int s, c, *sum = (size_sum ? size_sum : &s), *count = (item_count ? item_count : &c); > + *sum = *count = 0; > + if(l) > + { > + for(;l[*count];(*count)++) > + *sum += (item_size) ? item_size : strlen(l[*count]) + 1; > + } That's not a good helper function, with all the optional parameters and various behaviors. Also if you need a big documentation block like this for a 5-line function it's a sign that something is wrong. -- Alexandre Julliard julliard(a)winehq.org

2 1

WASAPI ISimpleAudioVolume::SetMasterVolume appears to not work
by adam smith 16 Apr '12

16 Apr '12

Hi, I've been trying to get WASAPI audio output working for our app as Directsound output was very glitchy post the new audio stack. All seems to work great with WASAPI however I need to able to set the volume level independently for individual audio sessions / streams for crossfading however I'm having problems controlling the volume levels of a shared mode audio session / stream. specifically, ISimpleAudioVolume::SetMasterVolume http://msdn.microsoft.com/en-us/library/windows/desktop/dd316537(v=vs.85).a… Doesn't seem to work under Wine i.e. HRESULT hr; hr=audioClient_->GetService(IID_ISimpleAudioVolume,(void**)&simpleAudioVolume_); if (SUCCESS(hr)) hr=simpleAudioVolume_->SetMasterVolume(0.1,NULL); returns success but does not change the volume level though interestingly HRESULT hr; hr=audioClient_->GetService(IID_ISimpleAudioVolume,(void**)&simpleAudioVolume_); if (SUCCESS(hr)) hr=simpleAudioVolume_->SetMute(TRUE,NULL); does work and mute the volume. I've also tried usingsthe IAudioStreamVolume interface to control the volume again to no avail. Does anyone have any experience of these interfaces or know equivalent interfaces to use instead working I've tested on : Ubuntu 10.04, 11.10 Wine 1.4 , 1.5 , 1.5.2 PulseAudio 0.9.21 , 1.0 , no pulse audio (and windows 7 where the code works fine) Thanks, Adam

2 1

Re: jscript: Make sure to jump out of switch before entering implicit default clausule
by Marvin 16 Apr '12

16 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17883 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Testbot certificate has expired
by Morten Rønne 16 Apr '12

16 Apr '12

Hi testbot.winehq.com certificate expired on 13-04-2012. So now the wininet tests in dlls/wininet/tests/http.c are failing with an error for me. I can't determine if the fault are in the test or the function called should handle the error case. The error message is: err:wininet:NETCON_secure_connect SSL_connect failed: 12037 http.c:2873: Test failed: HttpSendRequest failed: 12037 wine: Unhandled page fault on read access to 0x00000008 at address 0x68a876c5 (thread 0009), starting debugger... Unhandled exception: page fault on read access to 0x00000008 in 32-bit code (0x68a876c5). Best Regards Morten Rønne

2 1

Re: gdi32: Add support for emulating bold faces of bitmap fonts.
by Huw Davies 16 Apr '12

16 Apr '12

On Mon, Apr 16, 2012 at 04:37:39PM +0900, Dmitry Timoshkov wrote: > diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c > index 0a5d3c3..eb3f9b5 100644 > --- a/dlls/gdi32/freetype.c > +++ b/dlls/gdi32/freetype.c > @@ -5669,12 +5669,27 @@ static inline BYTE get_max_level( UINT format ) > return 255; > } > > -static const BYTE masks[8] = {0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01}; > +static void emulate_bold_glyph(BYTE *buf, int pitch, int height) > +{ > + int x, y; > + BYTE *p = buf; > + > + for (y = 0; y < height; y++) > + { > + for (x = pitch - 1; x >= 0; x--) > + { > + p[x] |= p[x] >> 1; > + if (x > 0) p[x] |= p[x - 1] << 7; > + } > + p += pitch; > + } > +} > Don't we need to add some bytes if glyph_width % 32 == 0 ? Huw.

2 3

Re: msvcp90: Add codecvt ctors and dtors
by Piotr Caban 16 Apr '12

16 Apr '12

Hi, On 04/16/12 02:50, William Panlener wrote: > typedef struct { > + locale_facet facet; > + /* FIXME: type definition in standard but var here */ > + enum { ok, partial, error, noconv } result; > +} codecvt_base; This structure has incorrect size. It should not contain result variable. > +typedef struct { > + codecvt_base base; > + /* FIXME: incomplete struct definition */ > +} codecvt_char_char_mbstate; It's better to define whole structure. Also other structures are named differently, it would be probably better to name it codecvt_char. > +typedef struct { > + codecvt_base base; > + /* FIXME: incomplete struct definition */ > +} codecvt_short_char_mbstate; > + > +typedef struct { > + codecvt_base base; > + /* FIXME: incomplete struct definition */ > +} codecvt_wchar_char_mbstate; There's no reason to define separate structures for unsigned short and wchar_t variants of the class. Probably the only difference between this classes is typeinfo structure. > +/* FIXME: Potentially unused */ > +/* ?id@?$codecvt@DDH@std@@2V0locale@2@A */ > +locale_id codecvt_char_char_mbstate_id = {0}; I don't understand this comment. The id fields will be used by locale class. > +/* ??1codecvt_base@std@@UAE@XZ */ > +/* ??1codecvt_base@std@@UEAA@XZ */ > +DEFINE_THISCALL_WRAPPER(codecvt_base_dtor, 4) > +void __thiscall codecvt_base_dtor(codecvt_base *this) > +{ > + FIXME("(%p)\n stub!", this); > + TRACE("(%p)\n", this); > +} There's no need to use both FIXME and TRACE. > +/* ??1?$codecvt@DDH@std@@MAE@XZ */ > +/* ??1?$codecvt@DDH@std@@MEAA@XZ */ > +DEFINE_THISCALL_WRAPPER(codecvt_char_char_mbstate_dtor, 4) > +void __thiscall codecvt_char_char_mbstate_dtor(codecvt_char_char_mbstate *this) > +{ > + /*FIXME stub*/ > + codecvt_base_dtor(&this->base); > + TRACE("(%p)\n", this); > +} It's better to print a fixme message here. > +DEFINE_RTTI_DATA(codecvt_char_char_mbstate, 0, 1,&locale_facet_rtti_base_descriptor, NULL, NULL, " ?AV?$codecvt@DDH@std@@"); > +DEFINE_RTTI_DATA(codecvt_short_char_mbstate, 0, 1,&locale_facet_rtti_base_descriptor, NULL, NULL, " ?AV?$codecvt@GDH@std@@"); > +DEFINE_RTTI_DATA(codecvt_wchar_char_mbstate, 0, 1,&locale_facet_rtti_base_descriptor, NULL, NULL, ".?AV?$codecvt@_WDH@std@@"); The RTTI data for these structures is incorrect. > + __ASM_VTABLE(codecvt_base, ""/*FIXME*/); > + __ASM_VTABLE(codecvt_char_char_mbstate, ""/*FIXME*/); > + __ASM_VTABLE(codecvt_short_char_mbstate, ""/*FIXME*/); > + __ASM_VTABLE(codecvt_wchar_char_mbstate, ""/*FIXME*/); It's better to create correct virtual functions table. This may lead to some strange crashes.

1 0

Re: [PATCH 1/3] browseui: Add IOleWindow to IProgressDialog
by Nikolay Sivov 15 Apr '12

15 Apr '12

On 4/15/2012 23:08, Detlef Riekenberg wrote: > Used by apps to adjust the dialog position > or remove the cancel button before vista > > -- > By by ... Detlef > --- > dlls/browseui/progressdlg.c | 69 ++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 68 insertions(+), 1 deletions(-) > > diff --git a/dlls/browseui/progressdlg.c b/dlls/browseui/progressdlg.c > index 9b970e2..2807622 100644 > --- a/dlls/browseui/progressdlg.c > +++ b/dlls/browseui/progressdlg.c > @@ -60,6 +60,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(browseui); > > typedef struct tagProgressDialog { > IProgressDialog IProgressDialog_iface; > + IOleWindow IOleWindow_iface; > LONG refCount; > CRITICAL_SECTION cs; > HWND hwnd; > @@ -79,6 +80,11 @@ static inline ProgressDialog *impl_from_IProgressDialog(IProgressDialog *iface) > return CONTAINING_RECORD(iface, ProgressDialog, IProgressDialog_iface); > } > > +static inline ProgressDialog *impl_from_IOleWindow(IOleWindow *iface) > +{ > + return CONTAINING_RECORD(iface, ProgressDialog, IOleWindow_iface); > +} > + > static void set_buffer(LPWSTR *buffer, LPCWSTR string) > { > static const WCHAR empty_string[] = {0}; > @@ -270,9 +276,20 @@ static HRESULT WINAPI ProgressDialog_QueryInterface(IProgressDialog *iface, REFI > ProgressDialog *This = impl_from_IProgressDialog(iface); > *ppvOut = NULL; > > - if (IsEqualIID(iid,&IID_IUnknown) || IsEqualIID(iid,&IID_IProgressDialog)) > + if (IsEqualIID(iid,&IID_IUnknown)) > { > *ppvOut = This; > + TRACE("(%p, IID_IUnknown, %p) -> %p\n", This, ppvOut, This); > + } > + else if (IsEqualIID(iid,&IID_IProgressDialog)) > + { > + *ppvOut = This; > + TRACE("(%p, IID_IProgressDialog, %p) -> %p\n", This, ppvOut, This); > + } > + else if (IsEqualIID(iid,&IID_IOleWindow)) > + { > + *ppvOut =&This->IOleWindow_iface; > + TRACE("(%p, IID_IOleWindow, %p) -> %p\n", This, ppvOut, *ppvOut); > } What's a point in different traces? A single one is enough, with printed riid of course. Also please return iface instead of This, like *ppvOut = iface;, or if you prefer instance pointer add a pointer to interface like - &This->... > > if (*ppvOut) > @@ -495,6 +512,55 @@ static const IProgressDialogVtbl ProgressDialogVtbl = > ProgressDialog_Timer > }; > > +static HRESULT WINAPI OleWindow_QueryInterface(IOleWindow *iface, REFIID iid, LPVOID *ppvOut) > +{ > + ProgressDialog *This = impl_from_IOleWindow(iface); > + > + TRACE("(%p, %s, %p)\n", iface, debugstr_guid(iid), ppvOut); > + return ProgressDialog_QueryInterface(&This->IProgressDialog_iface, iid, ppvOut); > +} > + This trace should go to ProgressDialog_QueryInterface(), no need to place it in every interface.

1 0

Re: ddraw/tests: Compare the correct surface pointers.
by Marvin 15 Apr '12

15 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17873 Your paranoid android. === W2KPROSP4 (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(00A70028) === WXPPROSP3 (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(00BA0028) === W2K3R2SESP2 (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(00E40028) === WVISTAADM (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(02210028) === W2K8SE (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(02800028) === W7PRO (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(01BF0028) === W7PROX64 (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(02740028) === TEST64_W7SP1 (32 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(00000000) differs from the one returned by Lock(02690028) === W7PROX64 (64 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(0000000000000000) differs from the one returned by Lock(0000000002390040) === TEST64_W7SP1 (64 bit dsurface) === dsurface.c:272: Test failed: lpSurface from GetSurfaceDesc(0000000000000000) differs from the one returned by Lock(0000000002360040)

2 1

Re: [PATCH 1/3] browseui: Add IOleWindow to IProgressDialog
by Michael Stefaniuc 15 Apr '12

15 Apr '12

Hello Detlef, On 04/15/2012 10:08 PM, Detlef Riekenberg wrote: > Used by apps to adjust the dialog position > or remove the cancel button before vista > > -- > By by ... Detlef > --- > dlls/browseui/progressdlg.c | 69 ++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 68 insertions(+), 1 deletions(-) > > diff --git a/dlls/browseui/progressdlg.c b/dlls/browseui/progressdlg.c > index 9b970e2..2807622 100644 > --- a/dlls/browseui/progressdlg.c > +++ b/dlls/browseui/progressdlg.c > @@ -270,9 +276,20 @@ static HRESULT WINAPI ProgressDialog_QueryInterface(IProgressDialog *iface, REFI > ProgressDialog *This = impl_from_IProgressDialog(iface); > *ppvOut = NULL; > > - if (IsEqualIID(iid, &IID_IUnknown) || IsEqualIID(iid, &IID_IProgressDialog)) > + if (IsEqualIID(iid, &IID_IUnknown)) > { > *ppvOut = This; > + TRACE("(%p, IID_IUnknown, %p) -> %p\n", This, ppvOut, This); > + } > + else if (IsEqualIID(iid, &IID_IProgressDialog)) > + { > + *ppvOut = This; this is not correct as ppvOut takes an interface and not an object. While at it please fix the above one too. thanks bye michael

1 0

[GSoC] Reduce dependency on native dlls
by William Panlener 15 Apr '12

15 Apr '12

My name is William Panlener, and I am currently attending the University of Mississippi as a first semester Masters student in Computer Science. This is my first time participating in GSoC. My proposal for GSoC involves choosing an application with high dependence on native dlls and implementing enough stubs so as to eliminate this dependency. I chose the game, League of Legends, and its installer, Pando Media Booster. Most stubs that require implementation are in msvcp90 and wininet. I have two concerns with this proposal: 1) Is it unusual to work towards improving a specific application rather than focusing on a specific component of wine? 2) The scope of the project is difficult to gauge in advance since the next problematic stub cannot be identified until all the previous problematic stubs are taken care of. Is this an issue for a GSoC proposal? I would be more than willing to revise my proposal (if this is allowed) if either of the above points is a problem.

2 1

Re: [1/2] comctl32/tests: Added tests for mouse events handling (try 3)
by Dmitry Timoshkov 15 Apr '12

15 Apr '12

Daniel Jelinski <djelinski1(a)gmail.com> wrote: > Skipping tests on WinNT - older versions of comctl send a different > set of events. Then you need to figure out what is different, and make the tests pass there as well (there is 'optional' flag for that), otherwise you risk breaking applications written for NT. > + /* WM_RBUTTONDOWN does not return until it gets another mouse event. > + Make sure it gets one by posting WM_RBUTTONUP to message queue */ > + PostMessageA(hTree, WM_RBUTTONUP, 0, (LPARAM)0x10001); > + /* this sequence should NOT send WM_CONTEXTMENU */ > + SendMessageA(hTree, WM_RBUTTONDOWN, 2, (LPARAM)0x10001); > + > + /* ditch the first sequence as it processed more messages than we need. Redo */ > + flush_sequences(sequences, NUM_MSG_SEQUENCES); > + > + PostMessageA(hTree, WM_RBUTTONUP, 0, (LPARAM)0x10001); > + SendMessageA(hTree, WM_RBUTTONDOWN, 2, (LPARAM)0x10001); This can't work. If you use PostMessage you need to flush message queue before testing the sequence. Using SendMessage after PostMessage doesn't guarantee that a posted message is handled before a sent one, you may try generating hardware messages instead. > + > + if(sequences[PARENT_SEQ_INDEX]->sequence->message == 0x133) > + { > + win_skip("Comctl32 versions prior to 5.80 send different set of events"); > + return; > + } Please use symbolic names for messages instead of magic numbers. -- Dmitry.

4 5

automate winedbg IDE integration
by Ilya Basin 15 Apr '12

15 Apr '12

Hi developers. There's a manual on this page: http://www.winehq.org/docs/winedev-guide/dbg-others > Start the Wine debugger with a command line like: > winedbg --gdb --no-start <name_of_exe_to_debug.exe> > Start ddd > In ddd, use the 'Open File' or 'Open Program' to point to the Wine executable (which is either wine-pthread or wine-kthread depending on your settings). > In the output of 1/, there's a line like > target remote localhost:32878 > copy that line and paste into ddd command pane (the one with the (gdb) prompt) So many steps. Is there an easier way? Or maybe everyone use plain winedbg without any IDE?

2 5

winedbg-eclipse - winedbg wrapper for Eclipse IDE
by Ilya Basin 14 Apr '12

14 Apr '12

- Get Eclipse 3.7.2 CDT - Import existing code as Makefile (a wine component or a standalone - winelib program) - Check "Automate discovery" in project properties/C/C++ Build/Discovery - Create a standart C/C++ Application debug configuration - Set *.exe.so as application - Set winedbg-eclipse as GDB debugger - fix bugs faster!

1 0

Re: Wine release 1.5.2
by mike＠yurick.ca 13 Apr '12

13 Apr '12

I am no longer using this e-mail. Please contact me for my new e-mail address. Thank you, Mike

1 0

Re: vbscript: Short if statement may have optional 'end if'
by Marvin 13 Apr '12

13 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17842 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

gdi32/dibdrv: don't add_join if points are coincident in wide_line_segments (try 2)
by Daniel Lehman 13 Apr '12

13 Apr '12

try 2 - integrate into existing tests; updated commit message I think the only points in the middle of the points array can cause the crash I was seeing. wide_pen_lines trims coincident points/empty segments from the beginning and end of the array, so the wine_line_segment calls before and after the for loop I modified will always return valid faces

1 0

Re: gdiplus: Use StretchBlt instead of GdiAlphaBlend if device doesn't support alpha blending. Resend.
by Vincent Povirk 13 Apr '12

13 Apr '12

I ran some of my drawing tests with GdipSetCompositingMode(graphics, CompositingModeSourceCopy), and it looks like native in that mode does roughly what your patch does, except that pixels that would be 100% transparent are not drawn. If a printer device behaves differently then that would be very strange. If it is the same, it probably makes sense to use (and maybe set at graphics creation time if it's appropriate for the HDC) the compositing mode flag. So I guess this is an acceptable approach, and could probably be committed as is and refined later, but it'd be nice if you could look into compositing modes and drawing with 100% transparent colors, since you have a test program set up for printing and I don't.

1 0

Re: [PATCH 4/4] d3drm: Implement IDirect3DRMFrameX_GetChildren method and IDirect3DRMFrameArray interface. (try 2) (resend)
by Marvin 13 Apr '12

13 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17840 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 3/4] d3drm: Implement IDirect3DRMFrameX_GetParent and update tests. (resend)
by Marvin 13 Apr '12

13 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17839 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 4/4] d3drm: Implement IDirect3DRMFrameX_GetChildren method and IDirect3DRMFrameArray interface. (try 2) (resend)
by Marvin 13 Apr '12

13 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17835 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 3/4] d3drm: Implement IDirect3DRMFrameX_GetParent and update tests. (resend)
by Marvin 13 Apr '12

13 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17832 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Regression testing
by Daniel Jeliński 12 Apr '12

12 Apr '12

Hello all, I am trying to get Microsoft SQL Server Management Studio to work flawlessly under Wine. For the most part I create and triage related bug reports, but recently I also started tinkering with code, specifically with comctl library, which I am most familiar with. Back on subject. I thought I found a regression - on Wine 1.4 package downloaded from launchpad the "New query" button works fine, while on my compiled Wine it produces an error. So I did: git reset --hard wine-1.4 make and, surprisingly, I still had the problem with the compiled version. However after some combination of deleting leftover files, running make clean, make depend and make the button started working, so I started bisecting, hoping for the best. At some point I started getting bad versions, and every subsequent compile was bad - even after I ended bisecting and returned to wine-1.4, the button still did not work (and it still works under packaged Wine - I use the same install for all tests). This time make clean && make depend && make did not help. I am comparing what I did to [1], and the only difference I see is that I did not run configure before make. I'll try doing that later, but I think this shouldn't affect the outcome. Any other ideas? Also [1] suggests to set CC="ccache gcc -m32" on amd64 machines. I did that and when doing make I noticed that many lines start with: ccache gcc -m32 -m32 I guess that CC="ccache gcc" would be enough - but I didn't test that yet. Also I noticed that running parallel make (make -j) is never mentioned on [1]. Is there any reason I shouldn't use it? Daniel [1] http://wiki.winehq.org/RegressionTesting

5 6

Re: [PATCH 1/4] d3drm: Implement IDirect3DRMFrameX_AddChild. (try 2) (resend)
by Alexandre Julliard 12 Apr '12

12 Apr '12

Christian Costa <titan.costa(a)gmail.com> writes: > + children = (IDirect3DRMFrame3**)HeapAlloc(GetProcessHeap(), 0, (This->nb_children + 1) * sizeof(IDirect3DRMFrame3*)); > + if (!children) > + return E_OUTOFMEMORY; > + > + CopyMemory(children, This->children, This->nb_children * sizeof(IDirect3DRMFrame3*)); > + HeapFree(GetProcessHeap(), 0, This->children); HeapReAlloc is your friend. Also you should grow the buffer quadratically. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: kernel32/tests: Add a test for creating named mapping after closing without unmapping.
by Dmitry Timoshkov 12 Apr '12

12 Apr '12

Alexander Morozov <amorozov(a)etersoft.ru> wrote: > + ptr = MapViewOfFile(mapping, FILE_MAP_WRITE, 0, 0, 0); > + ok( ptr != NULL, "MapViewOfFile failed with error %d\n", GetLastError() ); > + CloseHandle(mapping); > + > + SetLastError(0xdeadbeef); > + mapping = CreateFileMappingA(INVALID_HANDLE_VALUE, NULL, PAGE_READWRITE, 0, MAPPING_SIZE, name); > + ok( mapping != INVALID_HANDLE_VALUE, "CreateFileMappingA failed\n" ); It would be interesting to add OpenFileMapping before and after CloseHandle and test what it returns. -- Dmitry.

1 0

Re: [PATCH] d3drm: Implement IDirect3DRMFrameX_GetTransform.
by Marvin 12 Apr '12

12 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17814 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: GDI+ pageoffset
by Stepa Nick 11 Apr '12

11 Apr '12

> > Run with `WINEDEBUG=loaddll wine my.exe` and ensure that builtin gdiplus > is being used. I run `WINEDEBUG=loaddll wine my.exe` and get such output: > trace:loaddll:load_builtin_dll Loaded L"KERNEL32.dll" at 0x7b810000: builtin > trace:loaddll:load_native_dll Loaded L"Z:\\home\\user\\tmp\\test\\CalculateCPP.exe" at 0x400000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\msvcrt.dll" at 0x72540000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\advapi32.dll" at 0x68550000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\gdi32.dll" at 0x684b0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\version.dll" at 0x6e960000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\user32.dll" at 0x68380000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\rpcrt4.dll" at 0x70d00000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\ole32.dll" at 0x685c0000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\gdiplus.dll" at 0x4ec50000: native > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\MSVCR100.dll" at 0x78aa0000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\shlwapi.dll" at 0x686c0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\comctl32.dll" at 0x68720000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\msimg32.dll" at 0x68810000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\mfc100u.dll" at 0x785f0000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\imm32.dll" at 0x68bc0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\winex11.drv" at 0x68980000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\uxtheme.dll" at 0x68c10000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\dwmapi.dll" at 0x68c40000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\MFC100ENU.DLL" at 0x5d360000: native > fixme:heap:HeapSetInformation (nil) 1 (nil) 0 > fixme:win:EnumDisplayDevicesW ((null),0,0x32f744,0x00000000), stub! So, non-native gdiplus.dll used?! Hmm... Can I see into gdiplus.dll anyway? I really need to understand how pixeloffset works.

2 1

Re: ws2_32/tests: Update hostent struct tests
by Austin English 11 Apr '12

11 Apr '12

On Tue, Apr 10, 2012 at 21:13, Bruno Jesus <00cpxxx(a)gmail.com> wrote: > * Added a final required test for the h->h_addr_list[0] to show we > must put it in the right place. > * Removed the cached use of older gethostbyname result since the > memory is static and turned into garbage in windows if the current > call fails. > * Added a skip if we hit older OS allowing we to avoid using broken() > on each test. + if(h->h_addr_list == addr.mem) /* <= W2K */ + { + skip("Skipping hostent tests since this OS is unsupported\n"); + return; + } + You should use win_skip so that the tests will only be skipped on windows (we'd want the failure to be exposed on wine). -- -Austin

2 2

Re: gdi32/dibdrv: don't add_join if points are coincident in wide_line_segments
by Alexandre Julliard 11 Apr '12

11 Apr '12

Daniel Lehman <dlehman(a)esri.com> writes: > This fixes a crash in PolyPolyline if points are in the same spot > > There were timeouts in the Testbot but were unrelated to anything I changed > > From 23edf2c2b89b98ef18849806e74b5d77f7889160 Mon Sep 17 00:00:00 2001 > From: Daniel Lehman <dlehman(a)esri.com> > Date: Wed, 25 Jan 2012 13:47:31 -0800 > Subject: gdi32/dibdrv: don't add_join if points are coincident in wide_line_segments > > if 2 points coincide (dx == dy == 0), wide_line_segments returns immediately > in that case, face_1 is not changed. it is not initialized so junk values on the > stack are used. when passed to add_join they create a region that causes a huge > amount of memory to be allocated. depending on the values, all available/rlimit > memory is exhausted and the program crashes. reverting to 1.3.26 works fine > > the test case submitted is a drastically reduced, simplest possible form of what our > internal test does, based on real map data You should integrate the test with the other graphics tests, to reuse all the infrastructure and also verify the graphics results. Also there is probably the same problem with the first and last points. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: d3drm: Some includes are useless
by Alexandre Julliard 11 Apr '12

11 Apr '12

Nozomi Kodama <nozomi.kodama(a)yahoo.com> writes: > From 9772529c63a9c11bc169545b6d68722308591a2f Mon Sep 17 00:00:00 2001 > From: Nozomi Kodama <nozomi.kodama(a)yahoo.com> > Date: Tue, 10 Apr 2012 10:50:31 +0200 > Subject: Some includes are useless That sort of change is not useful, the headers will be included anyway. -- Alexandre Julliard julliard(a)winehq.org

1 0

GDI+ pageoffset
by Stepa Nick 11 Apr '12

11 Apr '12

Hello. I have following code: > Bitmap * bmp = new Bitmap(500, 500, PixelFormat32bppARGB); > { > Graphics gr(bmp); > > //FIRST LINE > PointF* local1PointF = new PointF(17.823631, 89.429169); > PointF* local2PointF = new PointF(17.823631, 87.577080); > > //SECOND LINE > PointF* local3PointF = new PointF(14.933444, 89.429169); > PointF* local4PointF = new PointF(14.933444, 87.577080); > Pen * pen = new Pen(Color(255, 0, 0), 0.1); > gr.SetPageUnit(UnitMillimeter); > > gr.SetPixelOffsetMode(PixelOffsetModeHalf); //PROBLEM! > > gr.DrawLine(pen,(*local1PointF),(*local2PointF)); > gr.DrawLine(pen,(*local3PointF),(*local4PointF)); > } When I run it without SetPixelOffsetMode i get equal line, but with it line are not identical(first line is longer). I explore "dlls/gdiplus/graphics.c" and i find this comment: "/* FIXME: Pixel offset mode is not used anywhere except the getter/setter. */". So, I what to understand why I have different images(how pixeloffset affect on drawing lines)? I try to run windbg, run `WINEDEBUG=gdiplus,gdi,ddraw,graphics wine ./my.exe`, but analyzing output didn`t give my any results. Thanks!

2 1

Re: [PATCH 3/3] comctl32: Truncate the info tip text when using ANSI version messages.
by Marvin 11 Apr '12

11 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17802 Your paranoid android. === WNT4WSSP6 (32 bit tooltips) === tooltips.c:755: Test failed: lpszText has 0 chars

1 0

Re: [PATCH 2/3] comctl32/tests: Add tests for long info tip texts.
by Marvin 11 Apr '12

11 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17801 Your paranoid android. === WNT4WSSP6 (32 bit tooltips) === tooltips.c:755: Test failed: lpszText has 0 chars

1 0

Re: [PATCH 1/2] server: Add mechanism to retrieve the connect time of a socket (try 2).
by Dmitry Timoshkov 11 Apr '12

11 Apr '12

"Erich E. Hoover" <ehoover(a)mines.edu> wrote: > +/* Get information about a socket */ > +DECL_HANDLER(get_socket_info) > +{ > + struct sock *sock; > + > + sock=(struct sock *)get_handle_obj( current->process, req->handle, FILE_WRITE_ATTRIBUTES, &sock_ops ); Shouldn't the required access be FILE_READ_ATTRIBUTES or FILE_READ_PROPERTIES? Why write access is requied in order to read the socket connect time? Also please avoid including auto generated code in the patch. -- Dmitry.

2 1

Re: [PATCH 1/6] msvcrt: Rewrite asctime function
by Alexandre Julliard 11 Apr '12

11 Apr '12

Piotr Caban <piotr(a)codeweavers.com> writes: > @@ -48,7 +49,7 @@ static const int MonthLengths[2][12] = > > static inline int IsLeapYear(int Year) > { > - return Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0); > + return (Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0)) ? 1 : 0; Maybe you should do something like: return (((Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0)) ? 1 : 0) ? 1 : 0) ? 1 : 0; to make really sure you get 0 or 1, you never know what might happen ;-) -- Alexandre Julliard julliard(a)winehq.org

3 2

Re: [1/2] comctl32/tests: Added tests for mouse events handling (try 5)
by Marvin 10 Apr '12

10 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17777 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1894: Test failed: right click (parent): the msg sequence is not complete: expected 0000 - actual 0133 treeview.c:1895: Test failed: right click (tree): the msg sequence is not complete: expected 0000 - actual 000f treeview.c:1902: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 0133 treeview.c:1903: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 000f

1 0

Re: [1/2] comctl32/tests: Added tests for mouse events handling (try 4)
by Marvin 10 Apr '12

10 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17772 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1892: Test failed: right click (parent): in msg 0x004e expecting id 0xfffffffb got 0xfffffff4 treeview.c:1892: Test failed: right click (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1893: Test failed: right click (tree): the msg 0x000f should NOT have been sent by DefWindowProc treeview.c:1893: Test failed: right click (tree): the msg sequence is not complete: expected 0000 - actual 000f treeview.c:1900: Test failed: right drag (parent): in msg 0x004e expecting id 0xfffffe68 got 0xfffffff4 treeview.c:1900: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1901: Test failed: right drag (tree): the msg 0x000f should NOT have been sent by DefWindowProc treeview.c:1901: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 000f

1 0

Re: [PATCH 1/6] msvcrt: Rewrite asctime function (try3)
by Alexandre Julliard 10 Apr '12

10 Apr '12

Piotr Caban <piotr(a)codeweavers.com> writes: > + || !MSVCRT_CHECK_PMT(asctime_buf(time, mstm) != NULL)) { You don't want to use MSVCRT_CHECK_PMT this way, the error won't make sense. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/2] server: Add mechanism to retrieve the connect time of a socket (resend 2, deferred, rebased).
by Alexandre Julliard 10 Apr '12

10 Apr '12

"Erich E. Hoover" <ehoover(a)mines.edu> writes: > Real Name: > Erich Hoover > > Description: > This patch implements storing the connect time of a socket for > retrieval with the SO_CONNECT_TIME socket option (part 2). > > Changelog: > server: Add mechanism to retrieve the connect time of a socket. You should make it a generic get_info request, and return the time in standard server format. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: msvcrt: Add bsearch_s implementation by reusing code and tests from ntdll (try 4)
by Alexandre Julliard 10 Apr '12

10 Apr '12

André Hentschel <nerv(a)dawncrow.de> writes: > /********************************************************************* > + * bsearch_s (msvcrt.@) > + */ > +void* CDECL MSVCRT_bsearch_s(const void *key, const void *base, > + MSVCRT_size_t nmemb, MSVCRT_size_t size, > + int (__cdecl *compare)(void *, const void *, const void *), void *ctx) > +{ > + ssize_t min = 0; > + ssize_t max; > + > + if (!MSVCRT_CHECK_PMT(key != NULL) || !MSVCRT_CHECK_PMT(base != NULL) || > + !MSVCRT_CHECK_PMT(size != 0) || !MSVCRT_CHECK_PMT(compare != NULL)) > + { > + *MSVCRT__errno() = MSVCRT_EINVAL; > + return NULL; > + } > + > + if (!MSVCRT_CHECK_PMT(nmemb != 0) || !MSVCRT_CHECK_PMT(ctx != NULL)) > + { > + *MSVCRT__errno() = 0; > + return NULL; > + } That looks even more suspicious... -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 6/6] msvcrt/tests: Added more strftime tests
by Marvin 10 Apr '12

10 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17760 Your paranoid android. === WNT4WSSP6 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 78013981 === W2KPROSP4 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 7801124A === WXPPROSP3 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 77C4900F === W2K3R2SESP2 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 77BD9F8B

1 0

Re: [PATCH 1/6] msvcrt: Rewrite asctime function
by Marvin 10 Apr '12

10 Apr '12

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17759 Your paranoid android. === WNT4WSSP6 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === W2KPROSP4 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === WXPPROSP3 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === W2K3R2SESP2 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737

1 0

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-devel