From: Richard Hughes <hughsient@gmail.com>
Date: Wed, 15 May 2013 08:17:38 +0000 (+0100)
Subject: cms-colord: Warn if reading or writing to the FD failed
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d561687ec829a05f9e5ee103b61f733a9bca15ad;p=platform%2Fupstream%2Fweston.git

cms-colord: Warn if reading or writing to the FD failed

This also fixes a compile warning when building the tarball.
---

diff --git a/src/cms-colord.c b/src/cms-colord.c
index af6b5fa..6056407 100644
--- a/src/cms-colord.c
+++ b/src/cms-colord.c
@@ -127,6 +127,7 @@ static void
 update_device_with_profile_in_idle(struct cms_output *ocms)
 {
 	gboolean signal_write = FALSE;
+	ssize_t rc;
 	struct cms_colord *cms = ocms->cms;
 
 	colord_idle_cancel_for_output(cms, ocms->o);
@@ -139,7 +140,9 @@ update_device_with_profile_in_idle(struct cms_output *ocms)
 	/* signal we've got updates to do */
 	if (signal_write) {
 		gchar tmp = '\0';
-		write(cms->writefd, &tmp, 1);
+		rc = write(cms->writefd, &tmp, 1);
+		if (rc == 0)
+			weston_log("colord: failed to write to pending fd");
 	}
 }
 
@@ -365,6 +368,7 @@ colord_dispatch_all_pending(int fd, uint32_t mask, void *data)
 {
 	gchar tmp;
 	GList *l;
+	ssize_t rc;
 	struct cms_colord *cms = data;
 	struct cms_output *ocms;
 
@@ -387,7 +391,9 @@ colord_dispatch_all_pending(int fd, uint32_t mask, void *data)
 	g_mutex_unlock(&cms->pending_mutex);
 
 	/* done */
-	read(cms->readfd, &tmp, 1);
+	rc = read(cms->readfd, &tmp, 1);
+	if (rc == 0)
+		weston_log("colord: failed to read from pending fd");
 	return 1;
 }