Re: [SSSD] [PATCH] SBUS: Fir error handling after closing container

[Lukas Slebodnik]

On (12/10/14 16:54), Jakub Hrozek wrote:
> On Sat, Oct 11, 2014 at 11:43:28PM +0200, Lukas Slebodnik wrote:
> > ehlo,
> >
> > If function dbus_message_iter_close_container fail the return variable ret will
> > be set to EINVAL, but function will not be immediately terminated.
> > "goto done" was missing.
> >
> > simple patch is attached.
> >
> > LS
>
> Can you also add goto done to the next EINVAL handler and ret=EOK just
> before the done: label? That way the code will also be safe against
> future additions.

Sure,

updated patch is attached.

LS