Bug 963835

Summary: virt-viewer with smartcard crashes in Enhanced Protected Mode of IE10
Product: Red Hat Enterprise Virtualization Manager Reporter: David Jaša <djasa>
Component: mingw-virt-viewerAssignee: Marc-Andre Lureau <marcandre.lureau>
Status: CLOSED ERRATA QA Contact: Desktop QE <desktop-qa-list>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.2.0CC: acathrow, cfergeau, cpelland, dblechte, djasa, iheim, jkt, marcandre.lureau, pvine, Rhev-m-bugs
Target Milestone: ---Keywords: ZStream
Target Release: 3.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: mingw-qemu-1.4.0-3.el6ev Doc Type: Bug Fix
Doc Text:
Previously, trying to use smartcard passthrough in Enhanced Protected Mode of Internet Explorer 10 caused virt-viewer to crash. Now, the code has been updated so that smartcards can be successfully used in Enhanced Protected Mode of Internet Explorer 10.
Story Points: ---
Clone Of:
: 973733 (view as bug list) Environment:
Last Closed: 2014-01-21 14:45:39 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 960709, 973733    
Attachments:
Description Flags
deps.txt
none
screenshot none

Description David Jaša 2013-05-16 16:31:21 UTC
Description of problem:
spice-activex-win + mingw-virt-viewer are not compatible with Enhanced Protected Mode of IE10

This issue has potential of blocking spice adoption in environments with policies that enforce EPM setting.

Version-Release number of selected component (if applicable):
spicex-3.2-7
mingw-virt-viewer-0.5.3-25

How reproducible:
always

Steps to Reproduce:
0. have a Windows x64 with IE10
1. go to Internet Options --> Advanced, enable Security/"Enable Enhanced Protected Mode*"
2. go to User Portal, open the console
3.
  
Actual results:
mingw-virt-viewer crashes (gets killed?) few seconds after its start up

Expected results:
mingw-virt-viewer starts up normally

Additional info:

Comment 1 Marc-Andre Lureau 2013-05-16 16:46:03 UTC
is this protected mode enable by default?

Comment 2 David Jaša 2013-05-17 08:49:00 UTC
(In reply to comment #1)
> is this protected mode enable by default?

It is when IE10 runs on Win8 with Metro UI. In other cases, it is not, but then it runs in 32b mode only (see also https://bugzilla.redhat.com/show_bug.cgi?id=918656#c15 ).

Comment 6 David Jaša 2013-05-17 12:29:12 UTC
Attached dump & associated files of m-r-v crash.

Comment 7 David Jaša 2013-05-17 12:30:38 UTC
Created attachment 749348 [details]
deps.txt

the dump is from mingw-virt-viewer-0.5.3-25, deps.txt file is attached

Comment 8 Marc-Andre Lureau 2013-05-21 19:00:45 UTC
I can't reproduce, using win7 x64, IE10.0.9200.16576 against RHEVM3.1 spice.lab.eng.brq and mingw-viewer -0.5.3-25, after enabled EPM and rebooting (2 times, first time didn't save the setting!)

Is this only a win8-IE-metro bug? metro isn't meant to work with plugins/activex.


So another reason to stop using activex (and deploying):

"The new Windows UI browsing experience doesn't support Microsoft ActiveX or any other binary extensibility."

http://msdn.microsoft.com/en-us/library/ie/hh920753%28v=vs.85%29.aspx

Comment 9 David Jaša 2013-05-22 10:00:16 UTC
(In reply to Marc-Andre Lureau from comment #8)
> I can't reproduce, using win7 x64, IE10.0.9200.16576 against RHEVM3.1
> spice.lab.eng.brq and mingw-viewer -0.5.3-25, after enabled EPM and
> rebooting (2 times, first time didn't save the setting!)

I can't reproduce either against 3.1 but it's 100% reproducible against 3.2, the mingw-virt-viewer version does not matter. That suggests that the issue is in spice-x or in User portal

> 
> Is this only a win8-IE-metro bug? metro isn't meant to work with
> plugins/activex.
> 

It's the same win/ie versions, just UP is different. We could bisect it further by installing newer cabs on 3.1 portal.

> 
> So another reason to stop using activex (and deploying):
> 
> "The new Windows UI browsing experience doesn't support Microsoft ActiveX or
> any other binary extensibility."
> 
> http://msdn.microsoft.com/en-us/library/ie/hh920753%28v=vs.85%29.aspx

Well, there's one big elegant thing on cab/activex that I don't see on it's alternatives without further work: automatic updates of the bits to latest software version with minimal work for corporate systems (allow spicex UUIDs to install automatically without Administrator permission) and with no work needed to upgrade things on user's computers (allowed by PowerUser privileges on XP or UAC prompts from Vista on). The only prerequisite is to keep RHEV-M itself up-to-date.

Mainaining the software up-to-date would mean either developing of yet another updater for spice bits or increase of windows administrator burden for corporate system together with total lack of automatic upgrades on user's personal computers...

(I'm wondering why various free software projects with considerable windows user base didn't join forces on a common updating service - where newly installed package could register it's repository and signing keys and the service could perform updates on it's own...)

Comment 10 Christophe Fergeau 2013-05-22 10:31:00 UTC
(In reply to David Jaša from comment #9)
> Well, there's one big elegant thing on cab/activex that I don't see on it's
> alternatives without further work: automatic updates of the bits to latest
> software version with minimal work for corporate systems (allow spicex UUIDs
> to install automatically without Administrator permission) and with no work
> needed to upgrade things on user's computers (allowed by PowerUser
> privileges on XP or UAC prompts from Vista on). The only prerequisite is to
> keep RHEV-M itself up-to-date.
> 
> Mainaining the software up-to-date would mean either developing of yet
> another updater for spice bits or increase of windows administrator burden
> for corporate system together with total lack of automatic upgrades on
> user's personal computers...
> 
> (I'm wondering why various free software projects with considerable windows
> user base didn't join forces on a common updating service - where newly
> installed package could register it's repository and signing keys and the
> service could perform updates on it's own...)

I suspect Windows admins of big deployment prefers a solution like http://support.microsoft.com/kb/816102 where they can control what gets installed and when. No clue if this comes for free with the msi installer work Marc-André has been doing or if this will need more work.

Comment 11 Marc-Andre Lureau 2013-05-22 11:22:17 UTC
Automatic updates is just a terribly bad idea.

Comment 12 Marc-Andre Lureau 2013-05-22 12:29:00 UTC
(In reply to David Jaša from comment #9)
> (In reply to Marc-Andre Lureau from comment #8)
> > I can't reproduce, using win7 x64, IE10.0.9200.16576 against RHEVM3.1
> > spice.lab.eng.brq and mingw-viewer -0.5.3-25, after enabled EPM and
> > rebooting (2 times, first time didn't save the setting!)
> 
> I can't reproduce either against 3.1 but it's 100% reproducible against 3.2,
> the mingw-virt-viewer version does not matter. That suggests that the issue
> is in spice-x or in User portal

I have no access to 3.2 atm, I will try again.

The mingw-virt-viewer-0.5.3-25 install the same activex version as 3.2. So it seems to be an issue with the portal itself... (the spice-x test html page seems to work too)

> > 
> > Is this only a win8-IE-metro bug? metro isn't meant to work with
> > plugins/activex.
> > 
> 
> It's the same win/ie versions, just UP is different. We could bisect it
> further by installing newer cabs on 3.1 portal.

Sorry to ask again, are you using metro UI?

Comment 13 Marc-Andre Lureau 2013-05-22 12:32:28 UTC
Just tried successfully again rhevm 3.2 instance too.

David, please try to narrow it down, so I can reproduce.

Comment 14 Marc-Andre Lureau 2013-05-22 12:34:14 UTC
(In reply to Christophe Fergeau from comment #10)
> I suspect Windows admins of big deployment prefers a solution like
> http://support.microsoft.com/kb/816102 where they can control what gets
> installed and when. No clue if this comes for free with the msi installer
> work Marc-André has been doing or if this will need more work.

Last I recall, according to WiX documentation it should. Would be nice if someone upstream could test that :)

Comment 15 David Jaša 2013-05-22 12:45:39 UTC
(In reply to Christophe Fergeau from comment #10)
...
> I suspect Windows admins of big deployment prefers a solution like
> http://support.microsoft.com/kb/816102 where they can control what gets
> installed and when. No clue if this comes for free with the msi installer
> work Marc-André has been doing or if this will need more work.

.msi package is a prerequisite for it if I understand the docs correctly. The manual work connected with it is there and that's what I meant by extra burden, to perform any upgrade, they have to check for new versions of the software and perform the steps described on the page you linked. What a difference from just running 'rhevm-upgrade' (or even plain 'yum update') on rhev-m machine...

(In reply to Marc-Andre Lureau from comment #11)
> Automatic updates is just a terribly bad idea.

Maybe in an AD-controlled corporate environment, but certainly not universally. Think about clients outside of AD. How would you deliver a security update to clients outside of AD?

(In reply to Marc-Andre Lureau from comment #12)
...
> Sorry to ask again, are you using metro UI?

I'm using Windows 7, so no metro UI.

(In reply to Marc-Andre Lureau from comment #13)
> Just tried successfully again rhevm 3.2 instance too.
> 
> David, please try to narrow it down, so I can reproduce.

Windows 7 x64 client, fully upgraded. Flip EPM on, go to rhevm32 setup, connect to a console. Confirm activex installation, wait for m-r-v window to come up, observe it crash within 5 seconds...

Comment 16 Marc-Andre Lureau 2013-05-22 13:40:15 UTC
Created attachment 751728 [details]
screenshot

The crash I could reproduce on David machine, seems to be related to smartcard.

Comment 17 Marc-Andre Lureau 2013-05-22 16:11:56 UTC
the bug is a missing NULL at the end of g_build_filename(), in one of the mingw-libcacard patch. It's an early version, and it doesn't exist upstream. moving to post, ack

Comment 18 David Blechter 2013-06-10 22:57:25 UTC
crash fix is the good candidate for z -stream

Comment 23 Charlie 2013-11-28 01:27:49 UTC
This bug is currently attached to errata RHEA-2013:15512. If this change is not to be documented in the text for this errata please either remove it from the errata, set the requires_doc_text flag to 
minus (-), or leave a "Doc Text" value of "--no tech note required" if you do not have permission to alter the flag.

Otherwise to aid in the development of relevant and accurate release documentation, please fill out the "Doc Text" field above with these four (4) pieces of information:

* Cause: What actions or circumstances cause this bug to present.
* Consequence: What happens when the bug presents.
* Fix: What was done to fix the bug.
* Result: What now happens when the actions or circumstances above occur. (NB: this is not the same as 'the bug doesn't present anymore')

Once filled out, please set the "Doc Type" field to the appropriate value for the type of change made and submit your edits to the bug.

For further details on the Cause, Consequence, Fix, Result format please refer to:

https://bugzilla.redhat.com/page.cgi?id=fields.html#cf_release_notes 

Thanks in advance.

Comment 25 errata-xmlrpc 2014-01-21 14:45:39 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2014-0034.html