int server_certificate_path(FILE **file, const char *hostname,
char *path, size_t size) {
+ int result;
+
/* Hostname too long. */
if (size - strlen(STORED_SERVER_CERT_FORMAT) <= strlen(hostname)) {
LOG(LOG_WARNING,
hostname);
return -1;
}
- snprintf(path, size, STORED_SERVER_CERT_FORMAT, hostname);
+ result = snprintf(path, size, STORED_SERVER_CERT_FORMAT, hostname);
+ if (result < 0) {
+ LOG_PERROR(LOG_ERROR,
+ "server_certificate_path(): snprintf failed");
+ return -1;
+ } else if ((size_t)result >= size) {
+ LOG(LOG_ERROR,
+ "server_certificate_path(): snprintf buffer too short");
+ return -1;
+ }
/* Open the stored certificate file. */
*file = fopen(path, "rb");
if (NULL == *file) {
- LOG(global_passthrough_unknown ? LOG_DEBUG : LOG_WARNING,
- "server_certificate_path(): failed to open '%s': %s",
- path, strerror(errno));
+ if (global_passthrough_unknown) {
+ LOG(LOG_DEBUG,
+ "server_certificate_path(): failed to open '%s': %s",
+ path, strerror(errno));
+ } else {
+ LOG(LOG_WARNING,
+ "server_certificate_path(): failed to open '%s': %s",
+ path, strerror(errno));
+ }
/* Couldn't open the file, special case. */
return -2;
}