Bug 1297018
| Summary: | [security] disable strict user checking does not work - users can steal already opened console by other user | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Jiri Belka <jbelka> |
| Component: | Frontend.WebAdmin | Assignee: | jniederm |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Jiri Belka <jbelka> |
| Severity: | high | Docs Contact: | |
| Priority: | high | ||
| Version: | 3.6.1.3 | CC: | bugs, mavital, mgoldboi, michal.skrivanek, sbonazzo, tjelinek |
| Target Milestone: | ovirt-3.6.5 | Keywords: | Regression |
| Target Release: | 3.6.5.1 | Flags: | rule-engine:
ovirt-3.6.z+
rule-engine: blocker+ mgoldboi: planning_ack+ tjelinek: devel_ack+ mavital: testing_ack+ |
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | 3.6.5-1 | Doc Type: | Bug Fix |
| Doc Text: |
Cause:
Consequence:
All users with sufficient permissions to be able to connect to newly started VM were able to steal opened consoles (SPICE/VNC) form any other user.
Fix:
Reconnecting to console (i.e. connecting to console someone else has earlier connected to) is now again controlled by "Override opened console session" permission.
Result:
|
Story Points: | --- |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-04-21 14:39:41 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Virt | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Jiri Belka
2016-01-08 18:15:10 UTC
This bug report has Keywords: Regression or TestBlocker. Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP. Another steps to Reproduce: 1. 1 vm assinged to an user with UserRole so he can open via User Portal 2. user in User Portal opens spice console of the vm 3. another user with UserRole to the same VM 4. the latter user in User Portal opens spice console (the former user has still the spice console opened) 4 = you will probably face https://bugzilla.redhat.com/show_bug.cgi?id=1311060 but iiuc currently the code does not check 'allow_console_reconnect'; if it is False (the default) then behaviour described as 'Additional info' in #0 applies. Thus ordinary users should not be allowed to open console of other users of same VM by default. FYI this was discovered while investigation why opening console with 2nd user causes ctrl-alt-del combo press in the VM guest. Example from guest agent which caused ctrl-alt-del combo press after VmLogonVDSCommand (??) from engine0. Dummy-2::INFO::2016-02-23 11:02:03,187::OVirtAgentLogic::296::root::Received an external command: login... Dummy-2::DEBUG::2016-02-23 11:02:03,187::OVirtAgentLogic::330::root::User log-in (credentials = '\x00\x00\x001ad-w2k12r2.com********\x00') Dummy-2::DEBUG::2016-02-23 11:02:03,187::GuestAgentWin32::261::root::Simulating a secure attention sequence (SAS). Dummy-2::ERROR::2016-02-23 11:02:04,203::GuestAgentWin32::289::root::Error writing credentials to pipe [1/3] (error = 2) Dummy-2::ERROR::2016-02-23 11:02:05,203::GuestAgentWin32::289::root::Error writing credentials to pipe [2/3] (error = 2) Dummy-2::ERROR::2016-02-23 11:02:06,203::GuestAgentWin32::289::root::Error writing credentials to pipe [3/3] (error = 2) Dummy-2::DEBUG::2016-02-23 11:02:06,203::OVirtAgentLogic::270::root::AgentLogicBase::doListen() - in loop before vio.read Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release. looks half-fixed, half-broken. But the "security" problem is resolved, so deferring to 3.6.5 Bugs moved pre-mature to ON_QA since they didn't have target release. Notice that only bugs with a set target release will move to ON_QA. ok, 3.6.5.3-0.1.el6.noarch works as expected, users can't steal their consoles by default (allow_console_reconnect = f). |