diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2020-04-20 17:03:18 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2020-04-20 17:03:18 +0200 |
| commit | 383cd49b11ae32e136263d7555d7c593fc57bfef (patch) | |
| tree | c25df07a9ae2cd5c2f08e8c5bbf471b4d0041ab6 | |
| parent | df28ed6116862921994b1abc5147cb64360e4241 (diff) | |
| download | curl-bagder/mqtt-break-better.tar.gz | |
mqtt: fix Curl_read() error handling while reading remaining lengthbagder/mqtt-break-better
Detected by Coverity. CID 1462320.
| -rw-r--r-- | lib/mqtt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/mqtt.c b/lib/mqtt.c index 35c1b3e83..44433361b 100644 --- a/lib/mqtt.c +++ b/lib/mqtt.c @@ -569,6 +569,8 @@ static CURLcode mqtt_doing(struct connectdata *conn, bool *done) Curl_debug(data, CURLINFO_HEADER_IN, (char *)&byte, 1); pkt[mq->npacket++] = byte; } while((byte & 0x80) && (mq->npacket < 4)); + if(result) + break; mq->remaining_length = mqtt_decode_len(&pkt[0], mq->npacket, NULL); mq->npacket = 0; if(mq->remaining_length) { |