Whamcloud - gitweb
e2image: fix overflow in l2 table processing
authorArtem Blagodarenko <artem.blagodarenko@gmail.com>
Thu, 22 Apr 2021 05:24:48 +0000 (01:24 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Fri, 7 May 2021 22:48:45 +0000 (18:48 -0400)
For a large partition during e2image capture process
it is possible to overflow offset at multiply operation.
This leads to the situation when data is written to the
position at the start of the image instead of the image end.

Let's use the right cast to avoid integer overflow.

Signed-off-by: Alexey Lyashkov <c17817@cray.com>
Signed-off-by: Artem Blagodarenko <c17828@cray.com>
HPE-bug-id: LUS-9368
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
lib/ext2fs/qcow2.c

index ee701f7..2082417 100644 (file)
@@ -238,7 +238,7 @@ int qcow2_write_raw_image(int qcow2_fd, int raw_fd,
                        if (offset == 0)
                                continue;
 
-                       off_out = (l1_index * img.l2_size) +
+                       off_out = ((__u64)l1_index * img.l2_size) +
                                  l2_index;
                        off_out <<= img.cluster_bits;
                        ret = qcow2_copy_data(qcow2_fd, raw_fd, offset,