[PATCH] winbindd: error handling in rpc_try_lookup_sids3()

[Jeremy Allison]

On Sat, Apr 01, 2017 at 10:53:24PM +0200, Ralph Böhme wrote:
> On Sat, Apr 01, 2017 at 02:21:14PM +0200, Ralph Böhme wrote:
> > On Fri, Mar 31, 2017 at 04:08:36PM -0700, Jeremy Allison wrote:
> > > Also, the function below:
> > > 
> > > rpc_lookup_sids() has exactly the same logic as that
> > > in rpc_try_lookup_sids3(). Should we do the same
> > > change there ? If not, why not ?
> > 
> > Yes, there are more places where it's done wrong: winbindd_lookup_sids() and
> > dcerpc_lsa_lookup_sids_noalloc().
> > 
> > Let's see what happens when I try to fix those as well... :)
> 
> ok, so how about this one? Fixes all lookupsids and lookupsids3 callers, adding
> a new macro NT_STATUS_LOOKUP_ERR. It just passed a private autobuild.
> 
> I don't really think the macro name is a good choice, but I couldn't come up
> with something else? Suggestions? It could be used for LSA and SAMR SID and name
> lookups.

I don't like it either, but it's not easy to think of
good names. RB+. Pushed, and we can update later if
we think of a better name :-).