2009-02-27 Vladimir Serbinenko <phcoder@gmail.com>

        Don't leak in SCSI code.
        * disk/scsi.c (grub_scsi_close): free `scsi'.

diff --git a/ChangeLog b/ChangeLog
index 711030629472186de808176b7cd08cd931b8f525..206dc8c3df96008db82e18fefbfcba1b22a19c84 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-02-27  Vladimir Serbinenko  <phcoder@gmail.com>
+
+       Don't leak in SCSI code.
+       * disk/scsi.c (grub_scsi_close): free `scsi'.
+
 2009-02-27  Robert Millan  <rmh@aybabtu.com>
 
        * loader/i386/pc/multiboot.c: Move from here ...

diff --git a/disk/scsi.c b/disk/scsi.c
index 5dce28873d18f18611eb2c62533ddb44df4f232b..75b92b4090c63d5e79d59b796a3874ee12182dc6 100644 (file)
--- a/disk/scsi.c
+++ b/disk/scsi.c
@@ -312,7 +312,8 @@ grub_scsi_close (grub_disk_t disk)
   grub_scsi_t scsi;
 
   scsi = disk->data;
-  return scsi->dev->close (scsi);
+  scsi->dev->close (scsi);
+  grub_free (scsi);
 }
 
 static grub_err_t