New issue
Advanced search Search tips

Issue 782571 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 867842
Owner: ----
Closed: Sep 25
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Zip Archiver: crashes by assertion failure with Debug build

Project Member Reported by yamaguchi@chromium.org, Nov 8 2017

Issue description

Chrome Version: 
Chrome OS Version: <From about:version: Platform x.x.x.x>
Chrome OS Platform: <Make/model of computer running Chrome OS>
Network info: <network, encryption type, router model (if known)>

Please specify Cr-* of the system to which this bug/feature applies (add
the label below).

Steps To Reproduce:
(1) Build chrome with is_debug=true option.
(2) Run chrome with --enable-zip-archiver-unpacker option.
(3) Open a zip file in the Files app.


The notification says either the extension is scanning the file, but fail to mount.


JS console
-------------------------
NativeClient: NaCl module crashed
-------------------------

native console
-------------------------
assertion "var_dict.Get(request::key::kOperation).is_int()" failed: file "../../chrome/browser/resources/chromeos/zip_archiver/cpp/module.cc", line 168, function: virtual void NaclArchiveInstance::HandleMessage(const pp::Var &)
** abort() called

** Signal 4 from untrusted code: pc=7ebd0050be80
-------------------------

 
Cc: yamaguchi@chromium.org fukino@chromium.org yawano@chromium.org
Same kind of messages (non-numeric parameter) are received with release build as well, but such messages are interpreted as READ_METADATA(0) command.
https://cs.chromium.org/chromium/src/chrome/browser/resources/chromeos/zip_archiver/cpp/request.h?sq=package:chromium&l=66

Comment 2 by sashab@chromium.org, Feb 22 2018

Labels: CrOS-FilesApp-Zip

Comment 3 by sashab@chromium.org, Feb 28 2018

Labels: -CrOS-FilesApp-Zip CrOSFilesFeature-Zip
Cc: noel@chromium.org
@noel: Didn't you fix this bug?
Mergedinto: 867842
Status: Duplicate (was: Available)
Yeap, in  issue 867842 .

Sign in to add a comment