[PATCH] ldbndr: Include context for error strings in ndr_check_array_{size,length}() and ndr_pull_array_length()

Andrew Bartlett abartlet at samba.org
Thu May 3 03:02:28 UTC 2018


On Thu, 2018-05-03 at 14:35 +1200, William Brown wrote:
> On Thu, 2018-05-03 at 12:13 +1200, Andrew Bartlett via samba-technical
> wrote:
> > I'm trying to track down the new AD DC LSA failure.  The attached
> > patch
> > may help us locate what is behind the failure and provide useful
> > debugging into the future.
> > 
> > Please review and push.
> > 
> > Andrew Bartlett
> 
> Pushing char pointers around when you have only a handful of
> predetermined combinations seems a bit overkill? Why not only provide a
> single char identifier and then use the error message to determine the
> fault?

It probably isn't clear, but the PIDL code generator calls this from
almost every DCE/RPC structure.  I was thinking to include __location__
as well but didn't yet.

Andrew Bartlett

-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba







More information about the samba-technical mailing list