Rooba
/
boinc
mirror of https://github.com/BOINC/boinc.git
1
0
Fork 0
Code Issues 1 Packages Projects Releases Wiki Activity

API: Cleanup low hanging fruit with regards to cleaning up strcpy and strcat use. 

Use safe_strcpy and safe_strcat when dealing with non-pointer data types.
This commit is contained in:
Rom Walton 2016-02-15 20:11:03 -05:00
parent dd1232ad63
commit 826437f244
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
api/boinc_api.cpp
View File

@ -670,12 +670,12 @@ int boinc_get_status(BOINC_STATUS *s) {
static void send_trickle_up_msg() {
char buf[MSG_CHANNEL_SIZE];
if (standalone) return;
strcpy(buf, "");
safe_strcpy(buf, "");
if (have_new_trickle_up) {
strcat(buf, "<have_new_trickle_up/>\n");
safe_strcat(buf, "<have_new_trickle_up/>\n");
}
if (have_new_upload_file) {
strcat(buf, "<have_new_upload_file/>\n");
safe_strcat(buf, "<have_new_upload_file/>\n");
}
if (strlen(buf)) {
BOINCINFO("Sending Trickle Up Message");
@ -893,15 +893,15 @@ int boinc_report_app_status_aux(
);
if (other_pid) {
sprintf(buf, "<other_pid>%d</other_pid>\n", other_pid);
strcat(msg_buf, buf);
safe_strcat(msg_buf, buf);
}
if (_bytes_sent) {
sprintf(buf, "<bytes_sent>%f</bytes_sent>\n", _bytes_sent);
strcat(msg_buf, buf);
safe_strcat(msg_buf, buf);
}
if (_bytes_received) {
sprintf(buf, "<bytes_received>%f</bytes_received>\n", _bytes_received);
strcat(msg_buf, buf);
safe_strcat(msg_buf, buf);
}
if (app_client_shm->shm->app_status.send_msg(msg_buf)) {
return 0;