June 2008 - Wine-Devel - list.winehq.org

Coverity reported buffer overrun error in "winebuild: Add support for 16-bit exe modules."
by Dan Kegel June 28, 2008

June 28, 2008

Coverity complains that http://source.winehq.org/git/wine.git/?a=commitdiff;h=0c214a7091af8efe39ffd… introduced a buffer overrun in winebuild. It looks like somebody forgot to dynamically grow an array? Here's the report. Can somebody familiar with the code (or with a little time on their hands) have a look? Thanks... - Dan CID: 682 Checker: OVERRUN_DYNAMIC (help) File: base/src/wine/tools/winebuild/spec16.c Function: BuildSpec16File Description: Overrun of dynamic array "(spec)->ordinals" of size 4 at position 4 with index variable "(i * 4)" 524 /******************************************************************* 525 * BuildSpec16File 526 * 527 * Build a Win16 assembly file from a spec file. 528 */ 529 void BuildSpec16File( DLLSPEC *spec ) 530 { 531 ORDDEF **typelist; 532 ORDDEF *entry_point = NULL; 533 int i, j, nb_funcs; 534 char header_name[256]; 535 536 /* File header */ 537 538 output_standard_file_header(); 539 540 if (!spec->file_name) 541 { 542 char *p; 543 spec->file_name = xstrdup( output_file_name ); 544 if ((p = strrchr( spec->file_name, '.' ))) *p = 0; 545 } 546 if (!spec->dll_name) /* set default name from file name */ 547 { 548 char *p; 549 spec->dll_name = xstrdup( spec->file_name ); 550 if ((p = strrchr( spec->dll_name, '.' ))) *p = 0; 551 } 552 553 /* store the main entry point as ordinal 0 */ 554 555 if (!spec->ordinals) 556 { Event buffer_alloc: Called allocating function "xmalloc" which allocated 4 bytes dictated by parameter 0 [model] Event var_assign: Assigned "(spec)->ordinals" to storage allocated by "xmalloc" Also see events: [var_assign][overrun-local] 557 spec->ordinals = xmalloc( sizeof(spec->ordinals[0]) ); 558 spec->ordinals[0] = NULL; 559 } At conditional (1): "(spec)->init_func != 0" taking true path At conditional (2): "(spec)->characteristics & 8192 == 0" taking true path 560 if (spec->init_func && !(spec->characteristics & IMAGE_FILE_DLL)) 561 { 562 entry_point = xmalloc( sizeof(*entry_point) ); 563 entry_point->type = TYPE_PASCAL; 564 entry_point->ordinal = 0; 565 entry_point->lineno = 0; 566 entry_point->flags = FLAG_REGISTER; 567 entry_point->name = NULL; 568 entry_point->link_name = xstrdup( spec->init_func ); 569 entry_point->export_name = NULL; 570 entry_point->u.func.arg_types[0] = 0; At conditional (3): "*((spec)->ordinals + 0) == 0" taking true path 571 assert( !spec->ordinals[0] ); 572 spec->ordinals[0] = entry_point; 573 } 574 575 /* Build sorted list of all argument types, without duplicates */ 576 577 typelist = xmalloc( (spec->limit + 1) * sizeof(*typelist) ); 578 At conditional (4): "i <= (spec)->limit" taking true path At conditional (6): "i <= (spec)->limit" taking true path At conditional (8): "i <= (spec)->limit" taking true path At conditional (10): "i <= (spec)->limit" taking true path At conditional (13): "i <= (spec)->limit" taking true path At conditional (16): "i <= (spec)->limit" taking false path 579 for (i = nb_funcs = 0; i <= spec->limit; i++) 580 { 581 ORDDEF *odp = spec->ordinals[i]; At conditional (5): "odp == 0" taking true path At conditional (7): "odp == 0" taking true path At conditional (9): "odp == 0" taking true path At conditional (11): "odp == 0" taking false path At conditional (14): "odp == 0" taking false path 582 if (!odp) continue; At conditional (12): "is_function != 0" taking false path At conditional (15): "is_function != 0" taking true path 583 if (is_function( odp )) typelist[nb_funcs++] = odp; 584 } 585 586 nb_funcs = sort_func_list( typelist, nb_funcs, callfrom16_type_compare ); 587 588 /* Output the module structure */ 589 590 sprintf( header_name, "__wine_spec_%s_dos_header", make_c_identifier(spec->dll_name) ); 591 output( "\n/* module data */\n\n" ); 592 output( "\t.data\n" ); 593 output( "\t.align %d\n", get_alignment(4) ); 594 output( "%s:\n", header_name ); 595 output( "\t%s 0x%04x\n", get_asm_short_keyword(), /* e_magic */ 596 IMAGE_DOS_SIGNATURE ); 597 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_cblp */ 598 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_cp */ 599 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_crlc */ 600 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_cparhdr */ 601 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_minalloc */ 602 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_maxalloc */ 603 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_ss */ 604 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_sp */ 605 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_csum */ 606 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_ip */ 607 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_cs */ 608 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_lfarlc */ 609 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_ovno */ 610 output( "\t%s 0,0,0,0\n", get_asm_short_keyword() ); /* e_res */ 611 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_oemid */ 612 output( "\t%s 0\n", get_asm_short_keyword() ); /* e_oeminfo */ 613 output( "\t%s 0,0,0,0,0,0,0,0,0,0\n", get_asm_short_keyword() ); /* e_res2 */ 614 output( "\t.long .L__wine_spec_ne_header-%s\n", header_name ); /* e_lfanew */ 615 616 output( ".L__wine_spec_ne_header:\n" ); 617 output( "\t%s 0x%04x\n", get_asm_short_keyword(), /* ne_magic */ 618 IMAGE_OS2_SIGNATURE ); 619 output( "\t.byte 0\n" ); /* ne_ver */ 620 output( "\t.byte 0\n" ); /* ne_rev */ 621 output( "\t%s .L__wine_spec_ne_enttab-.L__wine_spec_ne_header\n", /* ne_enttab */ 622 get_asm_short_keyword() ); 623 output( "\t%s .L__wine_spec_ne_enttab_end-.L__wine_spec_ne_enttab\n", /* ne_cbenttab */ 624 get_asm_short_keyword() ); 625 output( "\t.long 0\n" ); /* ne_crc */ At conditional (17): "(spec)->characteristics & 8192 != 0" taking false path 626 output( "\t%s 0x%04x\n", get_asm_short_keyword(), /* ne_flags */ 627 NE_FFLAGS_SINGLEDATA | 628 ((spec->characteristics & IMAGE_FILE_DLL) ? NE_FFLAGS_LIBMODULE : 0) ); 629 output( "\t%s 2\n", get_asm_short_keyword() ); /* ne_autodata */ 630 output( "\t%s %u\n", get_asm_short_keyword(), spec->heap_size ); /* ne_heap */ 631 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_stack */ At conditional (18): "entry_point == 0" taking false path 632 if (!entry_point) output( "\t.long 0\n" ); /* ne_csip */ 633 else output( "\t%s .L__wine_%s_0-.L__wine_spec_code_segment,1\n", 634 get_asm_short_keyword(), make_c_identifier(spec->dll_name) ); 635 output( "\t%s 0,2\n", get_asm_short_keyword() ); /* ne_sssp */ 636 output( "\t%s 2\n", get_asm_short_keyword() ); /* ne_cseg */ 637 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_cmod */ 638 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_cbnrestab */ 639 output( "\t%s .L__wine_spec_ne_segtab-.L__wine_spec_ne_header\n", /* ne_segtab */ 640 get_asm_short_keyword() ); 641 output( "\t%s .L__wine_spec_ne_rsrctab-.L__wine_spec_ne_header\n", /* ne_rsrctab */ 642 get_asm_short_keyword() ); 643 output( "\t%s .L__wine_spec_ne_restab-.L__wine_spec_ne_header\n", /* ne_restab */ 644 get_asm_short_keyword() ); 645 output( "\t%s .L__wine_spec_ne_modtab-.L__wine_spec_ne_header\n", /* ne_modtab */ 646 get_asm_short_keyword() ); 647 output( "\t%s .L__wine_spec_ne_imptab-.L__wine_spec_ne_header\n", /* ne_imptab */ 648 get_asm_short_keyword() ); 649 output( "\t.long 0\n" ); /* ne_nrestab */ 650 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_cmovent */ 651 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_align */ 652 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_cres */ 653 output( "\t.byte 0x%02x\n", NE_OSFLAGS_WINDOWS ); /* ne_exetyp */ 654 output( "\t.byte 0x%02x\n", NE_AFLAGS_FASTLOAD ); /* ne_flagsothers */ 655 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_pretthunks */ 656 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_psegrefbytes */ 657 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_swaparea */ 658 output( "\t%s 0\n", get_asm_short_keyword() ); /* ne_expver */ 659 660 /* segment table */ 661 662 output( "\n.L__wine_spec_ne_segtab:\n" ); 663 664 /* code segment entry */ 665 666 output( "\t%s .L__wine_spec_code_segment-%s\n", /* filepos */ 667 get_asm_short_keyword(), header_name ); 668 output( "\t%s .L__wine_spec_code_segment_end-.L__wine_spec_code_segment\n", /* size */ 669 get_asm_short_keyword() ); 670 output( "\t%s 0x%04x\n", get_asm_short_keyword(), NE_SEGFLAGS_32BIT ); /* flags */ 671 output( "\t%s .L__wine_spec_code_segment_end-.L__wine_spec_code_segment\n", /* minsize */ 672 get_asm_short_keyword() ); 673 674 /* data segment entry */ 675 676 output( "\t%s .L__wine_spec_data_segment-%s\n", /* filepos */ 677 get_asm_short_keyword(), header_name ); 678 output( "\t%s .L__wine_spec_data_segment_end-.L__wine_spec_data_segment\n", /* size */ 679 get_asm_short_keyword() ); 680 output( "\t%s 0x%04x\n", get_asm_short_keyword(), NE_SEGFLAGS_DATA ); /* flags */ 681 output( "\t%s .L__wine_spec_data_segment_end-.L__wine_spec_data_segment\n", /* minsize */ 682 get_asm_short_keyword() ); 683 684 /* resource directory */ 685 686 output_res16_directory( spec, header_name ); 687 688 /* resident names table */ 689 690 output( "\n\t.align %d\n", get_alignment(2) ); 691 output( ".L__wine_spec_ne_restab:\n" ); 692 output_resident_name( spec->dll_name, 0 ); At conditional (19): "i <= (spec)->limit" taking true path 693 for (i = 1; i <= spec->limit; i++) 694 { Event overrun-local: Overrun of dynamic array "(spec)->ordinals" of size 4 at position 4 with index variable "(i * 4)" Also see events: [buffer_alloc][var_assign] 695 ORDDEF *odp = spec->ordinals[i];

3 2

.pdb files for wine's gecko?
by Dan Kegel June 28, 2008

June 28, 2008

Hey Jacek, are .pdb files matching Wine's gecko available? Courtesy of John Reiser I now have a valgrind that can look at .pdb files for symbols, so I might be able to get you good backtraces for valgrind errors involving gecko if I had the right .pdb files. Thanks, Dan

1 0

Re: setupapi: Stub for SetupLogFile [resend]
by James Hawkins June 28, 2008

June 28, 2008

2008/6/28 Vitaly Perov <vitperov(a)etersoft.ru>: > Changelog: > - setupapi: Stub for SetupLogFile > + FIXME("(%p,'%s','%s','%s',%d,'%s','%s','%s',%d):stub\n", FileLogHandle, debugstr_w(LogSectionName), debugstr_w(SourceFileName), debugstr_w(TargetFileName), Checksum, debugstr_w(DiskTagfile), debugstr_w(DiskDescription), debugstr_w(OtherInfo), Flags); This line is more than 3 times the maximum length allowed for lines. The maximum length for lines is 80 columns, and while you can change that by, say, +-10 or so, it certainly shouldn't be 256 columns long. -- James Hawkins

1 0

Re: Made ENTER_GL() - LEAVE_GL() free X11 lock on exception inside opengl, code
by Roderick Colenbrander June 28, 2008

June 28, 2008

Hi, I don't think we want to go this way. First of all we want to emulate win32 opengl. If windows does this we MIGHT have to do something like this. Second all opengl calls hit ENTER_GL / LEAVE_GL and these few extra calls affect performance. Personally I think that when you are hitting such an X11 lock exception that there is just a bug somewhere in wine. E.g. in case of a Direct3D app a piece of wined3d not doing proper locking. The same can be the case for plain opengl apps where a call might not have called wine_tsx11_unlock. Regards, Roderick Colenbrander > From 31327546fc30520c80433fc964d7bd3ba4f80fa9 Mon Sep 17 00:00:00 2001 > From: Massimo Del Fedele <max(a)veneto.com> > Date: Sat, 28 Jun 2008 12:00:17 +0200 > Subject: Made ENTER_GL() - LEAVE_GL() free X11 lock on exception inside > opengl > code > > --- > dlls/opengl32/opengl_ext.h | 8 ++++++-- > dlls/opengl32/wgl.c | 6 ++++++ > 2 files changed, 12 insertions(+), 2 deletions(-) > > diff --git a/dlls/opengl32/opengl_ext.h b/dlls/opengl32/opengl_ext.h > index 8ae7c2f..9892351 100644 > --- a/dlls/opengl32/opengl_ext.h > +++ b/dlls/opengl32/opengl_ext.h > @@ -40,6 +40,9 @@ > #define WINAPI __stdcall > #define APIENTRY WINAPI > > +#include "wine/library.h" > +#include "wine/exception.h" > + > /* X11 locking */ > > extern void (*wine_tsx11_lock_ptr)(void); > @@ -47,8 +50,9 @@ extern void (*wine_tsx11_unlock_ptr)(void); > > /* As GLX relies on X, this is needed */ > void enter_gl(void); > -#define ENTER_GL() enter_gl() > -#define LEAVE_GL() wine_tsx11_unlock_ptr() > +void CALLBACK leave_gl(BOOL); > +#define ENTER_GL() enter_gl(); __TRY { > +#define LEAVE_GL() } __FINALLY(leave_gl) > > > typedef struct { > diff --git a/dlls/opengl32/wgl.c b/dlls/opengl32/wgl.c > index 6fbdeb4..7e31e20 100644 > --- a/dlls/opengl32/wgl.c > +++ b/dlls/opengl32/wgl.c > @@ -114,6 +114,12 @@ void enter_gl(void) > return; > } > > +void CALLBACK leave_gl(BOOL dummy) > +{ > + wine_tsx11_unlock_ptr(); > + return; > +} > + > const GLubyte * WINAPI wine_glGetString( GLenum name ); > > /*********************************************************************** > -- > 1.5.4.3 > > > -- Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen! Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer

1 0

Re: gdiplus benefits from valgrind testing when running DNS 9.0
by Susan Cragin June 28, 2008

June 28, 2008

-----Original Message----- >From: Nikolay Sivov <bunglehead(a)gmail.com> >Sent: Jun 28, 2008 9:23 AM >To: Susan Cragin <susancragin(a)earthlink.net> >Cc: Wine Developers <wine-devel(a)winehq.org> >Subject: Re: gdiplus benefits from valgrind testing when running DNS 9.0 > >Susan Cragin wrote: >> -----Original Message----- >> >>> From: Nikolay Sivov <bunglehead(a)gmail.com> >>> Sent: Jun 28, 2008 8:44 AM >>> To: Susan Cragin <susancragin(a)earthlink.net> >>> Subject: Re: gdiplus benefits from valgrind testing when running DNS 9.0 >>> >>> Susan Cragin wrote: >>> >>>> Whoops. Forgot one thing. >>>> When trying to run the program again, after the install, the following error prevents it from running again. >>>> >>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>> wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting >>>> fixme:dbghelp:dump_system_info fill in CPU vendorID and feature set >>>> >>>> >>>> >>>> -----Original Message----- >>>> >>>> >>>>> From: Susan Cragin <susancragin(a)earthlink.net> >>>>> Sent: Jun 28, 2008 8:27 AM >>>>> To: Wine Developers List <wine-devel(a)winehq.org> >>>>> Subject: gdiplus benefits from valgrind testing when running DNS 9.0 >>>>> >>>>> I tried installing and running Dragon NaturallySpeaking 9.0 yesterday with wine's gdiplus. >>>>> wine-1.1.0 >>>>> >>>>> The program installed well, and training worked, too. Recognition during training was fast enough to keep up. Running it worked up to a point, but when it finally crashed I was trying something that doesn't always work anyway. >>>>> >>>>> Here are the install gdiplus warnings: >>>>> >>>>> fixme:gdiplus:GdipSetLinePresetBlend not implemented >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=7c80a46c, edge=2081853302, rc=(589886,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> fixme:shell:SHAppBarMessage msg=2, data={cb=36, hwnd=0x60058, callback=7c1520e1, edge=1, rc=(0,0)-(800,28), lparam=ffffffec}: stub >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> err:ntdll:RtlpWaitForCriticalSection section 0xcc3bb8 "?" wait timed out in thread 00dd, blocked by 005e, retrying (60 sec) >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf41c, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>>>> >>>>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf470, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> fixme:oleacc:AccessibleObjectFromWindow 0x1009a 0 {618736e0-3c3d-11cf-810c-00aa00389b71} 0x33f17c >>>>> >>>>> ------------------------------- >>>>> Here are the run warnings: >>>>> ------------------------------- >>>>> >>>>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>>>> >>>>> wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting >>>>> >>>>> >>>>> err:ntdll:RtlpWaitForCriticalSection section 0x7bc932c4 "loader.c: loader_section" wait timed out in thread 00dd, blocked by 003a, retrying (60 sec) >>>>> >>>>> This next one eventually crashed the program. >>>>> >>>>> >>> I think I could fill this gap (GdipCreateLineBrushFromRectWithAngleI) >>> next week. There's nothing to stop this, I mean any >>> others unimplemented features of GDI+. >>> >> >> Let me clarify as well as I am able why I sent this message, because I am not a developer, just a tester, and I know you are not familiar with Dragon NaturallySpeaking. >> A couple of weeks ago, non-native gdiplus was essential to training the DNS 9.0 voice model. Training was so slow between speech and recognition that training wouldn't complete. That is no longer the case. The recognition model works fast enough to complete training. In fact, it works very fast. >> >Sounds good. So you think performance increases actually due gdiplus >changes? As for me I don't think so To be honest, I don't know because I'm not a programmer. I usually filter my findings through Dan Kegel. But I can tell you that training didn't complete last week, due to gdi problems, and it completes now. And the errors I see that led to the eventual program crash appear to be gdi errors.

1 0

Re: gdiplus benefits from valgrind testing when running DNS 9.0
by Susan Cragin June 28, 2008

June 28, 2008

-----Original Message----- >From: Nikolay Sivov <bunglehead(a)gmail.com> >Sent: Jun 28, 2008 8:44 AM >To: Susan Cragin <susancragin(a)earthlink.net> >Subject: Re: gdiplus benefits from valgrind testing when running DNS 9.0 > >Susan Cragin wrote: >> Whoops. Forgot one thing. >> When trying to run the program again, after the install, the following error prevents it from running again. >> >> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >> wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting >> fixme:dbghelp:dump_system_info fill in CPU vendorID and feature set >> >> >> >> -----Original Message----- >> >>> From: Susan Cragin <susancragin(a)earthlink.net> >>> Sent: Jun 28, 2008 8:27 AM >>> To: Wine Developers List <wine-devel(a)winehq.org> >>> Subject: gdiplus benefits from valgrind testing when running DNS 9.0 >>> >>> I tried installing and running Dragon NaturallySpeaking 9.0 yesterday with wine's gdiplus. >>> wine-1.1.0 >>> >>> The program installed well, and training worked, too. Recognition during training was fast enough to keep up. Running it worked up to a point, but when it finally crashed I was trying something that doesn't always work anyway. >>> >>> Here are the install gdiplus warnings: >>> >>> fixme:gdiplus:GdipSetLinePresetBlend not implemented >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=7c80a46c, edge=2081853302, rc=(589886,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> fixme:shell:SHAppBarMessage msg=2, data={cb=36, hwnd=0x60058, callback=7c1520e1, edge=1, rc=(0,0)-(800,28), lparam=ffffffec}: stub >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> err:ntdll:RtlpWaitForCriticalSection section 0xcc3bb8 "?" wait timed out in thread 00dd, blocked by 005e, retrying (60 sec) >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf41c, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>> >>> fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf470, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> fixme:oleacc:AccessibleObjectFromWindow 0x1009a 0 {618736e0-3c3d-11cf-810c-00aa00389b71} 0x33f17c >>> >>> ------------------------------- >>> Here are the run warnings: >>> ------------------------------- >>> >>> fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub >>> >>> wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting >>> >>> >>> err:ntdll:RtlpWaitForCriticalSection section 0x7bc932c4 "loader.c: loader_section" wait timed out in thread 00dd, blocked by 003a, retrying (60 sec) >>> >>> This next one eventually crashed the program. >>> >I think I could fill this gap (GdipCreateLineBrushFromRectWithAngleI) >next week. There's nothing to stop this, I mean any >others unimplemented features of GDI+. Let me clarify as well as I am able why I sent this message, because I am not a developer, just a tester, and I know you are not familiar with Dragon NaturallySpeaking. A couple of weeks ago, non-native gdiplus was essential to training the DNS 9.0 voice model. Training was so slow between speech and recognition that training wouldn't complete. That is no longer the case. The recognition model works fast enough to complete training. In fact, it works very fast.

2 1

gdiplus benefits from valgrind testing when running DNS 9.0
by Susan Cragin June 28, 2008

June 28, 2008

I tried installing and running Dragon NaturallySpeaking 9.0 yesterday with wine's gdiplus. wine-1.1.0 The program installed well, and training worked, too. Recognition during training was fast enough to keep up. Running it worked up to a point, but when it finally crashed I was trying something that doesn't always work anyway. Here are the install gdiplus warnings: fixme:gdiplus:GdipSetLinePresetBlend not implemented fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=7c80a46c, edge=2081853302, rc=(589886,13361216)-(2081853346,13361216), lparam=cc00b0}: stub fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub fixme:shell:SHAppBarMessage msg=2, data={cb=36, hwnd=0x60058, callback=7c1520e1, edge=1, rc=(0,0)-(800,28), lparam=ffffffec}: stub fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub err:ntdll:RtlpWaitForCriticalSection section 0xcc3bb8 "?" wait timed out in thread 00dd, blocked by 005e, retrying (60 sec) fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf41c, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf470, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub fixme:oleacc:AccessibleObjectFromWindow 0x1009a 0 {618736e0-3c3d-11cf-810c-00aa00389b71} 0x33f17c ------------------------------- Here are the run warnings: ------------------------------- fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting err:ntdll:RtlpWaitForCriticalSection section 0x7bc932c4 "loader.c: loader_section" wait timed out in thread 00dd, blocked by 003a, retrying (60 sec) This next one eventually crashed the program.

2 1

Re: gdiplus benefits from valgrind testing when running DNS 9.0
by Susan Cragin June 28, 2008

June 28, 2008

Whoops. Forgot one thing. When trying to run the program again, after the install, the following error prevents it from running again. fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting fixme:dbghelp:dump_system_info fill in CPU vendorID and feature set -----Original Message----- >From: Susan Cragin <susancragin(a)earthlink.net> >Sent: Jun 28, 2008 8:27 AM >To: Wine Developers List <wine-devel(a)winehq.org> >Subject: gdiplus benefits from valgrind testing when running DNS 9.0 > >I tried installing and running Dragon NaturallySpeaking 9.0 yesterday with wine's gdiplus. >wine-1.1.0 > >The program installed well, and training worked, too. Recognition during training was fast enough to keep up. Running it worked up to a point, but when it finally crashed I was trying something that doesn't always work anyway. > >Here are the install gdiplus warnings: > >fixme:gdiplus:GdipSetLinePresetBlend not implemented > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=7c80a46c, edge=2081853302, rc=(589886,13361216)-(2081853346,13361216), lparam=cc00b0}: stub > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >fixme:shell:SHAppBarMessage msg=2, data={cb=36, hwnd=0x60058, callback=7c1520e1, edge=1, rc=(0,0)-(800,28), lparam=ffffffec}: stub > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >err:ntdll:RtlpWaitForCriticalSection section 0xcc3bb8 "?" wait timed out in thread 00dd, blocked by 005e, retrying (60 sec) > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf41c, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub > >fixme:shell:SHAppBarMessage msg=6, data={cb=36, hwnd=0x60058, callback=cbf470, edge=2081853302, rc=(1638456,13361216)-(2081853346,13361216), lparam=cc00b0}: stub > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >fixme:oleacc:AccessibleObjectFromWindow 0x1009a 0 {618736e0-3c3d-11cf-810c-00aa00389b71} 0x33f17c > >------------------------------- >Here are the run warnings: >------------------------------- > >fixme:gdiplus:GdipCreateHBITMAPFromBitmap stub > >wine: Call from 0x7b845630 to unimplemented function gdiplus.dll.GdipCreateLineBrushFromRectWithAngleI, aborting > > >err:ntdll:RtlpWaitForCriticalSection section 0x7bc932c4 "loader.c: loader_section" wait timed out in thread 00dd, blocked by 003a, retrying (60 sec) > >This next one eventually crashed the program. > > > > > >

1 0

Re: README: Add Russian translation (4th try)
by Andrey Turkin June 28, 2008

June 28, 2008

Hi Vladimir, There are a few punctuation issues still left. >Wine это программа >реализует вызовы Windows API >используя >В директории содержащей >решения проблем, прочтите extra comma here? >операционные системы перечисленные выше >У FreeBSD 6.3 есть патч разрешающий comma missed, and "позволяющий" seems to be better here >гарантирует > что >Убедитесь что, опции comma in wrong place >особенностей файловой системы необходимые некоторым comma missing >убедиться что, у вас comma in wrong place >Вероятно это лучшая начальная точка sounds weird >Вы так же можете отправлять should be "также" Vladimir Pankratov wrote: > Hello. > > Corrected some wrong strings. > Added README.ru file. > > Thanks fo help. >

1 0

new valgrind errors in riched20
by Dan Kegel June 28, 2008

June 28, 2008

Seem to be four or so new valgrind warnings in riched20 today, probably due to Dylan's changes (though my cat may have been sufing to friskies.com and affected the results, who knows): http://kegel.com/wine/valgrind/logs-2008-06-27/vg-riched20_editor-diff.txt + Invalid read of size 4 + at ME_AddUndoItem (undo.c:59) + by ME_SetParaFormat (para.c:362) + by ME_PlayUndoItem (undo.c:288) + by ME_Undo (undo.c:360) + by RichEditWndProc_common (editor.c:2067) + by RichEditWndProcA (editor.c:3386) + by ??? (library.h:163) + by call_window_proc (winproc.c:457) + by WINPROC_call_window (winproc.c:2207) + by call_window_proc (message.c:1639) + by send_message (message.c:2463) + by SendMessageA (message.c:2608) + by test_EM_SETUNDOLIMIT (editor.c:2238) + by func_editor (editor.c:4581) + by run_test (test.h:449) + by main (test.h:498) ... + Conditional jump or move depends on uninitialised value(s) + at test_EM_AUTOURLDETECT (editor.c:1719) + by func_editor (editor.c:4600) + by run_test (test.h:449) + by main (test.h:498) + Uninitialised value was created by a client request + at mark_block_uninitialized (heap.c:164) + by RtlAllocateHeap (heap.c:1239) + by heap_alloc (editor.h:28) + by RichEditWndProc_common (editor.c:2573) + by RichEditWndProcA (editor.c:3386) + by ??? (library.h:163) + by call_window_proc (winproc.c:457) + by WINPROC_call_window (winproc.c:2207) + by call_window_proc (message.c:1639) + by send_message (message.c:2463) + by SendMessageA (message.c:2608) + by test_EM_AUTOURLDETECT (editor.c:1705) + by func_editor (editor.c:4600) + by run_test (test.h:449) + by main (test.h:498) ... + Invalid write of size 4 + at ME_AddUndoItem (undo.c:62) + by ME_InternalDeleteText (caret.c:322) + by ME_PlayUndoItem (undo.c:308) + by ME_Undo (undo.c:360) + by RichEditWndProc_common (editor.c:2067) + by RichEditWndProcW (editor.c:3381) + by ??? (library.h:163) + by call_window_proc (winproc.c:457) + by WINPROC_CallProcAtoW (winproc.c:1011) + by WINPROC_call_window (winproc.c:2209) + by call_window_proc (message.c:1639) + by send_message (message.c:2463) + by SendMessageA (message.c:2608) + by test_undo_coalescing (editor.c:4398) + by func_editor (editor.c:4602) + by run_test (test.h:449) + by main (test.h:498) ... + Conditional jump or move depends on uninitialised value(s) + at ME_AddUndoItem (undo.c:59) + by ME_InternalDeleteText (caret.c:322) + by ME_PlayUndoItem (undo.c:308) + by ME_Undo (undo.c:360) + by RichEditWndProc_common (editor.c:2067) + by RichEditWndProcW (editor.c:3381) + by ??? (library.h:163) + by call_window_proc (winproc.c:457) + by WINPROC_CallProcAtoW (winproc.c:1011) + by WINPROC_call_window (winproc.c:2209) + by call_window_proc (message.c:1639) + by send_message (message.c:2463) + by SendMessageA (message.c:2608) + by test_undo_coalescing (editor.c:4398) + by func_editor (editor.c:4602) + by run_test (test.h:449) + by main (test.h:498) + Uninitialised value was created by a client request + at mark_block_uninitialized (heap.c:164) + by RtlAllocateHeap (heap.c:1239) + by heap_alloc (editor.h:28) + by ME_MakeStringN (string.c:46) + by ME_InternalDeleteText (caret.c:326) + by ME_PlayUndoItem (undo.c:308) + by ME_Undo (undo.c:360) + by RichEditWndProc_common (editor.c:2067) + by RichEditWndProcW (editor.c:3381) + by ??? (library.h:163) + by call_window_proc (winproc.c:457) + by WINPROC_CallProcAtoW (winproc.c:1011) + by WINPROC_call_window (winproc.c:2209) + by call_window_proc (message.c:1639) + by send_message (message.c:2463) + by SendMessageA (message.c:2608) + by test_undo_coalescing (editor.c:4398) + by func_editor (editor.c:4602) + by run_test (test.h:449) + by main (test.h:498)

2 1