Re: [PATCH 0/8] target-iSCSI: Adjustments for several function implementations

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

 



On Thu, Feb 22, 2018 at 02:36:24PM +0100, David Disseldorp wrote:
> [cc list trimmed]
> 
> On Wed, 21 Feb 2018 17:45:35 +0100, SF Markus Elfring wrote:
> 
> ...
> > One of these update suggestions resulted in the commit “target: avoid NULL
> > dereference in CHAP auth error path” which is considered for integration
> > into Linux stable versions now.
> > https://patchwork.kernel.org/patch/10110459/
> > https://lkml.kernel.org/r/<20171213172230.12767-1-ddiss@xxxxxxx>
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/target/iscsi/iscsi_target_auth.c?id=ce512d79d0466a604793addb6b769d12ee326822
> 
> You're 1/8 patch had an actual bug fix hidden amongst the style churn.
> I don't see any such fixes in the other patches.
> 

Calling crypto_free_shash(NULL) is actually fine.  It doesn't
dereference the parameter, it just does pointer math on it in
crypto_shash_tfm() and returns if it's NULL in crypto_destroy_tfm().

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