On 04/04/2012 11:03 AM, Christophe Fergeau wrote:
The Firefox plugin exposes the UsbAutoShare property and the SetUsbFilter
method that were not wired until now. This commit propagates their value to
the nsPluginInstance object and send them through the controller socket.
---
  SpiceXPI/src/plugin/plugin.cpp |   18 ++++++++----------
  SpiceXPI/src/plugin/plugin.h   |    2 ++
  spice-protocol                 |    2 +-
  3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/SpiceXPI/src/plugin/plugin.cpp b/SpiceXPI/src/plugin/plugin.cpp
index 3459869..1f42214 100644
--- a/SpiceXPI/src/plugin/plugin.cpp
+++ b/SpiceXPI/src/plugin/plugin.cpp
@@ -194,6 +194,7 @@ nsPluginInstance::nsPluginInstance(NPP aInstance):
      m_admin_console(PR_FALSE),
      m_no_taskmgr_execution(PR_FALSE),
      m_send_ctrlaltdel(PR_TRUE),
+    m_usb_auto_share(PR_TRUE),
      m_scriptable_peer(NULL)
  {
      // create temporary directory in /tmp
@@ -231,6 +232,7 @@ NPBool nsPluginInstance::init(NPWindow *aWindow)
      m_number_of_monitors.clear();
      m_guest_host_name.clear();
      m_hot_keys.clear();
+    m_usb_filter.clear();
      m_language.clear();

The default is an empty string.
How does the client behave with such a filter string? Refuse redir for all devices ? Alternatively we can set the default to a simple rule (allow-all/allow-all-no-hid/allow-only-usb-disks/refuse-all/other)
(if needed, this can be done in a following patch)

_______________________________________________
Spice-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to