summaryrefslogtreecommitdiff
path: root/source3/winbindd/winbindd_wins.c
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2011-05-04 11:38:26 -0700
committerJeremy Allison <jra@samba.org>2011-05-04 12:12:15 -0700
commit8380835fc6de38706d9af29dc7f0fa4cec4f9c90 (patch)
tree5fdf238d5573c022a135e3f2e5590b656b0d64bf /source3/winbindd/winbindd_wins.c
parentdf023b8657cab232df88d3656aa5d87676bb7254 (diff)
downloadsamba-8380835fc6de38706d9af29dc7f0fa4cec4f9c90.tar.gz
samba-8380835fc6de38706d9af29dc7f0fa4cec4f9c90.tar.bz2
samba-8380835fc6de38706d9af29dc7f0fa4cec4f9c90.zip
Fix warning messages caused by addition of null check in fstrcpy macro.
Diffstat (limited to 'source3/winbindd/winbindd_wins.c')
-rw-r--r--source3/winbindd/winbindd_wins.c16
1 files changed, 10 insertions, 6 deletions
diff --git a/source3/winbindd/winbindd_wins.c b/source3/winbindd/winbindd_wins.c
index e1beb1d80b..5ac6109411 100644
--- a/source3/winbindd/winbindd_wins.c
+++ b/source3/winbindd/winbindd_wins.c
@@ -140,7 +140,9 @@ void winbindd_wins_byip(struct winbindd_cli_state *state)
response[strlen(response)-1] = '\n';
TALLOC_FREE(status);
}
- fstrcpy(state->response->data.winsresp,response);
+ strlcpy(state->response->data.winsresp,
+ response,
+ sizeof(state->response->data.winsresp));
request_ok(state);
}
@@ -181,8 +183,8 @@ void winbindd_wins_byname(struct winbindd_cli_state *state)
response[strlen(response)-1] = ' ';
}
}
- fstrcat(response,addr);
- fstrcat(response,"\t");
+ strlcat(response,addr,sizeof(response));
+ strlcat(response,"\t",sizeof(response));
}
size = strlen(state->request->data.winsreq) + strlen(response);
if (size > maxlen) {
@@ -190,15 +192,17 @@ void winbindd_wins_byname(struct winbindd_cli_state *state)
request_error(state);
return;
}
- fstrcat(response,state->request->data.winsreq);
- fstrcat(response,"\n");
+ strlcat(response,state->request->data.winsreq,sizeof(response));
+ strlcat(response,"\n",sizeof(response));
TALLOC_FREE(ip_list);
} else {
request_error(state);
return;
}
- fstrcpy(state->response->data.winsresp,response);
+ strlcpy(state->response->data.winsresp,
+ response,
+ sizeof(state->response->data.winsresp));
request_ok(state);
}