Reported by Arran Cudbard-Bell <email@example.com>.
Patch and regression tests to follow.
Created attachment 16296 [details]
Add additional debug text to memlimit tests
Created attachment 16297 [details]
Add additional test cases for memlimit
Created attachment 16298 [details]
Add note in doxygen about how memlimit interacts with pools
Jeremy, the final patch submitted to samba-technical looks good to me. I've attached the same patches as I posted.
I think once these are all merged we can considered this bug resolved.
Gonna need a little more - the additional test should go in first marked as "knownfail", then the patch that fixes them removes them from the knownfail directory again. I'm working on it..
running CI tests first.
Created attachment 16313 [details]
git-am fix for master.
Created attachment 16314 [details]
Updated with sigs files.
This bug was referenced in samba master:
Not sure if we need to back-port this to 4.13.next, or just leave until 4.14.
Arran, do you have an opinion on this ?
It's not especially urgent as we have a work around, so I'm fine with leaving it until 4.14.