util.sasl.scram: Parsing client-final-message in a more strict way. (thanks Marc Santamaria)
authorTobias Markmann <tm@ayena.de>
Sat, 22 May 2010 14:47:21 +0200
changeset 3077 3ee311f21d54
parent 3076 d19b2db64496
child 3078 22c22f8a6eb8
util.sasl.scram: Parsing client-final-message in a more strict way. (thanks Marc Santamaria)
util/sasl/scram.lua
--- a/util/sasl/scram.lua	Sat May 22 13:59:58 2010 +0200
+++ b/util/sasl/scram.lua	Sat May 22 14:47:21 2010 +0200
@@ -153,10 +153,7 @@
 			-- we are processing client_final_message
 			local client_final_message = message;
 			
-			-- TODO: more strict parsing of client_final_message
-			self.state["proof"] = client_final_message:match("p=(.+)");
-			self.state["nonce"] = client_final_message:match("r=(.+),p=");
-			self.state["channelbinding"] = client_final_message:match("c=(.+),r=");
+			self.state["channelbinding"], self.state["nonce"], self.state["proof"] = client_final_message:match("^c=(.*),r=(.*),.*p=(.*)");
 	
 			if not self.state.proof or not self.state.nonce or not self.state.channelbinding then
 				return "failure", "malformed-request", "Missing an attribute(p, r or c) in SASL message.";