btrfs-progs: send-stream: check command length before reading from stream
authorDavid Sterba <dsterba@suse.com>
Tue, 15 Nov 2016 13:38:19 +0000 (14:38 +0100)
committerDavid Sterba <dsterba@suse.com>
Wed, 23 Nov 2016 09:49:57 +0000 (10:49 +0100)
The command + header length could not fit to the intermediate buffer.

Signed-off-by: David Sterba <dsterba@suse.com>
send-stream.c

index 502e43e..450854f 100644 (file)
@@ -82,6 +82,7 @@ static int read_cmd(struct btrfs_send_stream *sctx)
 
        memset(sctx->cmd_attrs, 0, sizeof(sctx->cmd_attrs));
 
+       ASSERT(sizeof(*sctx->cmd_hdr) <= sizeof(sctx->read_buf));
        ret = read_buf(sctx, sctx->read_buf, sizeof(*sctx->cmd_hdr));
        if (ret < 0)
                goto out;
@@ -95,6 +96,13 @@ static int read_cmd(struct btrfs_send_stream *sctx)
        cmd = le16_to_cpu(sctx->cmd_hdr->cmd);
        cmd_len = le32_to_cpu(sctx->cmd_hdr->len);
 
+       if (cmd_len + sizeof(*sctx->cmd_hdr) >= sizeof(sctx->read_buf)) {
+               ret = -EINVAL;
+               error("command length %d too big for buffer %zu",
+                               cmd_len, sizeof(sctx->read_buf));
+               goto out;
+       }
+
        data = sctx->read_buf + sizeof(*sctx->cmd_hdr);
        ret = read_buf(sctx, data, cmd_len);
        if (ret < 0)