[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