summaryrefslogtreecommitdiffstats
path: root/source3/winbindd/idmap_autorid.c
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2014-04-11 18:25:44 +0200
committerMichael Adam <obnox@samba.org>2014-04-25 15:35:09 +0200
commit64e267c2feaecd9c2f0a60d548ccd2ff0ee37879 (patch)
tree0614e1ad801f848d7b36a11d3593d41270551cb3 /source3/winbindd/idmap_autorid.c
parent5d7b1363f0c31acc7f385f69e7dec028b192e4bd (diff)
downloadsamba-64e267c2feaecd9c2f0a60d548ccd2ff0ee37879.tar.gz
samba-64e267c2feaecd9c2f0a60d548ccd2ff0ee37879.tar.xz
samba-64e267c2feaecd9c2f0a60d548ccd2ff0ee37879.zip
autorid: make the checks for bumping num_mapped identical for alloc and rid case
in idmap_autorid_sids_to_unixids() Signed-off-by: Michael Adam <obnox@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source3/winbindd/idmap_autorid.c')
-rw-r--r--source3/winbindd/idmap_autorid.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c
index 0bdc96dd47..b66caec902 100644
--- a/source3/winbindd/idmap_autorid.c
+++ b/source3/winbindd/idmap_autorid.c
@@ -465,7 +465,7 @@ static NTSTATUS idmap_autorid_sids_to_unixids(struct idmap_domain *dom,
goto failure;
}
- if (ids[i]->status == ID_MAPPED) {
+ if (NT_STATUS_IS_OK(ret) && ids[i]->status == ID_MAPPED) {
num_mapped++;
}
@@ -524,7 +524,7 @@ static NTSTATUS idmap_autorid_sids_to_unixids(struct idmap_domain *dom,
goto failure;
}
- if (NT_STATUS_IS_OK(ret)) {
+ if (NT_STATUS_IS_OK(ret) && ids[i]->status == ID_MAPPED) {
num_mapped++;
}
}