diff options
| author | Kim Alvefur <zash@zash.se> | 2019-11-02 15:27:53 +0100 |
|---|---|---|
| committer | Kim Alvefur <zash@zash.se> | 2019-11-02 15:27:53 +0100 |
| commit | c66adb1b89d584c2ca07579c81e54043a3193a18 (patch) | |
| tree | 557b60702e044df2ec5f4025fa9ec341d8416214 | |
| parent | f563584fca3f6dff5ee1c98c5679018645f10ac0 (diff) | |
| download | prosody-c66adb1b89d584c2ca07579c81e54043a3193a18.tar.gz prosody-c66adb1b89d584c2ca07579c81e54043a3193a18.zip | |
mod_dialback: Abort early if request is missing addressing attributes
Prevents traceback from passing nil to nameprep()
| -rw-r--r-- | plugins/mod_dialback.lua | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/plugins/mod_dialback.lua b/plugins/mod_dialback.lua index eddc3209..dc843498 100644 --- a/plugins/mod_dialback.lua +++ b/plugins/mod_dialback.lua @@ -93,6 +93,11 @@ module:hook("stanza/jabber:server:dialback:result", function(event) -- he wants to be identified through dialback -- We need to check the key with the Authoritative server local attr = stanza.attr; + if not attr.to or not attr.from then + origin.log("debug", "Missing Dialback addressing (from=%q, to=%q)", attr.from, attr.to); + origin:close("improper-addressing"); + return true; + end local to, from = nameprep(attr.to), nameprep(attr.from); if not hosts[to] then |