diff options
author | Matthias Andree <matthias.andree@gmx.de> | 2020-03-29 00:38:23 +0100 |
---|---|---|
committer | Matthias Andree <matthias.andree@gmx.de> | 2020-03-29 00:38:23 +0100 |
commit | e9f7a61890f9ecf6eb20490f6f9936dc6c9ea250 (patch) | |
tree | 0a265ba8baae75a49d642a7769756eca80b967c6 | |
parent | 182db3462b3c7c12660fd0bff54633f40a9e4332 (diff) | |
download | fetchmail-e9f7a61890f9ecf6eb20490f6f9936dc6c9ea250.tar.gz fetchmail-e9f7a61890f9ecf6eb20490f6f9936dc6c9ea250.tar.bz2 fetchmail-e9f7a61890f9ecf6eb20490f6f9936dc6c9ea250.zip |
Rename s -> vecsiz (vector size) to hint to its purpose
-rw-r--r-- | socket.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -89,14 +89,14 @@ static char *const *parse_plugin(const char *plugin, const char *host, const cha char *cp, *plugin_copy; unsigned int plugin_copy_len; unsigned int plugin_offset = 0, plugin_copy_offset = 0; - unsigned int i, s = 2 * sizeof(char*), host_count = 0, service_count = 0; + unsigned int i, vecsiz = 2 * sizeof(char*), host_count = 0, service_count = 0; unsigned int plugin_len = strlen(plugin); unsigned int host_len = strlen(host); unsigned int service_len = strlen(service); for (c = p = plugin; *c; c++) { if (isspace((unsigned char)*c) && !isspace((unsigned char)*p)) - s += sizeof(char*); + vecsiz += sizeof(char*); if (*p == '%' && *c == 'h') host_count++; if (*p == '%' && *c == 'p') @@ -133,14 +133,14 @@ static char *const *parse_plugin(const char *plugin, const char *host, const cha /* XXX FIXME - is this perhaps a bit too simplistic to chop down the argument strings without any respect to quoting? * better write a generic function that tracks arguments instead... */ - argvec = (char **)malloc(s); + argvec = (char **)malloc(vecsiz); if (!argvec) { free(plugin_copy); report(stderr, GT_("fetchmail: malloc failed\n")); return NULL; } - memset(argvec, 0, s); + memset(argvec, 0, vecsiz); for (p = cp = plugin_copy, i = 0; *cp; cp++) { if ((!isspace((unsigned char)*cp)) && (cp == p ? 1 : isspace((unsigned char)*p))) { argvec[i] = cp; |