# HG changeset patch # User Kim Alvefur # Date 1457887342 -3600 # Node ID 7e659f87973d8d5a4e67118ed978b33d08284529 # Parent a23ca90d1984422bfc7db944234edaa4d53637c2 mod_storage_sql: Add LIMIT clause to queries where only a single row is expected diff -r a23ca90d1984 -r 7e659f87973d plugins/mod_storage_sql.lua --- a/plugins/mod_storage_sql.lua Sun Mar 13 17:38:49 2016 +0100 +++ b/plugins/mod_storage_sql.lua Sun Mar 13 17:42:22 2016 +0100 @@ -134,11 +134,11 @@ function map_store:get(username, key) local ok, result = engine:transaction(function() if type(key) == "string" and key ~= "" then - for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?", host, username or "", self.store, key) do + for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=? LIMIT 1", host, username or "", self.store, key) do return deserialize(row[1], row[2]); end else - for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?", host, username or "", self.store, "") do + for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=? LIMIT 1", host, username or "", self.store, "") do local data = deserialize(row[1], row[2]); return data and data[key] or nil; end @@ -163,7 +163,7 @@ end else local extradata = {}; - for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?", host, username or "", self.store, "") do + for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=? LIMIT 1", host, username or "", self.store, "") do extradata = deserialize(row[1], row[2]); break; end