committed
phcoder wrote:
Robert Millan wrote:
On Sun, Mar 22, 2009 at 02:09:52PM +0100, phcoder wrote:
- grub_disk_cache_invalidate (dev_id, disk_id, sector);
- index = grub_disk_cache_get_index (dev_id, disk_id, sector);
cache = grub_disk_cache_table + index;
+ + cache->lock = 1;
+
Robert Millan wrote:
On Sun, Mar 22, 2009 at 02:09:52PM +0100, phcoder wrote:
- grub_disk_cache_invalidate (dev_id, disk_id, sector);
-
index = grub_disk_cache_get_index (dev_id, disk_id, sector);
cache = grub_disk_cache_table + index;
+
+ cache->lock = 1;
+ grub_free (cache->da
On Sun, Mar 22, 2009 at 02:09:52PM +0100, phcoder wrote:
>
> - grub_disk_cache_invalidate (dev_id, disk_id, sector);
> -
>index = grub_disk_cache_get_index (dev_id, disk_id, sector);
>cache = grub_disk_cache_table + index;
> +
> + cache->lock = 1;
> + grub_free (cache->data);
> +
Rediffed
2009-03-22 Vladimir Serbinenko
Leak fixes
* kern/disk.c (grub_disk_cache_store): Invalidate previous cache
in case of collision
* disk/scsi.c (grub_scsi_open): free scsi in case of error
phcoder wrote:
Hello I discovered some memory leaks. Here is the fix
====
--- ChangeLog (revision 2023)
+++ ChangeLog (working copy)
@@ -1,3 +1,11 @@
+2009-03-09 Vladimir Serbinenko
+
+ Leak fixes
+
+ * kern/disk.c (grub_disk_cache_store): Invalidate previous cache
+ in case of collision
+ * disk/scsi.c (grub_