summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jeremy@jeremy-desktop.(none)>2008-12-23 09:50:10 -0800
committerJeremy Allison <jeremy@jeremy-desktop.(none)>2008-12-23 09:50:10 -0800
commitcb7d7beb99ab7952b5187c05b2660283d93e59fc (patch)
tree6043b5664700ae6cdbb6d7044ab271108978ff35
parent227c8000dfee4248fc2d13104382542e449a1897 (diff)
downloadsamba-cb7d7beb99ab7952b5187c05b2660283d93e59fc.tar.gz
samba-cb7d7beb99ab7952b5187c05b2660283d93e59fc.tar.bz2
samba-cb7d7beb99ab7952b5187c05b2660283d93e59fc.zip
Fix another "format not a string literal and no format arguments" warning.
Jeremy
-rw-r--r--source3/libnet/libnet_samsync_ldif.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/source3/libnet/libnet_samsync_ldif.c b/source3/libnet/libnet_samsync_ldif.c
index c72eadf03f..3068f8d3eb 100644
--- a/source3/libnet/libnet_samsync_ldif.c
+++ b/source3/libnet/libnet_samsync_ldif.c
@@ -586,7 +586,7 @@ static NTSTATUS fetch_account_info_to_ldif(TALLOC_CTX *mem_ctx,
uchar zero_buf[16];
uint32 rid = 0, group_rid = 0, gidNumber = 0;
time_t unix_time;
- int i;
+ int i, ret;
memset(zero_buf, '\0', sizeof(zero_buf));
@@ -660,7 +660,10 @@ static NTSTATUS fetch_account_info_to_ldif(TALLOC_CTX *mem_ctx,
return NT_STATUS_UNSUCCESSFUL;
}
gidNumber = groupmap[i].gidNumber;
- snprintf(sambaSID, sizeof(sambaSID), groupmap[i].sambaSID);
+ ret = snprintf(sambaSID, sizeof(sambaSID), "%s", groupmap[i].sambaSID);
+ if (ret < 0 || ret == sizeof(sambaSID)) {
+ return NT_STATUS_UNSUCCESSFUL;
+ }
/* Set up sambaAcctFlags */
flags = pdb_encode_acct_ctrl(r->acct_flags,