November 2007 - Wine-devel - list.winehq.org

re: ReadProcessMemory question
by Dan Kegel 17 Nov '07

17 Nov '07

Dave wrote: > I am trying to read the memory of another process with > ReadProcessmemory and I can only get it to work when > I commit the entire region I want to read with VirtualAllocEx - > but as you probably know when I do this it > initializes the entire memory region to 0s. > So my question is how do I get ReadProcessmemory to read from the region?? What are you trying to do? A little more context might help. FWIW, ReadProcessMemory has some drawbacks as an IPC mechanism: http://blogs.msdn.com/oldnewthing/archive/2006/01/17/513779.aspx Here are a few examples of how to use ReadProcessMemory: http://www.gamedev.net/community/forums/topic.asp?topic_id=328279 http://www.codeproject.com/threads/int64_memsteal.asp

1 0

ReadProcessMemory Question
by Dave Cartright 17 Nov '07

17 Nov '07

I am trying to read the memory of another process with ReadProcessMemory but I can only get it to work when I use VirtualAllocEx to commit the region of memory - problem is (as you very well know) that VirtualAllocEx initializes the region with 0s. So my question is how do I get ReadProcessMemory to work without using VirtualAllocEx?? --------------------------------- Be a better pen pal. Text or chat with friends inside Yahoo! Mail. See how.

1 0

#winehackers invite only?
by Stefan Dösinger 17 Nov '07

17 Nov '07

Hello, Did anyone change #winehackers to require an invite? What was the reason for this? Thanks, Stefan

2 2

Re: [Bug 10464] Regression when installing Skype
by James Hawkins 16 Nov '07

16 Nov '07

On 11/16/07, wine-bugs(a)winehq.org <wine-bugs(a)winehq.org> wrote: > http://bugs.winehq.org/show_bug.cgi?id=10464 > > --- Comment #5 from Austin English <austinenglish(a)gmail.com> 2007-11-16 18:33:31 --- > James, you can disable bugzilla from e-mailing you for CC's and/or disable all > e-mail. > > Austin, it's not about that. Disabling all emails is against the point. I'm subscribed to wine-bugs, so I don't need to be cc'ed. This is the position of several other developers, so I don't see where the confusion is coming from. If anything, ask the developer whose patch created a regression to assign the bug to himself. I'm going to stop now before I start ranting. -- James Hawkins

1 0

Re: USER32: win16 applications should not see long filenames returned from dialog control (try 4)
by Alex Villacís Lasso 16 Nov '07

16 Nov '07

Alex Villacís Lasso escribió: > I hope this version addresses the fact DlgDirSelect does not modify the > last error code, and therefore cannot be used to distinguish between a > valid and an invalid file. > > Changelog: > * Applications should not see long filenames returned from dialog control > > ------------------------------------------------------------------------ > > This patch has not been applied, and has not been commented on either. Is there something wrong with it? -- perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'

1 0

pglXMakeCurrent causes wine to crash on Mac os x
by Tobias Platen 16 Nov '07

16 Nov '07

Hello if i run wine on Mac os x 10.4.10 with Apples X11 (opengl linked staticly) i get the following runtime errors; X Error of failed request: BadValue (integer parameter out of range for operation) Major opcode of failed request: 129 (Apple-DRI) Minor opcode of failed request: 2 () Value in failed request: 0x57 Serial number of failed request: 16 Current serial number in output stream: 16 I have found out that commenting out the bad code will not cause that X Error wine crashes in a access violation (SIGSEGV) if (ctx) { // pglXMakeCurrent(gdi_display, win, ctx); } else { ERR(" couldn't initialize OpenGL, expect problems\n"); wine_tsx11_unlock(); return FALSE; } Does anybody know how to get opengl working on the mac.

1 0

Re: ntdll: send_debug_event: don't send uninitialized bytes
by Robert Shearman 16 Nov '07

16 Nov '07

Dan Kegel wrote: > +/* Returns size of initialized part of a particular EXCEPTION_RECORD */ > +static int exception_record_size( const EXCEPTION_RECORD *rec ) > +{ > + return sizeof(*rec) > + - (EXCEPTION_MAXIMUM_PARAMETERS-rec->NumberParameters) * sizeof(ULONG_PTR); > +} FIELD_OFFSET exists for this exact purpose. -- Rob Shearman

2 2

winetest_interactive : how does it work?
by Paul Vriens 16 Nov '07

16 Nov '07

Hi, I just ran winetest (by double clicking on the executable) on my WinXP box. I was missing some advpack popups that were there before. Francois sent in a patch to only do some particular tests when winetest_interactive is true. I would assume that clicking on a executable is interactive, not? AFAIK winetest_interactive is set to true|1 if the WINETEST_INTERACTIVE variable is set to 1 (include/wine/test.h). In programs/winetest/main.c we however only set WINETEST_INTERACTIVE to 0: 692 interactive_no[] = "WINETEST_INTERACTIVE=0", shouldn't WINETEST_INTERACTIVE be set according to the interactive variable in that same file? -- Cheers, Paul.

2 1

Re: Start: Convert start to Unicode. [try5]
by Alexandre Julliard 16 Nov '07

16 Nov '07

Anatoly Lyutin <vostok(a)etersoft.ru> writes: > @@ -202,9 +246,9 @@ int main(int argc, char *argv[]) > DWORD waitcode; > waitcode = WaitForSingleObject(sei.hProcess, INFINITE); > if (waitcode) > - fatal_error("WaitForSingleObject", GetLastError()); > + fatal_error(WaitForSingleObjectW, GetLastError()); > if (!GetExitCodeProcess(sei.hProcess, &exitcode)) > - fatal_error("GetExitCodeProcess", GetLastError()); > + fatal_error(GetExitCodeProcessW, GetLastError()); This is way overkill, there's no reason to convert errors like these to Unicode. WINE_ERR would work just as well. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] svchost: Added base implementation of svchost (1/12)
by Alexandre Julliard 16 Nov '07

16 Nov '07

Roy Shea <roy(a)cs.hmc.edu> writes: > +/* Allocate and initializes an ANSI version of the Unicode string */ > +static LPSTR UnicodeToAnsi(const WCHAR *unicode) > +{ > + int size; > + LPSTR ansi; > + > + size = WideCharToMultiByte(CP_UTF8, 0, unicode, -1, NULL, 0, NULL, NULL); > + ansi = HeapAlloc(GetProcessHeap(), 0, size); > + WideCharToMultiByte(CP_UTF8, 0, unicode, -1, ansi, size, NULL, NULL); > + return ansi; > +} UTF-8 is not the same thing as ANSI, this function is misleading. For procedure names what you probably want is a special-purpose function that rejects anything that isn't 7-bit ASCII. > + ret = RegOpenKeyExW(HKEY_LOCAL_MACHINE, service_param_key, 0, > + KEY_READ, &service_hkey); > + if (ret != ERROR_SUCCESS) > + { > + WINE_ERR("cannot open key %s, err=%d\n", > + wine_dbgstr_w(service_param_key), ret); > + HeapFree(GetProcessHeap(), 0, service_param_key); > + return FALSE; > + } > + > + /* Find DLL associate with service from key */ > + dll_name_short = GetRegValue(service_hkey, service_dll); > + if (!dll_name_short) > + { > + WINE_ERR("cannot find registry value %s for service %s\n", > + wine_dbgstr_w(service_dll), wine_dbgstr_w(service_name)); > + HeapFree(GetProcessHeap(), 0, service_param_key); > + return FALSE; > + } You are leaking registry handles pretty much everywhere. -- Alexandre Julliard julliard(a)winehq.org

1 0