diff options
| author | Chris Mason <clm@fb.com> | 2014-04-26 14:02:03 +0200 |
|---|---|---|
| committer | Chris Mason <clm@fb.com> | 2014-04-26 14:02:03 +0200 |
| commit | cfd4a535b68faf651b238586011f5bae128391c4 (patch) | |
| tree | 03380f86411c67a7ba84d3af697dcdfd01b9f66f /fs | |
| parent | Btrfs: correctly set profile flags on seqlock retry (diff) | |
| download | linux-cfd4a535b68faf651b238586011f5bae128391c4.tar.xz linux-cfd4a535b68faf651b238586011f5bae128391c4.zip | |
Btrfs: limit the path size in send to PATH_MAX
fs_path_ensure_buf is used to make sure our path buffers for
send are big enough for the path names as we construct them.
The buffer size is limited to 32K by the length field in
the struct.
But bugs in the path construction can end up trying to build
a huge buffer, and we'll do invalid memmmoves when the
buffer length field wraps.
This patch is step one, preventing the overflows.
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/send.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index 1ac3ca98c429..eb6537a08c1b 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -349,6 +349,11 @@ static int fs_path_ensure_buf(struct fs_path *p, int len) if (p->buf_len >= len) return 0; + if (len > PATH_MAX) { + WARN_ON(1); + return -ENOMEM; + } + path_len = p->end - p->start; old_buf_len = p->buf_len; |