[Bug 27248] New: cannot unpack *.crx (extensions or themes) in Chrome

http://bugs.winehq.org/show_bug.cgi?id=27248

Summary: cannot unpack *.crx (extensions or themes) in Chrome
           Product: Wine
           Version: 1.3.20
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=34835)
 --> (http://bugs.winehq.org/attachment.cgi?id=34835)
log: unpack crx file with chrome

1. Download the Chrome installer from
http://www.google.com/chrome?platform=win&hl=en

2. make sure bug 14864 is fixed, install Chrome:
wine-1.3.20-230-g456e48e

$ wine ChromeSetup.exe

3. start chrome with --no-sandbox 
==
without --no-sandbox, chrome can not open any web site. is this a know bug? or
should I submit a separate bug?
==

$ wine chrome.exe --no-sandbox

4. open
https://chrome.google.com/webstore/detail/faamegfkmmdnkpdipihbehibalfjpmpk ,
install the theme package .
then there will be a message box says:

Can not unpack extension.  To safely unpack an extension, there must be a path
to your profile directory that starts with a drive letter and does not contain
a junction, mount point, or symlink.  No such path exists for your profile.

full log is attached .

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome

http://bugs.winehq.org/show_bug.cgi?id=27248

fracting  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, source
                URL|                            |https://chrome.google.com/w
                   |                            |ebstore/detail/faamegfkmmdn
                   |                            |kpdipihbehibalfjpmpk

--- Comment #1 from fracting  2011-05-21 14:39:28 CDT ---
maybe K32GetMappedFileNameW is needed:

fixme:file:K32GetMappedFileNameW (0xffffffff, 0x1080000, 0xfde4a4, 270): stub
[192:197:1502177:ERROR:sandboxed_extension_unpacker.cc(113)] Could not get the
normalized path of C:\users\fracting\Local Settings\Application
Data\Google\Chrome\User Data\Temp\scoped_dir_13893\extension_1_0.crx
[192:194:1502415:ERROR:extension_error_reporter.cc(55)] Extension error: Can
not unpack extension.  To safely unpack an extension, there must be a path to
your profile directory that starts with a drive letter and does not contain a
junction, mount point, or symlink.  No such path exists for your profile.
fixme:file:ReplaceFileW Ignoring flags 2

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #2 from Austin English  2011-05-21 17:48:41 CDT ---
(In reply to comment #0)
> - start chrome with --no-sandbox
>
> ==
> without --no-sandbox, chrome can not open any web site. is this a know bug? or
> should I submit a separate bug?

bug 21232

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome

http://bugs.winehq.org/show_bug.cgi?id=27248

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #3 from Austin English  2011-05-21 18:28:01 CDT ---
Confirming.

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #4 from fracting  2011-08-08 14:04:47 CDT ---
Hello , --single-process work around for this.

$ wine chrome --no-sandbox --single-process

Reference:
http://code.google.com/p/chromium/issues/detail?id=59111#c19
Issue 59111:    All Extensions Fail to Install with "Can not unpack extension.
..."

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome unless --single-process is used

http://bugs.winehq.org/show_bug.cgi?id=27248

fracting  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|cannot unpack *.crx         |cannot unpack *.crx
                   |(extensions or themes) in   |(extensions or themes) in
                   |Chrome                      |Chrome unless
                   |                            |--single-process is used

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome unless --single-process is used

http://bugs.winehq.org/show_bug.cgi?id=27248

fracting  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |kernel32

--- Comment #5 from fracting  2011-12-23 01:09:13 CST ---
In chrome/browser/extensions/sandboxed_extension_unpacker.cc, the Chrome
extension unpacker start unpack a crx file with this function:

SandboxedExtensionUnpacker::Start()

This functioin use NormalizeFilePath to get the normalized path of the
extension crx file. If NormalizeFilePath return FALSE, the unpacking will fail.
--- snip ---
if (!file_util::NormalizeFilePath(temp_crx_path, &link_free_crx_path)) {
      LOG(ERROR) << "Could not get the normalized path of "
                 << temp_crx_path.value();
--- snip ---

In base/file_util_win.cc:
In NormalizeFilePath():
--- snip ---
if (!NormalizeToNativeFilePath(path, &mapped_file))
    return false
--- snip ---

In NormalizeToNativeFilePath():
--- snip ---
if (::GetMappedFileNameW(cp, file_view, mapped_file_path, kMaxPathLength)) {
    *nt_path = FilePath(mapped_file_path);
    success = true;
  }
...
return success;
--- snip ---

Since wine's GetMappedFileNameW is forwarding to K32GetMappedFileNameW, which
is a stub and always return 0, NormalizeFilePath will always return FALSE.
--- snip ---
DWORD WINAPI K32GetMappedFileNameW(HANDLE process, LPVOID lpv, LPWSTR
file_name, DWORD size)
{
    FIXME_(file)("(%p, %p, %p, %d): stub\n", process, lpv, file_name, size);

if (file_name && size)
        file_name[0] = '\0';

return 0;
}
--- snip ---

So, this bug needs K32GetMappedFileNameW to be implemented.

GetMappedFileNameW is implemented in native psapi.dll, but there is no
"psapi.dll" component for wine bugzilla, so I'm setting the component to
kernel32, which is where K32GetMappedFileNameW placed.

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome unless --single-process is used

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #6 from fracting  2011-12-23 01:10:06 CST ---
Native psapi.dll can not workaround for this issue, it cause a crash, Google
Chrome can not start at all with native psapi

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome unless --single-process is used

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #7 from fracting  2011-12-23 01:12:11 CST ---
Simply change the return value of K32GetMappedFileNameW won't work around this
bug as well. Chrome unpacker will failed in the later check.

Will someone implement K32GetMappedFileNameW?

[Bug 27248] cannot unpack *.crx (extensions or themes) in Chrome unless --single-process is used

http://bugs.winehq.org/show_bug.cgi?id=27248

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|kernel32                    |-unknown

--- Comment #8 from Dmitry Timoshkov  2011-12-23 02:01:27 CST ---
GetMappedFileNameW should be implemented as a wrapper to 
ntdll.NtQueryVirtualMemory(MemorySectionName) which is a stub currently.

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

http://bugs.winehq.org/show_bug.cgi?id=27248

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
          Component|-unknown                    |kernel32
            Summary|cannot unpack *.crx         |Multiple applications need
                   |(extensions or themes) in   |K32GetMappedFileNameW
                   |Chrome unless               |implemented (Chrome
                   |--single-process is used    |Browser, Yumina the
                   |                            |Ethereal)

--- Comment #9 from Anastasius Focht  2013-09-01 08:15:39 CDT ---
Hello folks,

refining summary to collect applications needing this here (dupes).
Obviously still present as of wine-1.7.1

Regards

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

http://bugs.winehq.org/show_bug.cgi?id=27248

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
                   |                            |m

--- Comment #10 from Anastasius Focht  2013-09-01 08:20:58 CDT ---
*** Bug 34253 has been marked as a duplicate of this bug. ***

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #11 from ~J~  2013-09-30 02:21:05 CDT ---
I noticed ReactOS have code for NtQueryVirtualMemory(MemorySectionName) which
can be seen here :
http://doxygen.reactos.org/d2/d2c/virtual_8c_a39ad5f8f1a5214f4874171695ab2bd...

Perhaps this can be adapted into WINE?

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

http://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #12 from Dmitry Timoshkov  2013-09-30 06:52:18 CDT ---
(In reply to comment #11)
> I noticed ReactOS have code for NtQueryVirtualMemory(MemorySectionName) which
> can be seen here :
> http://doxygen.reactos.org/d2/d2c/virtual_8c_a39ad5f8f1a5214f4874171695ab2bd...
>
> Perhaps this can be adapted into WINE?

Please don't post urls to ReactOS code, it won't be used in Wine.

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #13 from Anastasius Focht  ---
Hello folks,

obviously still present.

$ wine --version
wine-1.7.44

Regards

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented (Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

Roger Davis  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #14 from Roger Davis  ---
I have a strong need to be able to use activex to run a security CCTV program
in Linux.

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented ( Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

Wyrex  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #15 from Wyrex  ---
Still happens in wine 2.0-rc3.

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented ( Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

--- Comment #16 from ~J~  ---
Yumina the Ethereal starts and works well for me in Staging 3.6

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented ( Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |STAGED
         Depends on|                            |23999
    Staged patchset|                            |https://github.com/wine-sta
                   |                            |ging/wine-staging/tree/mast
                   |                            |er/patches/ntdll-NtQueryVir
                   |                            |tualMemory

--- Comment #17 from Anastasius Focht  ---
Hello folks,

--- quote ---
Yumina the Ethereal starts and works well for me in Staging 3.6
--- quote ---

that's expected because Wine-Staging implements
'kernel32.K32GetMappedFileName()' on top of 'ntdll.NtQueryVirtualMemory(
MemorySectionName, ..)'

https://github.com/wine-staging/wine-staging/tree/master/patches/ntdll-NtQue...

https://github.com/wine-staging/wine-staging/blob/master/patches/ntdll-NtQue...

Marking this one 'staged', with dependency to bug 23999 (the app there only
require the ntdll part).

$ wine --version
wine-3.7

Regards

[Bug 27248] Multiple applications need K32GetMappedFileNameW implemented ( Chrome Browser, Yumina the Ethereal)

https://bugs.winehq.org/show_bug.cgi?id=27248

tokktokk  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

[Bug 27248] Multiple games and applications need K32GetMappedFileNameW implemented (Riot Games' Valorant, Chrome Browser, Yumina the Ethereal)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=27248

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|https://chrome.google.com/w |https://web.archive.org/web
                   |ebstore/detail/faamegfkmmdn |/20200411110856/https://val
                   |kpdipihbehibalfjpmpk        |orant.secure.dyn.riotcdn.ne
                   |                            |t/channels/public/x/install
                   |                            |er/current/live.live.eu.exe
            Summary|Multiple applications need  |Multiple games and
                   |K32GetMappedFileNameW       |applications need
                   |implemented (Chrome         |K32GetMappedFileNameW
                   |Browser, Yumina the         |implemented (Riot Games'
                   |Ethereal)                   |Valorant, Chrome Browser,
                   |                            |Yumina the Ethereal)

--- Comment #18 from Anastasius Focht  ---
Hello folks,

revisiting, obviously still present.

This is also needed for 'Valorant' from Riot Games. Valorant is some overhyped
CS:GO / Overwatch hybrid, currently in closed beta.

https://beta.playvalorant.com

https://web.archive.org/web/20200411110856/https://valorant.secure.dyn.riotc...

I'm replacing the broken download link in this ticket (no mirror/snapshot) with
 Valorant bootstrap installer snapshot from Internet Archive.

The installer displays generic errors during bootstrapping. There is also a
nice window dragging bug which causes the window to be moved without being able
to click the exit button - different issue.

--- snip ---
Couldn't update. Something went wrong with the update, even after repairing.
Please contact our player support wizards so they can work their magic to get
you a fix.
--- snip ---

I seriously doubt that in case of Wine, but well ;-)

Relevant part of trace:

--- snip ---
...
0078:Call KERNEL32.CreateProcessW(055b4da8 L"c:/Riot Games/Riot
Client/RiotClientServices.exe",055b2150 L""c:/Riot Games/Riot
Client/RiotClientServices.exe" --agent --agent-bootstrap
--riotclient-app-port=35391 --riotclient-auth-token=wer-das-liest-ist-doof
--app-root=Z:/home/focht/Downloads "--data-root=C:/ProgramData/Riot
Games/Metadata" "--update-root=C:/ProgramData/Riot
Games/Metadat"...,00000000,00000000,00000000,00000008,00000000,055b4f80
L"c:/Riot Games/Riot Client",05c8f760,05c8f7a4) ret=005520d6 
...
0078:trace:process:CreateProcessInternalW started process pid 0079 tid 007a 
...
0078:Ret  KERNEL32.CreateProcessW() retval=00000001 ret=005520d6 
...
008a:Call KERNEL32.OutputDebugStringA(0fb3b6d8
"RiotClientServices.exe(0000008A): ALWAYS| Current install at 'c:/Riot
Games/Riot Client' at release 'AFB48F9F3F3CD2E1', but release
'55F84BB2B4A7A864' available; updating\n") ret=0053b885 
...
008a:Call KERNEL32.OutputDebugStringA(0fb3b96c
"RiotClientServices.exe(0000008A): ALWAYS| Starting launcher thread
'UpdateThread'.\n") ret=0053b885 
...
008d:Call KERNEL32.GetFileAttributesW(05b3b720 L"C:/ProgramData/Riot
Games/Metadata/Riot Client/Riot Client.ok") ret=005d7ad0
...
008d:Ret  KERNEL32.GetFileAttributesW() retval=00000020 ret=005d7ad0
008d:Call KERNEL32.DeleteFileW(05b3b720 L"C:/ProgramData/Riot
Games/Metadata/Riot Client/Riot Client.ok") ret=005d7ba2
...
008d:Ret  KERNEL32.DeleteFileW() retval=00000001 ret=005d7ba2
...
008d:Call psapi.GetMappedFileNameW(ffffffff,00400000,0fe6f7fc,00000104)
ret=0047cda6
008d:Call KERNEL32.K32GetMappedFileNameW(ffffffff,00400000,0fe6f7fc,00000104)
ret=7bc9de14
008d:fixme:file:K32GetMappedFileNameW (0xffffffff, 0x400000, 0xfe6f7fc, 260):
stub
008d:Ret  KERNEL32.K32GetMappedFileNameW() retval=00000000 ret=7bc9de14
008d:Ret  psapi.GetMappedFileNameW() retval=00000000 ret=0047cda6
...
008d:Call KERNEL32.GetLastError() ret=004c3cd9
008d:Ret  KERNEL32.GetLastError() retval=00000000 ret=004c3cd9
008d:Call
KERNEL32.FormatMessageA(00001200,00000000,00000000,00000409,0fe6efa4,00000100,00000000)
ret=0052d752
008d:Ret  KERNEL32.FormatMessageA() retval=0000000a ret=0052d752 
...
--- snip ---

Another symptom of this problem is the following error message, after the
bootstrapper has restarted:

--- snip ---
The drive looks full. There is not enough disk space to complete this action.
--- snip ---

Sure, sure, buddy ... pigs in the air! ;-)

Not sure why the patches from this bug 27248, bug 23999, bug 37487 are still
staged since they have been included in regular Wine-Staging releases for two
years or even longer now. The majority of the gamer user base uses Wine-Staging
or derived projects anyway hence they don't notice such issues.

With the patches and their dependencies applied, one arrives at the login page.
It doesn't get further after login unless you own a closed beta-key. I'm not
going to waste time watching boring twitch streams, waiting for those unlikely
key drops. I've read that people paid up to 150 bucks for those closed beta
keys - that's plain stupid.

$ sha1sum Install\ VALORANT.exe 
92c28e21122d5272deba5d826c161def55934c91  Install VALORANT.exe

$ du -sh Install\ VALORANT.exe 
66M    Install VALORANT.exe

$ wine --version
wine-5.6

Regards

[Bug 27248] Multiple games and applications need K32GetMappedFileNameW implemented (Riot Games' Valorant, Chrome Browser, Yumina the Ethereal)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=27248
Bug 27248 depends on bug 23999, which changed state.

Bug 23999 Summary: Multiple applications with DRM schemes need NtQueryVirtualMemory 'MemorySectionName' info class (Crossfire HGWC, EMS SQL Manager 2010 Lite for PostgreSQL v.4.7.08, Cygwin/MSYS2, Knight Online client)
https://bugs.winehq.org/show_bug.cgi?id=23999

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|STAGED                      |RESOLVED
         Resolution|---                         |FIXED

[Bug 27248] Multiple games and applications need K32GetMappedFileNameW implemented (Riot Games' Valorant, Chrome Browser, Yumina the Ethereal)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=27248

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |bd0a3c1a59db9e75500db6df0f3
                   |                            |598c981abf44e
             Status|STAGED                      |RESOLVED

--- Comment #19 from Anastasius Focht  ---
Hello folks,

this is fixed by commit
https://source.winehq.org/git/wine.git/commitdiff/bd0a3c1a59db9e75500db6df0f...
("kernelbase: Implement GetMappedFileNameA/W.").

Thanks Alexandre

$ wine --version
wine-6.1-315-gb922b5aeef1

Regards

[Bug 27248] Multiple games and applications need K32GetMappedFileNameW implemented (Riot Games' Valorant, Chrome Browser, Yumina the Ethereal)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=27248

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #20 from Alexandre Julliard  ---
Closing bugs fixed in 6.2.