]> git.baikalelectronics.ru Git - kernel.git/commitdiff
mm/damon/dbgfs: remove an unnecessary variable
authorSeongJae Park <sj@kernel.org>
Fri, 14 Jan 2022 22:10:41 +0000 (14:10 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 15 Jan 2022 14:30:33 +0000 (16:30 +0200)
Patch series "mm/damon: Hide unnecessary information disclosures".

DAMON is exposing some unnecessary information including kernel pointer
in kernel log and tracepoint.  This patchset hides such information.
The first patch is only for a trivial cleanup, though.

This patch (of 4):

This commit removes a unnecessarily used variable in
dbgfs_target_ids_write().

Link: https://lkml.kernel.org/r/20211229131016.23641-1-sj@kernel.org
Link: https://lkml.kernel.org/r/20211229131016.23641-2-sj@kernel.org
Fixes: ba056afad928 ("mm/damon: implement a debugfs-based user space interface")
Signed-off-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/damon/dbgfs.c

index 751c7b8356848563f15004ed0adf47b515a1cf38..5b899601e56c3fd13b434d118ea0e50cb7ab73ff 100644 (file)
@@ -364,7 +364,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
        struct damon_ctx *ctx = file->private_data;
        struct damon_target *t, *next_t;
        bool id_is_pid = true;
-       char *kbuf, *nrs;
+       char *kbuf;
        unsigned long *targets;
        ssize_t nr_targets;
        ssize_t ret;
@@ -374,14 +374,13 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
        if (IS_ERR(kbuf))
                return PTR_ERR(kbuf);
 
-       nrs = kbuf;
        if (!strncmp(kbuf, "paddr\n", count)) {
                id_is_pid = false;
                /* target id is meaningless here, but we set it just for fun */
                scnprintf(kbuf, count, "42    ");
        }
 
-       targets = str_to_target_ids(nrs, count, &nr_targets);
+       targets = str_to_target_ids(kbuf, count, &nr_targets);
        if (!targets) {
                ret = -ENOMEM;
                goto out;