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