mirrors/prosody
1
0
Fork 0
mirror of https://github.com/bjc/prosody.git synced 2025-04-03 21:27:38 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

mod_http_file_share: Validate that filename does not contain '/'

Browse source
This commit is contained in:
Kim Alvefur 2021-01-26 14:52:37 +01:00
parent 2c09796dda
commit 71267fd111
2 changed files with 19 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
plugins/mod_http_file_share.lua
View file

@ -36,6 +36,7 @@ end
local upload_errors = errors.init(module.name, namespace, {
access = { "auth"; "forbidden" };
filename = { "modify"; "bad-request", "Invalid filename" };
});
function may_upload(uploader, filename, filesize, filetype) -- > boolean, error
@ -44,6 +45,11 @@ function may_upload(uploader, filename, filesize, filetype) -- > boolean, error
return false, upload_errors.new("access");
end
if not filename or filename:find"/" then
-- On Linux, only '/' and '\0' are invalid in filenames and NUL can't be in XML
return false, upload_errors.new("filename");
end
return true;
end

13
spec/scansion/http_upload.scs
View file

@ -21,6 +21,19 @@ Romeo receives:
</slot>
</iq>
Romeo sends:
<iq to='upload.localhost' type='get' id='ac56d83f-a627-4732-8399-60492d1210b6' xml:lang='en'>
<request content-type='text/plain' filename='invalid/filename.dat' xmlns='urn:xmpp:http:upload:0' size='1000'/>
</iq>
Romeo receives:
<iq id='ac56d83f-a627-4732-8399-60492d1210b6' from='upload.localhost' type='error'>
<error type='modify'>
<bad-request xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'/>
<text xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'>Invalid filename</text>
</error>
</iq>
Romeo disconnects
# recording ended on 2021-01-27T22:10:46Z