vicente has uploaded this change for review.

View Change

idmapper: should not set idmapping_enabled to true by default

    We remove the one calling path for idmapper_set_owner_domain()
    on commit 69da0b4.

    But in some cases, we would not be able to call the
    idmapper_set_owner_domain(), which would cause the owner_domain
    not to be initialized.

    For example, the current idmapping_enabled default is true.
    We will pass the nfs_param.directory_services_param.idmapping_active
    as status_enabled. If the idmapping_active is also true, we will skip
    the first time init to calling idmapper_set_owner_domain().

    To resolve this, we should set idmapping_enabled as false by default.
    That will ensure we can execute the idmapper_set_owner_domain() for
    initialization.

Change-Id: I1a3d6e88a59d3943823892f72ee58265f9a9f156
Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
---
M src/idmapper/idmapper.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/48/1214048/1

To view, visit change 1214048. To unsubscribe, or for help writing mail filters, visit settings.