mirror of
https://github.com/curl/curl.git
synced 2026-05-07 02:07:29 +03:00
2bb5c9b555
Per MQTT 3.1.1 sections 3.13.1 and 3.14.1, PINGRESP and DISCONNECT fixed headers must have remaining_length set to zero. The previous code dispatched to mqtt->nextstate based on the queued state alone without validating remaining_length for these no-payload packet types, allowing a malicious broker to send a PINGRESP with non-zero remaining_length whose trailing bytes would be interpreted as the payload of whatever message type was queued (CONNACK, SUBACK, etc.). The exploitation path turned out to be narrow — curl sends data to the server the user chose to talk to — but the spec violation and the resulting protocol-state error are real. Reject the malformed packets with CURLE_WEIRD_SERVER_REPLY before state dispatch. Reported-by: Raymond Steen <raymond@vortiqxconsilium.com> Found by VORTIQ-X VXF Framework Bug: https://hackerone.com/reports/3702718 Signed-off-by: Raymond Steen <raymond@vortiqxconsilium.com> Closes #21465
59 lines
1.1 KiB
XML
59 lines
1.1 KiB
XML
<?xml version="1.0" encoding="US-ASCII"?>
|
|
<testcase>
|
|
<info>
|
|
<keywords>
|
|
MQTT
|
|
MQTT SUBSCRIBE
|
|
</keywords>
|
|
</info>
|
|
|
|
# Server-side
|
|
<reply>
|
|
<data nocheck="yes">
|
|
hello
|
|
</data>
|
|
|
|
# Send a DISCONNECT with remaining_length=2 after the PUBLISH.
|
|
# MQTT 3.1.1 s. 3.14.1 requires DISCONNECT to have remaining_length == 0.
|
|
# Curl must reject this with CURLE_WEIRD_SERVER_REPLY.
|
|
<servercmd>
|
|
DISCONNECT-malformed TRUE
|
|
</servercmd>
|
|
</reply>
|
|
|
|
# Client-side
|
|
<client>
|
|
<features>
|
|
mqtt
|
|
</features>
|
|
<server>
|
|
mqtt
|
|
</server>
|
|
<name>
|
|
MQTT reject DISCONNECT with nonzero remaining_length
|
|
</name>
|
|
<command option="binary-trace">
|
|
mqtt://%HOSTIP:%MQTTPORT/%TESTNUMBER
|
|
</command>
|
|
</client>
|
|
|
|
# Verify data after the test has been "shot"
|
|
<verify>
|
|
<strippart>
|
|
s/^(.* 00044d5154540402003c000c6375726c).*/$1/
|
|
</strippart>
|
|
<protocol>
|
|
client CONNECT 18 00044d5154540402003c000c6375726c
|
|
server CONNACK 2 20020000
|
|
client SUBSCRIBE 9 000100043232303700
|
|
server SUBACK 3 9003000100
|
|
server PUBLISH c 300c00043232303768656c6c6f0a
|
|
server DISCONNECT-malformed 2 e0020000
|
|
</protocol>
|
|
|
|
# 8 is CURLE_WEIRD_SERVER_REPLY
|
|
<errorcode>
|
|
8
|
|
</errorcode>
|
|
</verify>
|
|
</testcase>
|