From 05c6bf63bb08799ebbff76b73c50abb399e3a1e0 Mon Sep 17 00:00:00 2001
From: Přemysl Janouch 
Date: Sun, 5 Jul 2015 23:18:52 +0200
Subject: kike: actually use irc_is_valid_key()
---
 kike.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/kike.c b/kike.c
index c5c36da..2c911dc 100644
--- a/kike.c
+++ b/kike.c
@@ -1856,16 +1856,16 @@ mode_processor_do_key (struct mode_processor *self)
 		free (self->channel->key);
 		self->channel->key = NULL;
 	}
+	else if (!irc_is_valid_key (target))
+		// TODO: we should notify the user somehow
+		return;
+	else if (self->channel->key)
+		irc_send_reply (self->c, IRC_ERR_KEYSET, self->channel->name);
 	else
 	{
-		if (self->channel->key)
-			irc_send_reply (self->c, IRC_ERR_KEYSET, self->channel->name);
-		else
-		{
-			self->channel->key = xstrdup (target);
-			str_append_c (&self->added, self->mode_char);
-			str_vector_add (&self->added_params, self->channel->key);
-		}
+		self->channel->key = xstrdup (target);
+		str_append_c (&self->added, self->mode_char);
+		str_vector_add (&self->added_params, self->channel->key);
 	}
 }
 
-- 
cgit v1.2.3-70-g09d2