lkml.org 
[lkml]   [2001]   [Dec]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] 2.4.16: kmalloc tidying
Just please have a look on the binfmt_elf one, which im not absolutely sure
on whether there's nothing else to clean up.

--
____/| Ragnar Højland Freedom - Linux - OpenGL | Brainbench MVP
\ o.O| PGP94C4B2F0D27DE025BE2302C104B78C56 B72F0822 | for Unix Programming
=(_)= "Thou shalt not follow the NULL pointer for | (www.brainbench.com)
U chaos and madness await thee at its end."

diff -rup -x '*~' linux-2.4.16-O/drivers/i2c/i2c-proc.c linux-2.4.16/drivers/i2c/i2c-proc.c
--- linux-2.4.16-O/drivers/i2c/i2c-proc.c Tue Nov 13 05:31:33 2001
+++ linux-2.4.16/drivers/i2c/i2c-proc.c Mon Dec 3 16:33:55 2001
@@ -119,6 +119,10 @@ int i2c_create_name(char **name, const c
sprintf(name_buffer, "%s-i2c-%d-%02x", prefix, id, addr);
}
*name = kmalloc(strlen(name_buffer) + 1, GFP_KERNEL);
+ if (!*name) {
+ printk (KERN_WARNING "i2c_create_name: not enough memory\n");
+ return -ENOMEM;
+ }
strcpy(*name, name_buffer);
return 0;
}
diff -rup -x '*~' linux-2.4.16-O/drivers/net/wireless/airo_cs.c linux-2.4.16/drivers/net/wireless/airo_cs.c
--- linux-2.4.16-O/drivers/net/wireless/airo_cs.c Tue Nov 13 05:28:44 2001
+++ linux-2.4.16/drivers/net/wireless/airo_cs.c Mon Dec 3 17:12:32 2001
@@ -244,6 +244,11 @@ static dev_link_t *airo_attach(void)

/* Allocate space for private device-specific data */
local = kmalloc(sizeof(local_info_t), GFP_KERNEL);
+ if (!local) {
+ printk(KERN_ERR "airo_cs: no memory for new device\n");
+ kfree (link);
+ return NULL;
+ }
memset(local, 0, sizeof(local_info_t));
link->priv = local;

diff -rup -x '*~' linux-2.4.16-O/drivers/parport/parport_cs.c linux-2.4.16/drivers/parport/parport_cs.c
--- linux-2.4.16-O/drivers/parport/parport_cs.c Tue Nov 13 05:30:14 2001
+++ linux-2.4.16/drivers/parport/parport_cs.c Mon Dec 3 16:01:09 2001
@@ -311,6 +311,10 @@ void parport_config(dev_link_t *link)
#if (LINUX_VERSION_CODE >= VERSION(2,2,8))
p->private_data = kmalloc(sizeof(struct parport_pc_private),
GFP_KERNEL);
+ if (p->private_data == NULL) {
+ printk(KERN_WARNING "parport_cs: not enough memory\n");
+ goto failed;
+ }
((struct parport_pc_private *)(p->private_data))->ctr = 0x0c;
#endif
parport_proc_register(p);
diff -rup -x '*~' linux-2.4.16-O/drivers/sbus/char/envctrl.c linux-2.4.16/drivers/sbus/char/envctrl.c
--- linux-2.4.16-O/drivers/sbus/char/envctrl.c Mon Dec 3 17:17:13 2001
+++ linux-2.4.16/drivers/sbus/char/envctrl.c Tue Nov 13 05:31:02 2001
@@ -897,10 +897,6 @@ static void envctrl_init_i2c_child(struc
}

pchild->tables = kmalloc(tbls_size, GFP_KERNEL);
- if (!pchild->tables) {
- printk("envctrl: Failed to get table, not enough memory.\n");
- return;
- }
len = prom_getproperty(node, "tables",
(char *) pchild->tables, tbls_size);
if (len <= 0) {
diff -rup -x '*~' linux-2.4.16-O/drivers/sbus/dvma.c linux-2.4.16/drivers/sbus/dvma.c
--- linux-2.4.16-O/drivers/sbus/dvma.c Thu Feb 22 13:40:13 2001
+++ linux-2.4.16/drivers/sbus/dvma.c Mon Dec 3 16:26:05 2001
@@ -90,7 +90,10 @@ void __init dvma_init(struct sbus_bus *s

/* Found one... */
dma = kmalloc(sizeof(struct sbus_dma), GFP_ATOMIC);
-
+ if (!dma) {
+ printk (KERN_WARNING "dvma_init: not enough memory\n");
+ return;
+ }
dma->sdev = this_dev;

/* Put at end of dma chain */
@@ -126,7 +129,12 @@ void __init sun4_dvma_init(void)

if(sun4_dma_physaddr) {
dma = kmalloc(sizeof(struct sbus_dma), GFP_ATOMIC);
-
+ if (!dma) {
+ printk (KERN_WARNING "sun4_dvma_init: not enough memory\n");
+ dma_chain = NULL;
+ return;
+ }
+
/* No SBUS */
dma->sdev = NULL;

diff -rup -x '*~' linux-2.4.16-O/drivers/sound/btaudio.c linux-2.4.16/drivers/sound/btaudio.c
--- linux-2.4.16-O/drivers/sound/btaudio.c Tue Nov 13 05:31:48 2001
+++ linux-2.4.16/drivers/sound/btaudio.c Mon Dec 3 16:09:27 2001
@@ -885,6 +885,12 @@ static int __devinit btaudio_probe(struc
}

bta = kmalloc(sizeof(*bta),GFP_ATOMIC);
+ if (!bta) {
+ printk(KERN_WARNING
+ "btaudio: not enough memory\n");
+ rc = -ENOMEM;
+ goto fail1;
+ }
memset(bta,0,sizeof(*bta));

bta->pci = pci_dev;
diff -rup -x '*~' linux-2.4.16-O/fs/binfmt_elf.c linux-2.4.16/fs/binfmt_elf.c
--- linux-2.4.16-O/fs/binfmt_elf.c Tue Nov 13 05:31:51 2001
+++ linux-2.4.16/fs/binfmt_elf.c Mon Dec 3 17:05:53 2001
@@ -609,7 +609,9 @@ static int load_elf_binary(struct linux_
/* Do this so that we can load the interpreter, if need be. We will
change some of these later */
current->mm->rss = 0;
- setup_arg_pages(bprm); /* XXX: check error */
+ retval = setup_arg_pages(bprm);
+ if (retval)
+ goto out_free_dentry;
current->mm->start_stack = bprm->p;

/* Now we do a little grungy work by mmaping the ELF image into
diff -rup -x '*~' linux-2.4.16-O/fs/qnx4/inode.c linux-2.4.16/fs/qnx4/inode.c
--- linux-2.4.16-O/fs/qnx4/inode.c Tue Nov 13 05:30:44 2001
+++ linux-2.4.16/fs/qnx4/inode.c Mon Dec 3 14:37:14 2001
@@ -318,6 +318,10 @@ static const char *qnx4_checkroot(struct
if (!strncmp(rootdir->di_fname, QNX4_BMNAME, sizeof QNX4_BMNAME)) {
found = 1;
sb->u.qnx4_sb.BitMap = kmalloc( sizeof( struct qnx4_inode_entry ), GFP_KERNEL );
+ if (!sb->u.qnx4_sb.BitMap) {
+ brelse (bh);
+ return "not enough memory for bitmap inode";
+ }
memcpy( sb->u.qnx4_sb.BitMap, rootdir, sizeof( struct qnx4_inode_entry ) ); /* keep bitmap inode known */
break;
}
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:13    [W:0.057 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site