summaryrefslogtreecommitdiff
path: root/source3/utils
diff options
context:
space:
mode:
authorSimo Sorce <idra@samba.org>2003-06-22 10:09:52 +0000
committerSimo Sorce <idra@samba.org>2003-06-22 10:09:52 +0000
commitf5974dfaae680d98b78d600cd1f1aaece332a085 (patch)
treebd24ad803125c21cdb82a27d051e0029cb21ad65 /source3/utils
parent4830a878455fe0e525aa3bd2a59e55d1f30bddad (diff)
downloadsamba-f5974dfaae680d98b78d600cd1f1aaece332a085.tar.gz
samba-f5974dfaae680d98b78d600cd1f1aaece332a085.tar.bz2
samba-f5974dfaae680d98b78d600cd1f1aaece332a085.zip
Found out a good number of NT_STATUS_IS_ERR used the wrong way.
As abartlet rememberd me NT_STATUS_IS_ERR != !NT_STATUS_IS_OK This patch will cure the problem. Working on this one I found 16 functions where I think NT_STATUS_IS_ERR() is used correctly, but I'm not 100% sure, coders should check the use of NT_STATUS_IS_ERR() in samba is ok now. Simo. (This used to be commit c501e84d412563eb3f674f76038ec48c2b458687)
Diffstat (limited to 'source3/utils')
-rw-r--r--source3/utils/pdbedit.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/utils/pdbedit.c b/source3/utils/pdbedit.c
index 571775a741..aa11f2bfdf 100644
--- a/source3/utils/pdbedit.c
+++ b/source3/utils/pdbedit.c
@@ -485,7 +485,7 @@ static int delete_user_entry (struct pdb_context *in, const char *username)
return -1;
}
- if (NT_STATUS_IS_ERR(in->pdb_getsampwnam(in, samaccount, username))) {
+ if (!NT_STATUS_IS_OK(in->pdb_getsampwnam(in, samaccount, username))) {
fprintf (stderr, "user %s does not exist in the passdb\n", username);
return -1;
}
@@ -511,7 +511,7 @@ static int delete_machine_entry (struct pdb_context *in, const char *machinename
return -1;
}
- if (NT_STATUS_IS_ERR(in->pdb_getsampwnam(in, samaccount, name))) {
+ if (!NT_STATUS_IS_OK(in->pdb_getsampwnam(in, samaccount, name))) {
fprintf (stderr, "machine %s does not exist in the passdb\n", name);
return -1;
}