December 2014 - Wine-bugs - list.winehq.org

[Bug 37098] New: Checking Wine with PVS-Studio and Clang Static Analyzer
by wine-bugs＠winehq.org 04 Jan '15

04 Jan '15

http://bugs.winehq.org/show_bug.cgi?id=37098 Bug ID: 37098 Summary: Checking Wine with PVS-Studio and Clang Static Analyzer Product: Wine Version: 1.7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: nick.smallville(a)gmail.com Hello, I checked out the Wine using two static analyzers(Clang Static Analyzer && PVS-Studio) in the openSUSE 13.1. In my article, I described the errors found: http://www.viva64.com/en/b/0272/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 5

[Bug 37127] New: PVS-Studio Error: Unreliable tests
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

https://bugs.winehq.org/show_bug.cgi?id=37127 Bug ID: 37127 Summary: PVS-Studio Error: Unreliable tests Product: Wine Version: 1.7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: lukebenes(a)hotmail.com PVS-Studio identifies Unreliable tests. Some of the modules' folders contain the test folder with source files for tests. Debug information is printed through the 'ok' macro. Here are a few suspicious fragments: V501 There are identical sub-expressions to the left and to the right of the '==' operator: ddsd3.lpSurface == ddsd3.lpSurface dsurface.c 272 ... ok(ddsd3.lpSurface == ddsd3.lpSurface, //<== "lpSurface from GetSurfaceDesc(%p) differs\ from the one returned by Lock(%p)\n", ddsd3.lpSurface, ddsd2.lpSurface); //<== ... It very much looks like a typo. I suspect this code should compare the same variables that are printed. V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. url.c 767 ... ok(size == no_callback ? 512 : 13, "size=%d\n", size); ... The precedence of the "==" operator is higher than that of '?:', so the size variable is not compared to the values 512 and 13. The expression is always true as it evaluates either to 512 or 13, which means this check doesn't check anything. Other similar fragments: V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. string.c 1086 V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. string.c 1111 V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. reader.c 761 V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. protocol.c 2928 V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. dde.c 1594 V502 Perhaps the '?:' operator works in a different way than it was expected. The '?:' operator has a lower priority than the '==' operator. reader.c 761 details on the V501 error here: http://www.viva64.com/en/d/0090/ details on the V502 error here: http://www.viva64.com/en/d/0091/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 6

[Bug 37126] New: PVS-Studio Error: Printing the result of identical functions
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

https://bugs.winehq.org/show_bug.cgi?id=37126 Bug ID: 37126 Summary: PVS-Studio Error: Printing the result of identical functions Product: Wine Version: 1.7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: lukebenes(a)hotmail.com PVS-Studio identifies Printing the result of identical functions Error,V681. The language standard does not define an order in which the 'tlb_read_byte' functions will be called during evaluation of arguments. tlb.c 650 ... printf("\\%2.2x \\%2.2x\n", tlb_read_byte(), tlb_read_byte()); ... According to the C++ language standard, the sequence of computing a function's actual arguments is not defined. Which function will be called first depends on the compiler, compilation parameters, etc. details on the V681 error here: http://www.viva64.com/en/d/0315/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 5

[Bug 37123] New: PVS-Studio Error: Double type conversion
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

https://bugs.winehq.org/show_bug.cgi?id=37123 Bug ID: 37123 Summary: PVS-Studio Error: Double type conversion Product: Wine Version: 1.7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: lukebenes(a)hotmail.com PVS-Studio identifies a Double type conversion Error. The *void pointer is cast to other types twice: first to char*, then to DWORD*, after which an offset is added. Either the expression lacks parentheses or the code is excessive. Whether or not there is an error here depends on how much the programmer wanted to increment the pointer. V650 Type casting operation is utilized 2 times in succession. Next, the '+' operation is executed. Probably meant: (T1)((T2)a + b). typelib.c 9147 ... struct WMSFT_SegContents arraydesc_seg; typedef struct tagWMSFT_SegContents { DWORD len; void *data; } WMSFT_SegContents; ... DWORD offs = file->arraydesc_seg.len; DWORD *encoded; encoded = (DWORD*)((char*)file->arraydesc_seg.data) + offs;//<== Another similar issue: V650 Type casting operation is utilized 2 times in succession. Next, the '+' operation is executed. Probably meant: (T1)((T2)a + b). protocol.c 194 INT WINAPI EnumProtocolsW(LPINT protocols, LPVOID buffer, LPDWORD buflen) { ... unsigned int string_offset; ... pi[i].lpProtocol = (WCHAR*)(char*)buffer + string_offset;//<== ... } details on the V650 error here: http://www.viva64.com/en/d/0269/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 7

[Bug 37120] New: PVS-Studio Error: Equivalent branches of if operator
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

https://bugs.winehq.org/show_bug.cgi?id=37120 Bug ID: 37120 Summary: PVS-Studio Error: Equivalent branches of if operator Product: Wine Version: 1.7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: lukebenes(a)hotmail.com Equivalent branches of if operator V523 The 'then' statement is equivalent to the 'else' statement. filedlg.c 3302 if(pDIStruct->itemID == liInfos->uSelectedItem) { ilItemImage = (HIMAGELIST) SHGetFileInfoW ( (LPCWSTR) tmpFolder->pidlItem, 0, &sfi, sizeof (sfi), shgfi_flags ); } else { ilItemImage = (HIMAGELIST) SHGetFileInfoW ( (LPCWSTR) tmpFolder->pidlItem, 0, &sfi, sizeof (sfi), shgfi_flags ); } This code is either excessive or contains a typo. V523 The 'then' statement is equivalent to the 'else' statement. genres.c 1130 ... if(win32) { put_word(res, 0); /* Reserved */ /* FIXME: The ResType in the NEWHEADER structure should * contain 14 according to the MS win32 doc. This is * not the case with the BRC compiler and I really doubt * the latter. Putting one here is compliant to win16 spec, * but who knows the true value? */ put_word(res, 1); /* ResType */ put_word(res, icog->nicon); for(ico = icog->iconlist; ico; ico = ico->next) { ... } } else /* win16 */ { put_word(res, 0); /* Reserved */ put_word(res, 1); /* ResType */ put_word(res, icog->nicon); for(ico = icog->iconlist; ico; ico = ico->next) { ... } } ... One of the repeating branches is commented. Perhaps this is an incomplete fragment, not an error, but I decided to point it out anyway. details on the V523 error here: http://www.viva64.com/en/d/0112/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 5

[Bug 37119] New: PVS-Studio Error: Cascade of conditional operators
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

https://bugs.winehq.org/show_bug.cgi?id=37119 Bug ID: 37119 Summary: PVS-Studio Error: Cascade of conditional operators Product: Wine Version: 1.7.22 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: lukebenes(a)hotmail.com V517 The use of 'if (A) {...} else if (A) {...}' pattern was detected. There is a probability of logical error presence. Check lines: 1754, 1765. msi.c 1754 if (!strcmpW( szProperty, INSTALLPROPERTY_LOCALPACKAGEW )) //<== { ... } else if (!strcmpW( szProperty, INSTALLPROPERTY_INSTALLDATEW )) { ... } else if (!strcmpW( szProperty, INSTALLPROPERTY_LOCALPACKAGEW )) //<== { ... } else if (!strcmpW( szProperty, INSTALLPROPERTY_UNINSTALLABLEW ) || !strcmpW( szProperty, INSTALLPROPERTY_PATCHSTATEW ) || !strcmpW( szProperty, INSTALLPROPERTY_DISPLAYNAMEW ) || !strcmpW( szProperty, INSTALLPROPERTY_MOREINFOURLW )) { ... } else { ... } If identical conditions are checked in a cascade of conditional operators, those conditions never get control. Perhaps there is a typo in the constant INSTALLPROPERTY_* passed to this fragment. details on the V517 error here: http://www.viva64.com/en/d/0106/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 5

[Bug 36800] New: XMP failed to start, services.exe has crashed
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

http://bugs.winehq.org/show_bug.cgi?id=36800 Bug ID: 36800 Summary: XMP failed to start, services.exe has crashed Product: Wine Version: 1.7.20 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: penghao(a)linuxdeepin.com download from: http://xmp.down.sandai.net/kankan/XMPSetup_4.9.16.2203-www.exe after install, start XMP from: ~/.wine/drive_c/Program Files/Thunder Network/Xmp/Program/ with: wine XMP.exe then the services.exe report crashed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 17

[Bug 36253] New: Add Nvidia gtx 750 support to the device list
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

http://bugs.winehq.org/show_bug.cgi?id=36253 Bug ID: 36253 Summary: Add Nvidia gtx 750 support to the device list Product: Wine Version: 1.7.17 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: dyvazar(a)numericable.fr Get that card on my 32 bits system, and hope for it to be recognized and used. http://www.gigabyte.com/products/product-page.aspx?pid=4947#sp -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 5

[Bug 36100] New: Civilization III Complete (Steam) crashes if OSMesa support was compiled in
by wine-bugs＠winehq.org 01 Jan '15

01 Jan '15

http://bugs.winehq.org/show_bug.cgi?id=36100 Bug ID: 36100 Summary: Civilization III Complete (Steam) crashes if OSMesa support was compiled in Product: Wine Version: 1.7.17 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: gyebro69(a)gmail.com Created attachment 48286 --> http://bugs.winehq.org/attachment.cgi?id=48286 terminal output This bug was addressed in http://forum.winehq.org/viewtopic.php?f=8&t=21105 some time ago. The game starts fine and a new game can be started, but it crashes usually in a few minutes. I believe it has something to do with scrolling the map in the game: if you move the mouse pointer near the edges of the screen the crash occurs in a matter of seconds. If I compile Wine without OSMesa support then the crash doesn't occur. Fedora 20 x86 mesa-libOSMesa-10.1-6.20140305.fc20.i686 Nvidia binary drivers 337.12 X.Org X Server 1.14.4 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 4

[Bug 1408] Crimson Trial instaler starts, but won't complete the installation becouse of missing ADPCM
by wine-bugs＠winehq.org 31 Dec '14

31 Dec '14

https://bugs.winehq.org/show_bug.cgi?id=1408 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |focht(a)gmx.net Component|-unknown |tools Version|unspecified |20030618 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

1 0