On Wed, 2018-08-01 at 16:57 +0200, Victor Toso wrote:
> From: Victor Toso <me@xxxxxxxxxxxxxx>
>
> The lack of UsbDk is logged with messages like:
>
> | GSpice-WARNING **: Error initializing USB support: Entity not
> found [-5]
> | Spice-DEBUG: usb-device-
> manager.c:272:spice_usb_device_manager_init:
> | UsbDk driver is not installed
This warning message doesn't seem to match the code you're changing
below? For example, if usbdk_api is null, I would expect that the
_usbdk_hider_clear() function would abort at the g_return_if_fail(priv-
>usbdk_api) line and produce a critical warning that is something like:
GLib-CRITICAL **: _usbdk_hider_clear: assertion 'priv->usbdi_api !=
NULL' failed
So if you were getting the warning above, I'm not sure how this patch
would change it? Maybe I'm missing something.
>
> We don't need to log a critical for every check on usbdk_api handle
> that might be done. That's really not necessary for
> _usbdk_hider_clear() and debug message on _usbdk_hider_update()
> should
> be enough to track bugs when UsbDk should be working.
>
> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> ---
> src/usb-device-manager.c | 17 ++++++++++-------
> 1 file changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
> index 55bf67e..5e0469b 100644
> --- a/src/usb-device-manager.c
> +++ b/src/usb-device-manager.c
> @@ -1932,13 +1932,13 @@ void _usbdk_hider_clear(SpiceUsbDeviceManager
> *manager)
> {
> SpiceUsbDeviceManagerPrivate *priv = manager->priv;
>
> - g_return_if_fail(priv->usbdk_api != NULL);
> -
> - if (priv->usbdk_hider_handle != NULL) {
> - usbdk_clear_hide_rules(priv->usbdk_api, priv-
> >usbdk_hider_handle);
> - usbdk_close_hider_handle(priv->usbdk_api, priv-
> >usbdk_hider_handle);
> - priv->usbdk_hider_handle = NULL;
> + if (priv->usbdk_api == NULL || priv->usbdk_hider_handle == NULL)
> {
> + return;
> }
> +
> + usbdk_clear_hide_rules(priv->usbdk_api, priv-
> >usbdk_hider_handle);
> + usbdk_close_hider_handle(priv->usbdk_api, priv-
> >usbdk_hider_handle);
> + priv->usbdk_hider_handle = NULL;
> }
>
> static
> @@ -1946,7 +1946,10 @@ void _usbdk_hider_update(SpiceUsbDeviceManager
> *manager)
> {
> SpiceUsbDeviceManagerPrivate *priv = manager->priv;
>
> - g_return_if_fail(priv->usbdk_api != NULL);
> + if (priv->usbdk_api == NULL) {
> + SPICE_DEBUG("UsbDk is not being used, hider setup can't be
> done");
> + return;
> + }
>
> if (priv->auto_connect_filter == NULL) {
> SPICE_DEBUG("No autoredirect rules, no hider setup needed");
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel