about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorJason A. Donenfeld2013-03-20 20:44:20 +0100
committerJason A. Donenfeld2013-03-20 21:08:32 +0100
commit6d7e3596ebb387265d8cfdc5b312e0ea76da8c8a (patch)
treee160033edb9a9077b077c7cf2d6d65f8856d6643
parentui-shared: squelch compiler warning. (diff)
downloadcgit-6d7e3596ebb387265d8cfdc5b312e0ea76da8c8a.tar.gz
cgit-6d7e3596ebb387265d8cfdc5b312e0ea76da8c8a.zip
html: check return value of write
This squelches a gcc warning. It's also correct that we check to see if
there are any partial or failed writes. For now, we just print a warning
to stderr. In the future, perhaps it will prove wise to exit(1) on
partial writes.
-rw-r--r--html.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/html.c b/html.c index b5c6903..d60a41f 100644 --- a/html.c +++ b/html.c
@@ -63,12 +63,13 @@ char *fmt(const char *format, ...)
63 63
64void html_raw(const char *data, size_t size) 64void html_raw(const char *data, size_t size)
65{ 65{
66 write(htmlfd, data, size); 66 if (write(htmlfd, data, size) != size)
67 fprintf(stderr, "[html.c] html output truncated.\n");
67} 68}
68 69
69void html(const char *txt) 70void html(const char *txt)
70{ 71{
71 write(htmlfd, txt, strlen(txt)); 72 html_raw(txt, strlen(txt));
72} 73}
73 74
74void htmlf(const char *format, ...) 75void htmlf(const char *format, ...)