[SSSD] [PATCH] Minor fixes in various places
Jan Zelený
jzeleny at redhat.com
Tue Aug 31 11:57:19 UTC 2010
Stephen Gallagher <sgallagh at redhat.com> wrote:
> >> 0005-Deleted-some-needless-assignments.patch
> >> Ticket: #582
> >
> > I think we should check for potential errors from sdap_id_op_done()
> > calls instead of removing the assignments altogether.
>
> I agree with Jakub. These assignments shouldn't be removed, they should
> be tested for failures.
Sorry for an extra email, forgot to mention this in the previous one. I don't
think this is the case. The result of operation is subsequently checked (it is
stored also in dp_error variable). We can check ret value as well as dp_error,
but I don't think it will bring any code improvement. The same applies for
call of sysdb_attrs_get_string() - its return value isn't checked, but the
content of "value" variable is tested.
Jan
More information about the sssd-devel
mailing list