fs/pipe: remove redundant initialization of pointer buf
authorColin Ian King <colin.i.king@gmail.com>
Fri, 18 Aug 2023 14:45:56 +0000 (15:45 +0100)
committerChristian Brauner <brauner@kernel.org>
Sat, 19 Aug 2023 11:36:18 +0000 (13:36 +0200)
The pointer buf is being initializated with a value that is never read,
it is being re-assigned later on at the pointer where it is being used.
The initialization is redundant and can be removed. Cleans up clang scan
build warning:

fs/pipe.c:492:24: warning: Value stored to 'buf' during its
initialization is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Message-Id: <20230818144556.1208082-1-colin.i.king@gmail.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/pipe.c

index a388612f0f8617c6fbfe633d135bbfae378c369d..34b7bf4d7adb55127d58b780e8157d8a1629a611 100644 (file)
--- a/fs/pipe.c
+++ b/fs/pipe.c
@@ -489,7 +489,7 @@ pipe_write(struct kiocb *iocb, struct iov_iter *from)
                head = pipe->head;
                if (!pipe_full(head, pipe->tail, pipe->max_usage)) {
                        unsigned int mask = pipe->ring_size - 1;
-                       struct pipe_buffer *buf = &pipe->bufs[head & mask];
+                       struct pipe_buffer *buf;
                        struct page *page = pipe->tmp_page;
                        int copied;