GitLab/ocserv
1
0
Fork 0
mirror of https://gitlab.com/openconnect/ocserv.git synced 2026-03-14 14:58:06 +08:00
Code Issues Packages Projects Releases Wiki Activity

Ignore the return code of snprintf(); it is useless.

Browse Source
This commit is contained in:
Nikos Mavrogiannopoulos
2014-06-23 17:26:01 +02:00
parent 1d2f36f9bf
commit 85288fd96f
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/worker-auth.c
View File

@@ -877,6 +877,8 @@ int post_common_handler(worker_st * ws, unsigned http_ver)
ws->config->banner); ws->config->banner);
if (size <= 0) if (size <= 0)
return -1; return -1;
/* snprintf() returns not a very useful value, so we need to recalculate */
size = strlen(msg);
} else { } else {
msg[0] = 0; msg[0] = 0;
size = 0; size = 0;
@@ -964,8 +966,11 @@ int parse_reply(worker_st * ws, char *body, unsigned body_length,
field_size = xml_field_size; field_size = xml_field_size;
} }
temp1_len = snprintf(temp1, sizeof(temp1), "<%s>", field); snprintf(temp1, sizeof(temp1), "<%s>", field);
temp2_len = snprintf(temp2, sizeof(temp2), "</%s>", field); snprintf(temp2, sizeof(temp2), "</%s>", field);
temp1_len = strlen(temp1);
temp2_len = strlen(temp2);
/* body should contain <username>test</username><password>test</password> */ /* body should contain <username>test</username><password>test</password> */
*value = *value =
@@ -988,7 +993,8 @@ int parse_reply(worker_st * ws, char *body, unsigned body_length,
len++; len++;
} }
} else { /* non-xml version */ } else { /* non-xml version */
temp1_len = snprintf(temp1, sizeof(temp1), "%s=", field); snprintf(temp1, sizeof(temp1), "%s=", field);
temp1_len = strlen(temp1);
/* body should be "username=test&password=test" */ /* body should be "username=test&password=test" */
*value = *value =