[SSSD] [PATCH] sudo: print message if old protocol is used

Jakub Hrozek jhrozek at redhat.com
Thu Nov 29 12:36:42 UTC 2012


On Thu, Nov 15, 2012 at 05:54:26PM +0100, Pavel Březina wrote:
> >Nitpicking, but for this kind of situations I think a switch/case
> >statement would be much more clear (also avoids adding a variable as you
> >use it once):
> >
> >switch(cli_ctx->cli_protocol_version->version) {
> >case 0:
> >     DEBUG(..);
> >     break;
> >case SSS_SUDO_PROTOCOL_VERSION:
> >     break;
> >default:
> >     DEBUG("invalid"...);
> >     break;
> >}
> >
> >Simo.
> >
> 
> OK. New patch is attached.
> 

Ack



More information about the sssd-devel mailing list