[PATCHES] Merge loadparm further
nivanova at samba.org
Fri Jul 4 07:22:17 MDT 2014
Hi Garming and Andrew,
Thanks for adding the test! I have reviewed all the patches in the
branch, stating from "param: ensure CMD_LIST is freed when freeing a
parameter" to the last one, and they look fine to me.
Reviewed-by: Nadezhda Ivanova <nivanova at samba.org>
On Thu, Jul 3, 2014 at 10:42 AM, Andrew Bartlett <abartlet at samba.org> wrote:
> On Thu, 2014-07-03 at 18:17 +1200, Garming Sam wrote:
>> I've added an additional patch to the set of loadparm patches to attempt
>> to ensure that the flags and the display of defaults are correct.
>> What the patch does is extend the docs.py tests, with an empty smb.conf
>> test and with an overriding smb.conf defaults test.
>> git://git.catalyst.net.nz/samba.git before-service-ptrs-review2
> Thanks Garming, this gives me much more confidence in the patch series.
> While bugs are always possible, loadparm has never been this well tested
> Reviewed-by: Andrew Bartlett <abartlet at samba.org>
> 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