Inshell Security Advisory http://www.inshell.net 1. ADVISORY INFORMATION ----------------------- Product: WinAmp Vendor URL: www.winamp.com Type: Stack-based Buffer Overflow [CWE-121] Date found: 2013-06-05 Date published: 2013-07-01 CVSSv2 Score: Bug #1: 7,5 (AV:N/AC:L/Au:N/C:P/I:P/A:P) Bug #2: 3,7 (AV:L/AC:H/Au:N/C:P/I:P/A:P) CVE: CVE-2013-4694 2. CREDITS ---------- These vulnerabilities were discovered and researched by Julien Ahrens from Inshell Security. 3. VERSIONS AFFECTED -------------------- WinAmp v5.63, older versions may be affected too. 4. VULNERABILITY DESCRIPTION (BUG #1) ------------------------------------- The application loads the directories in %PROGRAMFILES%\WinAmp\Skins on startup to determine the skins that have been installed and to list them in the application menu point "Skins" and in the Skins Browser. But the application does not properly validate the length of the directory name before passing it as argument to a lstrcpynW call in the library gen_jumpex.dll, which leads to a buffer overflow condition with possible code execution. This flaw is also exploitable via the %APPDATA%\WinAmp\winamp.ini. The application loads the contents on startup, but does not properly validate the length of the string loaded from the "skin" key before passing it as an argument to the same lstrcpynW call in the library gen_jumpex.dll, which leads to the same buffer overflow condition. An attacker either needs to trick the victim to download and apply an arbitrary skin package in order to exploit the vulnerability or to copy an arbitrary winamp.ini into the %APPDATA%\WinAmp directory. Successful exploits can allow attackers to execute arbitrary code with the privileges of the user running the application. Failed exploits will result in a denial-of-service condition. 4. VULNERABILITY DESCRIPTION (BUG #2) ------------------------------------- The application loads the string of the GUI "Search" field from the "WinAmp Library" when entered by a user and after switching to another menu point, but does not properly validate the length of the string before passing it as an argument to a GetDlgItemTextW call in the library ml_local.dll, which leads to a buffer overflow condition with possible code execution. An attacker needs local access to the client in order to exploit the vulnerability. Successful exploits can allow attackers to execute arbitrary code with the privileges of the user running the application. Failed exploits will result in a denial-of-service condition. 5. PROOF-OF-CONCEPT (DEBUG) (Bug #1) ------------------------------------ Registers: EAX 3B3C08EB ECX 7C80BAFC kernel32.7C80BAFC EDX 00430010 winamp.00430010 EBX 0000007E ESP 00C1F290 UNICODE "CCCCCCCCCCCCCCCCCCCCCCCCCCCC" EBP 00430043 winamp.00430043 ESI 001961E8 EDI 0000060B EIP 00430060 winamp.00430060 C 0 ES 0023 32bit 0(FFFFFFFF) P 1 CS 001B 32bit 0(FFFFFFFF) A 0 SS 0023 32bit 0(FFFFFFFF) Z 1 DS 0023 32bit 0(FFFFFFFF) S 0 FS 003B 32bit 7FFDE000(FFF) T 0 GS 0000 NULL D 0 O 0 LastErr ERROR_INVALID_WINDOW_HANDLE (00000578) EFL 00010246 (NO,NB,E,BE,NS,PE,GE,LE) ST0 empty +NaN ST1 empty ST2 empty ST3 empty ST4 empty ST5 empty ST6 empty ST7 empty 3 2 1 0 E S P U O Z D I FST 0120 Cond 0 0 0 1 Err 0 0 1 0 0 0 0 0 (LT) FCW 027F Prec NEAR,53 Mask 1 1 1 1 1 1 Stackview: ESP-20 > 00430043 CC winamp.00430043 ESP-1C > 0043004B KC winamp.0043004B ESP-18 > 7C80BAFC kernel32.7C80BAFC ESP-14 > 00430043 CC winamp.00430043 ESP-10 > 00430043 CC winamp.00430043 ESP-C > 00430043 CC winamp.00430043 ESP-8 > 00430043 CC winamp.00430043 ESP-4 > 00430043 CC winamp.00430043 ESP ==> > 00430043 CC winamp.00430043 ESP+4 > 00430043 CC winamp.00430043 ESP+8 > 00430043 CC winamp.00430043 ESP+C > 00430043 CC winamp.00430043 ESP+10 > 00430043 CC winamp.00430043 ESP+14 > 00430043 CC winamp.00430043 ESP+18 > 00430043 CC winamp.00430043 ESP+1C > 00430043 CC winamp.00430043 ESP+20 > 00430043 CC winamp.00430043 Vulnerable code part: .text:1001A5B8 push eax ; lpString2 .text:1001A5B9 lea eax, [ebp+String1] .text:1001A5BF push eax ; lpString1 .text:1001A5C0 call ds:lstrcpynW .text:1001A5C6 cmp word ptr [ebp+wParam], si .text:1001A5CD jnz short loc_1001A5E2 .text:1001A5CF mov dword_100310B4, 1 .text:1001A5D9 cmp [ebp+String1], si .text:1001A5E0 jz short loc_1001A5E8 .text:1001A5E2 .text:1001A5E2 loc_1001A5E2: ; CODE XREF: sub_1001A551+7Cj .text:1001A5E2 mov dword_100310B4, esi .text:1001A5E8 .text:1001A5E8 loc_1001A5E8: ; CODE XREF: sub_1001A551+8Fj .text:1001A5E8 pop esi .text:1001A5E9 leave .text:1001A5EA retn .text:1001A5EA sub_1001A551 endp 5. PROOF-OF-CONCEPT (DEBUG) (Bug #2) ------------------------------------ Registers: EAX 00000000 ECX 079A9D68 ml_local.079A9D68 EDX 00380608 EBX 00000000 ESP 00C1E46C UNICODE "CCCCCCCCCCCCCCCCCCCCCCCCCCCCCCCCCCCC" EBP 00430043 winamp.00430043 ESI 00000000 EDI 00000000 EIP 00430043 winamp.00430043 C 0 ES 0023 32bit 0(FFFFFFFF) P 1 CS 001B 32bit 0(FFFFFFFF) A 0 SS 0023 32bit 0(FFFFFFFF) Z 1 DS 0023 32bit 0(FFFFFFFF) S 0 FS 003B 32bit 7FFDE000(FFF) T 0 GS 0000 NULL D 0 O 0 LastErr ERROR_INVALID_WINDOW_HANDLE (00000578) EFL 00010246 (NO,NB,E,BE,NS,PE,GE,LE) ST0 empty ST1 empty ST2 empty ST3 empty ST4 empty ST5 empty ST6 empty ST7 empty 3 2 1 0 E S P U O Z D I FST 0000 Cond 0 0 0 0 Err 0 0 0 0 0 0 0 0 (GT) FCW 027F Prec NEAR,53 Mask 1 1 1 1 1 1 Stackview: ESP-20 > 00430043 CC winamp.00430043 ESP-1C > 00430043 CC winamp.00430043 ESP-18 > 00430043 CC winamp.00430043 ESP-14 > 00430043 CC winamp.00430043 ESP-10 > 00430043 CC winamp.00430043 ESP-C > 00430043 CC winamp.00430043 ESP-8 > 00430043 CC winamp.00430043 ESP-4 > 00430043 CC winamp.00430043 ESP ==> > 00430043 CC winamp.00430043 ESP+4 > 00430043 CC winamp.00430043 ESP+8 > 00430043 CC winamp.00430043 ESP+C > 00430043 CC winamp.00430043 ESP+10 > 00430043 CC winamp.00430043 ESP+14 > 00430043 CC winamp.00430043 ESP+18 > 00430043 CC winamp.00430043 ESP+1C > 00430043 CC winamp.00430043 ESP+20 > 00430043 CC winamp.00430043 Vulnerable code part: .text:07990871 lea eax, [ebp+WideCharStr] .text:07990877 push eax ; lpString .text:07990878 push 3EEh ; nIDDlgItem .text:0799087D push [ebp+hDlg] ; hDlg .text:07990880 call ds:GetDlgItemTextW .text:07990886 lea eax, [ebp+WideCharStr] [...] .text:0799097C mov dword_79A9D68, eax .text:07990981 mov dword_79A9D70, eax .text:07990986 mov dword_79A9D6C, eax .text:0799098B mov dword_79ACB54, eax .text:07990990 pop ebx .text:07990991 leave .text:07990992 retn 6. SOLUTION ----------- Update to latest version v5.64 or newer. 7. REPORT TIMELINE ------------------ 2013-06-05: Discovery of the vulnerability 2013-06-06: Vendor acknowledgement of the issue 2013-06-11: Vendor already fixed this issue in v5.7 Beta build 3403 2013-06-12: Confirmation that the issue is fixed 2013-06-19: Vendor releases v5.64 which includes the fix 2013-07-01: Coordinated Disclosure 8. REFERENCES ------------- http://security.inshell.net http://forums.winamp.com/showthread.php?t=364291