Re: [PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
Ping. Regards Andrzej On 08/07/2015 09:59 AM, Andrzej Hajda wrote: > The patch was generated using fixed coccinelle semantic patch > scripts/coccinelle/api/memdup.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 > > Signed-off-by: Andrzej Hajda > --- > fs/omfs/inode.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c > index 3d935c8..02af1a3 100644 > --- a/fs/omfs/inode.c > +++ b/fs/omfs/inode.c > @@ -333,12 +333,11 @@ static int omfs_get_imap(struct super_block *sb) > bh = sb_bread(sb, block++); > if (!bh) > goto nomem_free; > - *ptr = kmalloc(sb->s_blocksize, GFP_KERNEL); > + *ptr = kmemdup(bh->b_data, sb->s_blocksize, GFP_KERNEL); > if (!*ptr) { > brelse(bh); > goto nomem_free; > } > - memcpy(*ptr, bh->b_data, sb->s_blocksize); > if (count < sb->s_blocksize) > memset((void *)*ptr + count, 0xff, > sb->s_blocksize - count); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
Ping. Regards Andrzej On 08/07/2015 09:59 AM, Andrzej Hajda wrote: > The patch was generated using fixed coccinelle semantic patch > scripts/coccinelle/api/memdup.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 > > Signed-off-by: Andrzej Hajda> --- > fs/omfs/inode.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c > index 3d935c8..02af1a3 100644 > --- a/fs/omfs/inode.c > +++ b/fs/omfs/inode.c > @@ -333,12 +333,11 @@ static int omfs_get_imap(struct super_block *sb) > bh = sb_bread(sb, block++); > if (!bh) > goto nomem_free; > - *ptr = kmalloc(sb->s_blocksize, GFP_KERNEL); > + *ptr = kmemdup(bh->b_data, sb->s_blocksize, GFP_KERNEL); > if (!*ptr) { > brelse(bh); > goto nomem_free; > } > - memcpy(*ptr, bh->b_data, sb->s_blocksize); > if (count < sb->s_blocksize) > memset((void *)*ptr + count, 0xff, > sb->s_blocksize - count); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
On Fri, Aug 07, 2015 at 09:59:30AM +0200, Andrzej Hajda wrote: > The patch was generated using fixed coccinelle semantic patch > scripts/coccinelle/api/memdup.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 > > Signed-off-by: Andrzej Hajda Thanks - Acked-by: Bob Copeland -- Bob Copeland %% http://bobcopeland.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
The patch was generated using fixed coccinelle semantic patch scripts/coccinelle/api/memdup.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 Signed-off-by: Andrzej Hajda --- fs/omfs/inode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c index 3d935c8..02af1a3 100644 --- a/fs/omfs/inode.c +++ b/fs/omfs/inode.c @@ -333,12 +333,11 @@ static int omfs_get_imap(struct super_block *sb) bh = sb_bread(sb, block++); if (!bh) goto nomem_free; - *ptr = kmalloc(sb->s_blocksize, GFP_KERNEL); + *ptr = kmemdup(bh->b_data, sb->s_blocksize, GFP_KERNEL); if (!*ptr) { brelse(bh); goto nomem_free; } - memcpy(*ptr, bh->b_data, sb->s_blocksize); if (count < sb->s_blocksize) memset((void *)*ptr + count, 0xff, sb->s_blocksize - count); -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
The patch was generated using fixed coccinelle semantic patch scripts/coccinelle/api/memdup.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 Signed-off-by: Andrzej Hajda a.ha...@samsung.com --- fs/omfs/inode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c index 3d935c8..02af1a3 100644 --- a/fs/omfs/inode.c +++ b/fs/omfs/inode.c @@ -333,12 +333,11 @@ static int omfs_get_imap(struct super_block *sb) bh = sb_bread(sb, block++); if (!bh) goto nomem_free; - *ptr = kmalloc(sb-s_blocksize, GFP_KERNEL); + *ptr = kmemdup(bh-b_data, sb-s_blocksize, GFP_KERNEL); if (!*ptr) { brelse(bh); goto nomem_free; } - memcpy(*ptr, bh-b_data, sb-s_blocksize); if (count sb-s_blocksize) memset((void *)*ptr + count, 0xff, sb-s_blocksize - count); -- 1.9.1 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 24/31] fs/omfs: use kmemdup rather than duplicating its implementation
On Fri, Aug 07, 2015 at 09:59:30AM +0200, Andrzej Hajda wrote: The patch was generated using fixed coccinelle semantic patch scripts/coccinelle/api/memdup.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 Signed-off-by: Andrzej Hajda a.ha...@samsung.com Thanks - Acked-by: Bob Copeland m...@bobcopeland.com -- Bob Copeland %% http://bobcopeland.com/ -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/