Attachment 10855 Details for Bug 11154 – Harden logging by checking I/O error, etc.

[patch] Harden logging by checking I/O error, etc.

0001-Harden-logging-perror-replaced-with-cc_log-and-fatal.patch (text/plain), 2.74 KB, created by ishikawa on 2015-03-11 15:40:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ishikawa

Created: 2015-03-11 15:40:07 UTC

Size: 2.74 KB

patch

obsolete

>From 61b347f47a82564dc0ef52d3abc258b64801e355 Mon Sep 17 00:00:00 2001
>From: Chiaki ISHIKAWA <ishikawa@yk.rim.or.jp>
>Date: Tue, 10 Mar 2015 21:21:06 +0900
>Subject: [PATCH 1/8] Harden logging, perror() replaced with cc_log() and
> fatal(). unlink() failure recorded.
>
>---
> util.c | 43 ++++++++++++++++++++++++++++++++++++-------
> 1 file changed, 36 insertions(+), 7 deletions(-)
>
>diff --git a/util.c b/util.c
>index 4b8dd15..7e1bed5 100644
>--- a/util.c
>+++ b/util.c
>@@ -107,16 +107,33 @@ path_max(const char *path)
> #endif
> }
> 
>+/*
>+ * warn the failure of logging function.
>+ * exit then.
>+ */
>+static void
>+warn_log_fail() {
>+	fprintf(stderr,
>+		"Writing to logfile failed.\n"
>+		"Check the permission and make sure there is enough room for writing to the filesystem.\n");
>+	/* Maybe we should exit here! */
>+        fatal("ccache log could not be written.");
>+}
>+
> static void
> vlog(const char *format, va_list ap, bool log_updated_time)
> {
>+	int rc1, rc2;
> 	if (!init_log()) {
> 		return;
> 	}
> 
> 	log_prefix(log_updated_time);
>-	vfprintf(logfile, format, ap);
>-	fprintf(logfile, "\n");
>+	rc1 = vfprintf(logfile, format, ap);
>+	rc2 = fprintf(logfile, "\n");
>+	if (rc1 < 0 || rc2 < 0) {
>+		warn_log_fail();
>+	}
> }
> 
> /*
>@@ -153,6 +170,7 @@ cc_bulklog(const char *format, ...)
> void
> cc_log_argv(const char *prefix, char **argv)
> {
>+	int rc;
> 	if (!init_log()) {
> 		return;
> 	}
>@@ -160,7 +178,9 @@ cc_log_argv(const char *prefix, char **argv)
> 	log_prefix(true);
> 	fputs(prefix, logfile);
> 	print_command(logfile, argv);
>-	fflush(logfile);
>+	rc = fflush(logfile);
>+	if (rc)
>+		warn_log_fail();
> }
> 
> /* something went badly wrong! */
>@@ -827,7 +847,8 @@ traverse(const char *dir, void (*fn)(const char *, struct stat *))
> 		fname = format("%s/%s", dir, de->d_name);
> 		if (lstat(fname, &st)) {
> 			if (errno != ENOENT) {
>-				perror(fname);
>+				cc_log("lstat on file %s failed.", fname);
>+                                fatal("lstat on file %s failed.", fname);
> 			}
> 			free(fname);
> 			continue;
>@@ -1413,8 +1434,13 @@ x_rename(const char *oldpath, const char *newpath)
> int
> tmp_unlink(const char *path)
> {
>-	cc_log("Unlink %s", path);
>-	return unlink(path);
>+	int rc;
>+	cc_log("tmp_unlink:Unlink %s", path);
>+	rc = unlink(path);
>+	if (rc) {
>+		cc_log("tmp_unlink:Unlink failed: rc = %d", rc);
>+	}
>+	return rc;
> }
> 
> /*
>@@ -1430,7 +1456,7 @@ x_unlink(const char *path)
> 	 */
> 	char *tmp_name = format("%s.rm.%s", path, tmp_string());
> 	int result = 0;
>-	cc_log("Unlink %s via %s", path, tmp_name);
>+	cc_log("x_unlink: Unlink %s via %s", path, tmp_name);
> 	if (x_rename(path, tmp_name) == -1) {
> 		result = -1;
> 		goto out;
>@@ -1443,6 +1469,9 @@ x_unlink(const char *path)
> 	}
> out:
> 	free(tmp_name);
>+	if (result) {
>+		cc_log("x_unlink: failed result = %d", result);
>+	}
> 	return result;
> }
> 
>-- 
>2.1.4
>

Actions: View

Attachments on bug 11154: 10855 | 10880 | 10890