2022-10-08 10:06:02

by Li zeming

[permalink] [raw]
Subject: [PATCH] proc: Remove unnecessary ‘NULL’ and '0' values

Remove unnecessary initialization assignments, which are used after the
assignment.

Signed-off-by: Li zeming <[email protected]>
---
fs/proc/proc_sysctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index 021e83fe831f..5fc367f0044a 100644
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -694,7 +694,7 @@ static bool proc_sys_fill_cache(struct file *file,
struct dentry *child, *dir = file->f_path.dentry;
struct inode *inode;
struct qstr qname;
- ino_t ino = 0;
+ ino_t ino;
unsigned type = DT_UNKNOWN;

qname.name = table->procname;
@@ -1246,7 +1246,7 @@ static bool get_links(struct ctl_dir *dir,
static int insert_links(struct ctl_table_header *head)
{
struct ctl_table_set *root_set = &sysctl_table_root.default_set;
- struct ctl_dir *core_parent = NULL;
+ struct ctl_dir *core_parent;
struct ctl_table_header *links;
int err;

--
2.18.2


2022-10-09 15:18:45

by Kees Cook

[permalink] [raw]
Subject: Re: [PATCH] proc: Remove unneces sary ‘NULL’ and '0' values

On Sat, Oct 08, 2022 at 05:30:26PM +0800, Li zeming wrote:
> Remove unnecessary initialization assignments, which are used after the
> assignment.

They don't hurt anything, and in fact show the intent of the initial
variable state. Additionally, the compiler will drop it once it finds
the later assignment.

-Kees

--
Kees Cook