From: Pierre-Marc Fournier <pierre-marc.fournier@polymtl.ca>
Date: Thu, 11 Mar 2010 23:57:28 +0000 (-0500)
Subject: fix FIXMEs
X-Git-Tag: v1.9.1~644
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=b88f0774143e1478fd3a85209e8d2571ef998105;p=lttng-ust.git

fix FIXMEs
---

diff --git a/ustd/ustd.c b/ustd/ustd.c
index aae4b024..c1dd0cb8 100644
--- a/ustd/ustd.c
+++ b/ustd/ustd.c
@@ -465,7 +465,6 @@ int write_current_subbuffer(struct buffer_info *buf)
 	result = patient_write(buf->file_fd, subbuf_mem, cur_sb_size);
 	if(result == -1) {
 		PERROR("write");
-		/* FIXME: maybe drop this trace */
 		return -1;
 	}
 
@@ -495,8 +494,10 @@ int consumer_loop(struct buffer_info *buf)
 		}
 
 		/* write data to file */
-		write_current_subbuffer(buf);
-		/* FIXME: handle return value? */
+		result = write_current_subbuffer(buf);
+		if(result == -1) {
+			ERR("Failed writing a subbuffer to file (channel=%s). Dropping this buffer.", buf->name);
+		}
 
 		/* put the subbuffer */
 		result = put_subbuffer(buf);