about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJune McEnroe <june@causal.agency>2020-02-26 02:28:35 -0500
committerJune McEnroe <june@causal.agency>2020-02-26 02:37:10 -0500
commitfe437b2ea18df5ecd1baa5086ccc1e866af21df5 (patch)
tree55084737d377d8f99d38fa7cc46868a90975d5aa
parentAdd time tag to lines missing it (diff)
downloadpounce-fe437b2ea18df5ecd1baa5086ccc1e866af21df5.tar.gz
pounce-fe437b2ea18df5ecd1baa5086ccc1e866af21df5.zip
Tweak buffer sizes
Filter functions are dealing with lines not including CRLF, so they
already have extra space. serverFormat is using snprintf which wants to
always write a NUL at the end of the string.
Diffstat (limited to '')
-rw-r--r--client.c6
-rw-r--r--server.c2
2 files changed, 4 insertions, 4 deletions
diff --git a/client.c b/client.c
index 9b55fdd..2405004 100644
--- a/client.c
+++ b/client.c
@@ -426,7 +426,7 @@ static const char *filterChghost(const char *line) {
 static const char *filterExtendedJoin(const char *line) {
 	if (wordcmp(line, 1, "JOIN")) return line;
 	static regex_t regex;
-	static char buf[MessageCap + 1];
+	static char buf[MessageCap];
 	return snip(buf, sizeof(buf), line, compile(&regex, "(JOIN [^ ]+).+"));
 }
 
@@ -440,7 +440,7 @@ static const char *filterMessageTags(const char *line) {
 }
 
 static const char *filterMultiPrefix(const char *line) {
-	static char buf[MessageCap + 1];
+	static char buf[MessageCap];
 	if (!wordcmp(line, 1, "352")) {
 		static regex_t regex;
 		return snip(
@@ -461,7 +461,7 @@ static const char *filterMultiPrefix(const char *line) {
 static const char *filterUserhostInNames(const char *line) {
 	if (wordcmp(line, 1, "353")) return line;
 	static regex_t regex;
-	static char buf[MessageCap + 1];
+	static char buf[MessageCap];
 	return snip(
 		buf, sizeof(buf), line,
 		compile(&regex, "( :?[^!]+)![^ ]+")
diff --git a/server.c b/server.c
index abc7d40..20d94e3 100644
--- a/server.c
+++ b/server.c
@@ -139,7 +139,7 @@ void serverSend(const char *ptr, size_t len) {
 }
 
 void serverFormat(const char *format, ...) {
-	char buf[MessageCap];
+	char buf[MessageCap + 1];
 	va_list ap;
 	va_start(ap, format);
 	int len = vsnprintf(buf, sizeof(buf), format, ap);
d518a8bf2229b3e463b10f&follow=1'>Add head-include configuration option.Mark Lodato 2009-03-15CGIT 0.8.2.1Lars Hjemli 2009-03-15Fix doc-related glitches in Makefile and .gitignoreLars Hjemli 2009-03-15ui-snapshot: avoid segfault when no filename is specifiedLars Hjemli 2009-03-15fix segfault when displaying empty blobsEric Wong 2009-02-19Add support for HEAD requestsLars Hjemli 2009-02-19Add support for ETag in 'plain' viewLars Hjemli 2009-02-12ui-tree: escape ascii-text properly in hexdump viewLars Hjemli 2009-02-12Makefile: add doc-related targetsLars Hjemli