diff options
author | Andrew Bartlett <abartlet@samba.org> | 2003-03-18 11:22:52 +0000 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2003-03-18 11:22:52 +0000 |
commit | c5b604e2ee67d74241ae2fa07ae904647d35a2be (patch) | |
tree | 83e80f648c8c3a02a1f2fe5166a6b4b23a147180 /source/utils/net_rap.c | |
parent | 196628432130f826505187a27207a7b947e034cf (diff) | |
download | samba-c5b604e2ee67d74241ae2fa07ae904647d35a2be.tar.gz samba-c5b604e2ee67d74241ae2fa07ae904647d35a2be.tar.xz samba-c5b604e2ee67d74241ae2fa07ae904647d35a2be.zip |
Jeremy merged across my string parinoia fixes, but forgot to enable them! :-)
This patch catches up on the rest of the work - as much string checking
as is possible is done at compile time, and the rest at runtime.
Lots of code converted to pstrcpy() etc, and other code reworked to correctly
call sizeof().
Andrew Bartlett
Diffstat (limited to 'source/utils/net_rap.c')
-rw-r--r-- | source/utils/net_rap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/utils/net_rap.c b/source/utils/net_rap.c index 8f3dd53fa6a..f52eabf494e 100644 --- a/source/utils/net_rap.c +++ b/source/utils/net_rap.c @@ -638,7 +638,7 @@ static int rap_user_add(int argc, const char **argv) if (!(cli = net_make_ipc_connection(0))) return -1; - safe_strcpy(userinfo.user_name, argv[0], sizeof(userinfo.user_name)); + safe_strcpy(userinfo.user_name, argv[0], sizeof(userinfo.user_name)-1); if (opt_flags == -1) opt_flags = 0x21; @@ -755,7 +755,7 @@ static int rap_group_add(int argc, const char **argv) return -1; /* BB check for length 21 or smaller explicitly ? BB */ - safe_strcpy(grinfo.group_name, argv[0], sizeof(grinfo.group_name)); + safe_strcpy(grinfo.group_name, argv[0], sizeof(grinfo.group_name)-1); grinfo.reserved1 = '\0'; grinfo.comment = smb_xstrdup(opt_comment); |