[PATCHSET] Add support for fall through attribute

Andrew Bartlett abartlet at samba.org
Tue Jan 16 18:31:44 UTC 2018


On Tue, 2018-01-16 at 15:11 +0100, Andreas Schneider via samba-
technical wrote:
> On Thursday, 21 December 2017 20:00:41 CET Andreas Schneider via samba-
> technical wrote:
> > I'm not sure. Either you need to remove the p++ and FALL_THROUGH or we have
> > to add a break.
> > 
> > However I think we should make the TLDAP test work which currently fails.
> 
> I've talked to Simo, he confirmed that we need to break in this switch 
> statement or we stop parsing in the middle. I've also enabled the TLDAP test.

Did the TLDAP test cover this, that is did it break previously?  It
isn't clear from a skim of the patch. 

If not, could you add such a test case to prove the fix?

Thanks,

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




More information about the samba-technical mailing list