Re: [spice-gtk][PATCH 4/4] ppc: Fix auth endianess

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 18, 2015 at 03:11:08PM +0200, Fabiano Fidêncio wrote:
> ---
>  gtk/spice-channel.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c
> index 0da377b..273cf3a 100644
> --- a/gtk/spice-channel.c
> +++ b/gtk/spice-channel.c
> @@ -1750,14 +1750,14 @@ static gboolean spice_channel_recv_link_msg(SpiceChannel *channel)
>  #if HAVE_SASL
>          if (spice_channel_test_common_capability(channel, SPICE_COMMON_CAP_AUTH_SASL)) {
>              CHANNEL_DEBUG(channel, "Choosing SASL mechanism");
> -            auth.auth_mechanism = SPICE_COMMON_CAP_AUTH_SASL;
> +            auth.auth_mechanism = GUINT32_TO_LE(SPICE_COMMON_CAP_AUTH_SASL);
>              spice_channel_write(channel, &auth, sizeof(auth));
>              if (!spice_channel_perform_auth_sasl(channel))
>                  return FALSE;
>          } else
>  #endif
>          if (spice_channel_test_common_capability(channel, SPICE_COMMON_CAP_AUTH_SPICE)) {
> -            auth.auth_mechanism = SPICE_COMMON_CAP_AUTH_SPICE;
> +            auth.auth_mechanism = GUINT32_TO_LE(SPICE_COMMON_CAP_AUTH_SPICE);
>              spice_channel_write(channel, &auth, sizeof(auth));
>              spice_channel_send_spice_ticket(channel);
>          } else {
> -- 

ACK, though my feeling by looking for spice_channel_read() occurrences
throughout this file is that there may be some more work to do for SASL
auth to work.

Christophe

Attachment: pgpNoLhLsx5PO.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]