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
| ||
|
Date: Fri, 2 May 2014 11:15:11 -0500 From: Seth Jennings <sjennings@...iantweb.net> To: Fabian Frederick <fabf@...net.be> Cc: linux-kernel <linux-kernel@...r.kernel.org>, akpm <akpm@...ux-foundation.org> Subject: Re: [PATCH 1/1] MM/ZBUD: make size unsigned like unique callsite On Wed, Apr 30, 2014 at 09:18:36PM +0200, Fabian Frederick wrote: > zbud_alloc is only called by zswap_frontswap_store with unsigned int len. > Change function parameter + update >= 0 check. Thanks for the cleanup! Acked-by: Seth Jennings <sjennings@...iantweb.net> > > Cc: Seth Jennings <sjenning@...ux.vnet.ibm.com> > Cc: Andrew Morton <akpm@...ux-foundation.org> > Signed-off-by: Fabian Frederick <fabf@...net.be> > --- > include/linux/zbud.h | 2 +- > mm/zbud.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/linux/zbud.h b/include/linux/zbud.h > index 2571a5c..13af0d4 100644 > --- a/include/linux/zbud.h > +++ b/include/linux/zbud.h > @@ -11,7 +11,7 @@ struct zbud_ops { > > struct zbud_pool *zbud_create_pool(gfp_t gfp, struct zbud_ops *ops); > void zbud_destroy_pool(struct zbud_pool *pool); > -int zbud_alloc(struct zbud_pool *pool, int size, gfp_t gfp, > +int zbud_alloc(struct zbud_pool *pool, unsigned int size, gfp_t gfp, > unsigned long *handle); > void zbud_free(struct zbud_pool *pool, unsigned long handle); > int zbud_reclaim_page(struct zbud_pool *pool, unsigned int retries); > diff --git a/mm/zbud.c b/mm/zbud.c > index 9451361..01df13a 100644 > --- a/mm/zbud.c > +++ b/mm/zbud.c > @@ -247,7 +247,7 @@ void zbud_destroy_pool(struct zbud_pool *pool) > * gfp arguments are invalid or -ENOMEM if the pool was unable to allocate > * a new page. > */ > -int zbud_alloc(struct zbud_pool *pool, int size, gfp_t gfp, > +int zbud_alloc(struct zbud_pool *pool, unsigned int size, gfp_t gfp, > unsigned long *handle) > { > int chunks, i, freechunks; > @@ -255,7 +255,7 @@ int zbud_alloc(struct zbud_pool *pool, int size, gfp_t gfp, > enum buddy bud; > struct page *page; > > - if (size <= 0 || gfp & __GFP_HIGHMEM) > + if (!size || (gfp & __GFP_HIGHMEM)) > return -EINVAL; > if (size > PAGE_SIZE - ZHDR_SIZE_ALIGNED - CHUNK_SIZE) > return -ENOSPC; > -- > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists