Re: ntoskrnl.exe: unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces

2011-02-01 Thread Qian Hong
Dear Nikolay and James, On Sun, Jan 30, 2011 at 11:09 PM, James McKenzie wrote: > Qian: > > I would like to echo Nikolay's comment and add one more: > > Please search through the Bug Reports before submitting a new one. I really feel very sorry, I apologize for posting such issues the wrong plac

Re: Wine FAQ edits

2011-02-01 Thread Vitaliy Margolen
On 02/01/2011 09:47 PM, Albert Lee wrote: On Sat, Jan 29, 2011 at 4:36 PM, Vitaliy wrote: -cd ~/.wine/drive_c/Program\ Files +cd .wine/drive_c/Program\ Files This will break if current directory is not $HOME. 'cd' immediately precedes this. That still doesn't make the command valid. It won

Re: Wine FAQ edits

2011-02-01 Thread Albert Lee
On Sat, Jan 29, 2011 at 4:36 PM, Vitaliy wrote: > On 01/29/2011 12:21 PM, Dan Kegel wrote: >> >> Hi Albert, >> http://wiki.winehq.org/FAQ?action=diff&rev2=384&rev1=383 looks >> like it was a pretty big edit. >> >> Try smaller ones, and try to do just one kind of change per edit. > > That, and the

RE: Correction to crash inside RtlCaptureStackBackTrace() + test case

2011-02-01 Thread Janne Hakonen
> > It doesn't need a try-except block, just better wrap-around handling. > Ah, you mean to enhance the if-check? Well..., to me it seems that there is a problem with the 2nd check: "(void *)(frame + 1) >= NtCurrentTeb()->Tib.StackBase" If frame is 0x or close to it (e.g.

Re: crypt32: Fix quoting-behaviour

2011-02-01 Thread Juan Lang
Hi Christian, I'm afraid this patch is incorrect. In particular, -chars = CertRDNValueToStrA( - rdn->rgRDNAttr[j].dwValueType, - &rdn->rgRDNAttr[j].Value, psz ? psz + ret : NULL, - psz ? csz - ret : 0); +if ((dwStrTyp

Re: RFC: Adding Mac support to secur32/schannel.c

2011-02-01 Thread Henri Verbeet
On 1 February 2011 17:19, Juan Lang wrote: >> what IMO complicates writing them is that only the client part of >> schannel is currently implemented. > > That might be true for writing tests against Wine's implementation, > but there's nothing to stop them from being skipped if a server > implemen

Re: [dlls/shdocvw] Initialize variable (Coverity)

2011-02-01 Thread Nikolay Sivov
On 2/1/2011 19:21, Paul Vriens wrote: On 02/01/2011 03:26 PM, Nikolay Sivov wrote: On 2/1/2011 17:14, Paul Vriens wrote: Hi, Addresses Coverity #1090. If CoCreateInstance() fails we check the uninitialized urlfile variable. Same story with urlobj. In fact it looks better to try CoCreateInst

Re: [dlls/shdocvw] Initialize variable (Coverity)

2011-02-01 Thread Paul Vriens
On 02/01/2011 03:26 PM, Nikolay Sivov wrote: On 2/1/2011 17:14, Paul Vriens wrote: Hi, Addresses Coverity #1090. If CoCreateInstance() fails we check the uninitialized urlfile variable. Same story with urlobj. In fact it looks better to try CoCreateInstance first, before anything else like ls

Re: RFC: Adding Mac support to secur32/schannel.c

2011-02-01 Thread Juan Lang
> Well, I think that regardless of what schannel ends up using, wininet > and winhttp should be implemented on top schannel in the long term, > instead of using OpenSSL directly. I don't think GnuTLS is really the Well, that's certainly true, as there are features of at least wininet that can't be

Re: [dlls/shdocvw] Initialize variable (Coverity)

2011-02-01 Thread Nikolay Sivov
On 2/1/2011 17:14, Paul Vriens wrote: Hi, Addresses Coverity #1090. If CoCreateInstance() fails we check the uninitialized urlfile variable. Same story with urlobj. In fact it looks better to try CoCreateInstance first, before anything else like lstrcpy and just return earlier on failure.

Re: gdiplus: C++ fixes for headers

2011-02-01 Thread Iain Arnell
On Tue, Feb 1, 2011 at 10:28 AM, Paul Chitescu wrote: > On Tuesday 01 February 2011 09:49:46 am Iain Arnell wrote: >> To fix inclusion of gdiplus.h under C++, this patch: >> * adds missing class declaration for GpPen >> * adds missing class declaration for CharacterRange >> * fixes PenAlignment/Gp

Re: [PATCH 1/5] shell32: Added SHCNRF_InterruptLevel flag support to SHChangeNotifyRegister

2011-02-01 Thread Alexandre Julliard
Piotr Caban writes: > void InitChangeNotifications(void) > { > +watched_directories = 0; > +interrupts_server_terminate = CreateEventW(NULL, FALSE, FALSE, NULL); > +interrupts_server = CreateThread(NULL, 0, interrupts_server_thread, > NULL, 0, NULL); > } This needs to be done on

Re: programs: Add winemsibuilder.

2011-02-01 Thread Joris Huizer
While skimming through the new patches, I found: > +WCHAR *encode_stream( const WCHAR *in ) > +{ > +    DWORD c, next, count; > +    WCHAR *out, *p; > + > +    count = strlenW( in ); > +    if (count > MAX_STREAM_NAME) > +        return NULL; > + > +    count += 2; > +    out = HeapAlloc( GetProce

Re: RFC: Adding Mac support to secur32/schannel.c

2011-02-01 Thread Henri Verbeet
On 1 February 2011 02:08, Juan Lang wrote: > Sure, I can buy that.  I'll note that OpenSSL is also available for > the Mac, and already loaded by wininet and winhttp.  It could be > appropriate to move from GnuTLS to OpenSSL for schannel, so we'd only > have a single implementation for both Linux

Re: gdiplus: C++ fixes for headers

2011-02-01 Thread Paul Chitescu
On Tuesday 01 February 2011 09:49:46 am Iain Arnell wrote: > To fix inclusion of gdiplus.h under C++, this patch: > * adds missing class declaration for GpPen > * adds missing class declaration for CharacterRange > * fixes PenAlignment/GpPenAlignment declaration > * avoids redeclaration of PixelFor

Re: RFC: Adding Mac support to secur32/schannel.c

2011-02-01 Thread Damjan Jovanovic
On Tue, Feb 1, 2011 at 3:08 AM, Juan Lang wrote: > Hi Ken, thanks for the reply. > > > As Henri said, it's that it's a set of external dependencies (not just > one; GnuTLS has its own dependencies) and that they are security-related. > To the greatest extent practical, security-related libraries