smb/client: Fix logically dead code
authorAdvait Dhamorikar <advaitdhamorikar@gmail.com>
Mon, 7 Oct 2024 15:23:34 +0000 (20:53 +0530)
committerSteve French <stfrench@microsoft.com>
Wed, 16 Oct 2024 05:30:52 +0000 (00:30 -0500)
The if condition in collect_sample: can never be satisfied
because of a logical contradiction. The indicated dead code
may have performed some action; that action will never occur.

Fixes: 94ae8c3fee94 ("smb: client: compress: LZ77 code improvements cleanup")
Signed-off-by: Advait Dhamorikar <advaitdhamorikar@gmail.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/smb/client/compress.c

index 63b5a55..766b4de 100644 (file)
@@ -166,7 +166,6 @@ static int collect_sample(const struct iov_iter *iter, ssize_t max, u8 *sample)
        loff_t start = iter->xarray_start + iter->iov_offset;
        pgoff_t last, index = start / PAGE_SIZE;
        size_t len, off, foff;
-       ssize_t ret = 0;
        void *p;
        int s = 0;
 
@@ -193,9 +192,6 @@ static int collect_sample(const struct iov_iter *iter, ssize_t max, u8 *sample)
                                memcpy(&sample[s], p, len2);
                                kunmap_local(p);
 
-                               if (ret < 0)
-                                       return ret;
-
                                s += len2;
 
                                if (len2 < SZ_2K || s >= max - SZ_2K)