Re: [patch] iscsi-target: remove some dead code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 17, 2012 at 11:58:40AM +0200, walter harms wrote:
> 
> 
> Am 17.05.2012 09:08, schrieb Dan Carpenter:
> > Neither "acceptor_values" nor "proposer_values" can be NULL here.
> > Smatch complains because we are not allowed to pass NULL pointers to
> > strchr().
> > 
> > Also I removed a second later check for "!acceptor_values" because it
> > gets checked on the next line in the do while condition.
> > 
> > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > ---
> > Compile tested only.  Please review carefully.
> > 
> > diff --git a/drivers/target/iscsi/iscsi_target_parameters.c b/drivers/target/iscsi/iscsi_target_parameters.c
> > index ad3b3c1..ed5241e 100644
> > --- a/drivers/target/iscsi/iscsi_target_parameters.c
> > +++ b/drivers/target/iscsi/iscsi_target_parameters.c
> > @@ -1037,13 +1037,6 @@ static char *iscsi_check_valuelist_for_support(
> >  			tmp2 = strchr(acceptor_values, ',');
> >  			if (tmp2)
> >  				*tmp2 = '\0';
> this looks like strchrnul(). I realy do not know is this supported inside the kernel ?
> 

It can't be NULL here.  Adding bogus checks is the wrong idea and
static checkers are right to ask about needless checks.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux