vbatts/grub
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Be paranoid in relocator allocations

Browse source
This commit is contained in:
Vladimir 'phcoder' Serbinenko 2010-01-15 11:34:22 +01:00
parent 9205ac07e3
commit def6307401
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/relocator.c
View file

@ -163,6 +163,9 @@ get_best_header (struct grub_relocator *rel,
allowable_start = (grub_addr_t) h;
allowable_end = (grub_addr_t) (h + h->size);
if (h->magic != GRUB_MM_FREE_MAGIC)
grub_fatal ("free magic is broken at %p: 0x%x", h, h->magic);
try_addr (allowable_start, allowable_end);
if ((grub_addr_t) h == (grub_addr_t) (rb + 1))
@ -299,7 +302,8 @@ malloc_in_range (struct grub_relocator *rel,
{
struct grub_mm_header *foll = NULL;
if (best_addr + size <= (grub_addr_t) (hb + hb->size))
if (ALIGN_UP (best_addr + size, GRUB_MM_ALIGN) + GRUB_MM_ALIGN
<= (grub_addr_t) (hb + hb->size))
{
foll = (void *) ALIGN_UP (best_addr + size, GRUB_MM_ALIGN);
foll->magic = GRUB_MM_FREE_MAGIC;
@ -324,11 +328,11 @@ malloc_in_range (struct grub_relocator *rel,
else
foll = hb->next;
hbp->next = foll;
if (rb->first == hb)
rb->first = foll;
if (rb->first == hb)
rb->first = (void *) (rb + 1);
hbp->next = foll;
}
*res = best_addr;
return 1;