From df63760e0e345e1ec390c647060a157bf627b67f Mon Sep 17 00:00:00 2001
From: Till Höppner
Date: Thu, 25 Feb 2016 18:07:18 +0100
Subject: More robust weechat parsing

---
 formats/weechat/src/lib.rs | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

(limited to 'formats/weechat')

diff --git a/formats/weechat/src/lib.rs b/formats/weechat/src/lib.rs
index ebc3c2f..65bb4fa 100644
--- a/formats/weechat/src/lib.rs
+++ b/formats/weechat/src/lib.rs
@@ -74,9 +74,11 @@ impl<'a> Iterator for Iter<'a> {
             // Don't match on the --> arrows, those are apparently often configured, and
             // that would break parsing for many users.
 
-            if tokens[5] == "has" {
+            let len = tokens.len();
+
+            if len >= 6 && tokens[5] == "has" {
                 // 2016-02-25 01:15:05 --> Foo (host@mask.foo) has joined #example
-                if tokens[6] == "joined" {
+                if len >= 8 && tokens[6] == "joined" {
                     return Some(Ok(Event {
                         ty: Type::Join {
                             nick: tokens[3].to_owned().into(),
@@ -87,7 +89,7 @@ impl<'a> Iterator for Iter<'a> {
                     }));
                 }
                 // 2016-02-25 01:36:13 <-- Foo (host@mask.foo) has left #channel (Some reason)
-                else if tokens[6] == "left" {
+                else if len >= 9 && tokens[6] == "left" {
                     return Some(Ok(Event {
                         ty: Type::Part {
                             nick: tokens[3].to_owned().into(),
@@ -100,7 +102,7 @@ impl<'a> Iterator for Iter<'a> {
                     }));
                 }
                 // 2016-02-25 01:38:55 <-- Foo (host@mask.foo) has quit (Some reason)
-                else if tokens[6] == "quit" {
+                else if len >= 8 && tokens[6] == "quit" {
                     return Some(Ok(Event {
                         ty: Type::Quit {
                             nick: tokens[3].to_owned().into(),
@@ -112,12 +114,12 @@ impl<'a> Iterator for Iter<'a> {
                         channel: self.context.channel.clone().map(Into::into),
                     }));
                 }
-            } else if tokens[2] == "--" {
+            } else if len >= 3 && tokens[2] == "--" {
                 // 2016-02-25 04:32:15	--	Notice(playbot-veno): ""
-                if tokens[3].starts_with("Notice(") {
+                if len >= 5 && tokens[3].starts_with("Notice(") {
                     return Some(Ok(Event {
                         ty: Type::Notice {
-                            from: tokens[3]["Notice(".len()..tokens.len() - 2].to_owned().into(),
+                            from: tokens[3]["Notice(".len()..tokens[3].len() - 2].to_owned().into(),
                             content: rejoin(&tokens[4..], &split_tokens[4..]),
                         },
                         time: parse_time(&self.context, tokens[0], tokens[1]),
@@ -125,8 +127,8 @@ impl<'a> Iterator for Iter<'a> {
                     }));
                 }
                 // 2014-07-11 15:00:03	--	irc: disconnected from server
-                else if tokens[3] == "irc:" && tokens[4] == "disconnected" && tokens[5] == "from" &&
-                   tokens[6] == "server" {
+                else if len >= 7 && tokens[3] == "irc:" && tokens[4] == "disconnected" &&
+                   tokens[5] == "from" && tokens[6] == "server" {
                     return Some(Ok(Event {
                         ty: Type::Disconnect,
                         time: parse_time(&self.context, tokens[0], tokens[1]),
@@ -135,7 +137,7 @@ impl<'a> Iterator for Iter<'a> {
                 }
                 // 2014-07-11 15:00:03	--	Foo|afk is now known as Foo
                 // 2015-05-09 13:56:05	--	You are now known as foo
-                else if tokens[5] == "now" && tokens[6] == "known" && tokens[7] == "as" &&
+                else if len >= 9 && tokens[5] == "now" && tokens[6] == "known" && tokens[7] == "as" &&
                    (tokens[4] == "is" || tokens[4] == "are") {
                     return Some(Ok(Event {
                         ty: Type::Nick {
-- 
cgit v1.2.3