From c601ebccb8a63207789e87c66577bcd7d948db76 Mon Sep 17 00:00:00 2001 From: Vlad Grigorescu Date: Fri, 31 Oct 2014 16:24:48 -0400 Subject: [PATCH] Fix a logic bug with handling quits after the cleanup. --- scripts/base/protocols/mysql/main.bro | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/scripts/base/protocols/mysql/main.bro b/scripts/base/protocols/mysql/main.bro index 924cc09133..0920ef323b 100644 --- a/scripts/base/protocols/mysql/main.bro +++ b/scripts/base/protocols/mysql/main.bro @@ -71,14 +71,11 @@ event mysql_command_request(c: connection, command: count, arg: string) &priorit event mysql_command_request(c: connection, command: count, arg: string) &priority=-5 { - if ( !c?$mysql ) + if ( c?$mysql && c$mysql?$cmd && c$mysql$cmd == "quit" ) { - if ( command == 1 ) - { - # We get no response for quits, so let's just log it now. - Log::write(mysql::LOG, c$mysql); - delete c$mysql; - } + # We get no response for quits, so let's just log it now. + Log::write(mysql::LOG, c$mysql); + delete c$mysql; } }