From: Nathan Chancellor <nathan@kernel.org>
To: akpm@linux-foundation.org, sjenning@redhat.com,
ddstreet@ieee.org, vitaly.wool@konsulko.com
Cc: ndesaulniers@google.com, trix@redhat.com, nphamcs@gmail.com,
linux-mm@kvack.org, llvm@lists.linux.dev,
patches@lists.linux.dev, kernel test robot <lkp@intel.com>,
Nathan Chancellor <nathan@kernel.org>
Subject: [PATCH] zswap: Avoid uninitialized use of ret in zswap_frontswap_store()
Date: Thu, 01 Jun 2023 08:26:16 -0700 [thread overview]
Message-ID: <20230601-zswap-cgroup-wsometimes-uninitialized-v1-1-35debdd19293@kernel.org> (raw)
Clang warns:
mm/zswap.c:1183:6: error: variable 'ret' is used uninitialized whenever 'if' condition is true [-Werror,-Wsometimes-uninitialized]
1183 | if (objcg && !obj_cgroup_may_zswap(objcg))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/zswap.c:1327:9: note: uninitialized use occurs here
1327 | return ret;
| ^~~
mm/zswap.c:1183:2: note: remove the 'if' if its condition is always false
1183 | if (objcg && !obj_cgroup_may_zswap(objcg))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1184 | goto reject;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/zswap.c:1158:9: note: initialize the variable 'ret' to silence this warning
1158 | int ret;
| ^
| = 0
1 error generated.
Set ret to -EINVAL, which seems to best match the situation the comment
mentions and matches an earlier bailout due to an unsupported type or
situation.
Fixes: 6804144bf1cf ("zswap: do not shrink if cgroup may not zswap")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/202306011435.2BxsHFUE-lkp@intel.com/
Closes: https://lore.kernel.org/202306012152.o4FL7Y6J-lkp@intel.com/
Closes: https://github.com/ClangBuiltLinux/linux/issues/1861
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
---
mm/zswap.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/mm/zswap.c b/mm/zswap.c
index cff93643a6ab..e8313e2f5807 100644
--- a/mm/zswap.c
+++ b/mm/zswap.c
@@ -1180,8 +1180,10 @@ static int zswap_frontswap_store(unsigned type, pgoff_t offset,
* local cgroup limits.
*/
objcg = get_obj_cgroup_from_page(page);
- if (objcg && !obj_cgroup_may_zswap(objcg))
+ if (objcg && !obj_cgroup_may_zswap(objcg)) {
+ ret = -EINVAL;
goto reject;
+ }
/* reclaim space if needed */
if (zswap_is_full()) {
---
base-commit: b2424568bd9b947b2698b693ff24fec63bb4b36a
change-id: 20230601-zswap-cgroup-wsometimes-uninitialized-b549ff4247ed
Best regards,
--
Nathan Chancellor <nathan@kernel.org>
next reply other threads:[~2023-06-01 15:26 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-01 15:26 Nathan Chancellor [this message]
2023-06-01 18:09 ` [PATCH] zswap: Avoid uninitialized use of ret in zswap_frontswap_store() Yosry Ahmed
2023-06-01 18:18 ` Nathan Chancellor
2023-06-01 18:25 ` Yosry Ahmed
2023-06-01 18:46 ` Nhat Pham
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230601-zswap-cgroup-wsometimes-uninitialized-v1-1-35debdd19293@kernel.org \
--to=nathan@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=ddstreet@ieee.org \
--cc=linux-mm@kvack.org \
--cc=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=ndesaulniers@google.com \
--cc=nphamcs@gmail.com \
--cc=patches@lists.linux.dev \
--cc=sjenning@redhat.com \
--cc=trix@redhat.com \
--cc=vitaly.wool@konsulko.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.