-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 03/04/2010 11:02 AM, Jakub Hrozek wrote:
On 03/03/2010 08:44 PM, Jakub Hrozek wrote:
Fixes: 290
Self-nack, the attached patch fixes a bug related to error handling in the patch.
Nack.
If you set filter_users or filter_groups in a domain, but do not also have it set in [NSS], the configuration fails with ENOENT, because if (ret == ENOENT && !filter_set) is false.
Also, while you're in there, could you fix the impending segfault after filter_list = talloc_array(tmpctx, char *, 2);
Instead of checking for NULL here, it's not tested until after attempting to assign to filter_list[0].
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/