lkml.org 
[lkml]   [2001]   [May]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] memory leak in scsi_proc.c (fixed patch error)
Hi,

fixed patch poster earlier. PINE's default editor munged it up. Also changed
the 8 spaces indentation to a tab character.

Sorry about that.

> If someone writes to a scsi adapter's /proc entry and that scsi adapter
> has not defined a proc_info() entry point, proc_scsi_write() will leak a
> page. Furthermore, no sense asking for a page if said proc_info() entry
> point does not exist. This patch fixes the above problem and patches
> cleanly against 2.4.4


--- drivers/scsi/scsi_proc.c.orig Fri May 25 06:01:20 2001
+++ drivers/scsi/scsi_proc.c Fri May 25 06:04:16 2001
@@ -99,6 +99,9 @@
char * page;
char *start;

+ if (hpnt->hostt->proc_info == NULL)
+ ret = -ENOSYS;
+
if (count > PROC_BLOCK_SIZE)
return -EOVERFLOW;

@@ -106,11 +109,9 @@
return -ENOMEM;
copy_from_user(page, buf, count);

- if (hpnt->hostt->proc_info == NULL)
- ret = -ENOSYS;
- else
- ret = hpnt->hostt->proc_info(page, &start, 0, count,
- hpnt->host_no, 1);
+ ret = hpnt->hostt->proc_info(page, &start, 0, count,
+ hpnt->host_no, 1);
+
free_page((ulong) page);
return(ret);
}
-
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 12:53    [W:0.020 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site