linux-kernel - Re: [PATCH] kernel/resource: Remove unnecessary initial values of variables

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <65133772ea921_e18a62948f@iweiny-mobl.notmuch>
Date:   Tue, 26 Sep 2023 12:56:34 -0700
From:   Ira Weiny <ira.weiny@...el.com>
To:     Li kunyu <kunyu@...china.com>, <gregkh@...uxfoundation.org>,
        <dan.j.williams@...el.com>, <pasha.tatashin@...een.com>,
        <andriy.shevchenko@...ux.intel.com>, <Jonathan.Cameron@...wei.com>,
        <ira.weiny@...el.com>
CC:     <linux-kernel@...r.kernel.org>, Li kunyu <kunyu@...china.com>
Subject: Re: [PATCH] kernel/resource: Remove unnecessary initial values of
 variables

Li kunyu wrote:
> err, new_res and dr is assigned first, so it does not need to initialize
>  the assignment.

A good commit message will explain _why_ this change is required or
desired.  Changing code just for the sake of changing it is not
productive.

Ira

> 
> Signed-off-by: Li kunyu <kunyu@...china.com>
> ---
>  kernel/resource.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/kernel/resource.c b/kernel/resource.c
> index b1763b2fd7ef..bbd7c113307a 100644
> --- a/kernel/resource.c
> +++ b/kernel/resource.c
> @@ -656,7 +656,7 @@ static int reallocate_resource(struct resource *root, struct resource *old,
>  			       resource_size_t newsize,
>  			       struct resource_constraint *constraint)
>  {
> -	int err=0;
> +	int err;
>  	struct resource new = *old;
>  	struct resource *conflict;
>  
> @@ -1310,7 +1310,7 @@ EXPORT_SYMBOL(__release_region);
>  void release_mem_region_adjustable(resource_size_t start, resource_size_t size)
>  {
>  	struct resource *parent = &iomem_resource;
> -	struct resource *new_res = NULL;
> +	struct resource *new_res;
>  	bool alloc_nofail = false;
>  	struct resource **p;
>  	struct resource *res;
> @@ -1556,7 +1556,7 @@ struct resource *
>  __devm_request_region(struct device *dev, struct resource *parent,
>  		      resource_size_t start, resource_size_t n, const char *name)
>  {
> -	struct region_devres *dr = NULL;
> +	struct region_devres *dr;
>  	struct resource *res;
>  
>  	dr = devres_alloc(devm_region_release, sizeof(struct region_devres),
> -- 
> 2.18.2
>