Correct a bug in the last commit. The whole point of creating a 'done:'
goto target was so the cache could be freed. So free the cache after done: rather then before done: (!) Submitted by: Gavin Atkinson <gavin@ury.york.ac.uk>
This commit is contained in:
parent
cd25d4648f
commit
46d8228e36
@ -211,11 +211,11 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld)
|
||||
goto done;
|
||||
}
|
||||
}
|
||||
r = 0;
|
||||
done:
|
||||
if (cache)
|
||||
munmap(cache, bytes);
|
||||
r = 0;
|
||||
done:
|
||||
return(r);
|
||||
return(r);
|
||||
}
|
||||
|
||||
/* Process the PLT relocations. */
|
||||
|
@ -211,11 +211,11 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld)
|
||||
goto done;
|
||||
}
|
||||
}
|
||||
r = 0;
|
||||
done:
|
||||
if (cache)
|
||||
munmap(cache, bytes);
|
||||
r = 0;
|
||||
done:
|
||||
return(r);
|
||||
return(r);
|
||||
}
|
||||
|
||||
/* Process the PLT relocations. */
|
||||
|
Loading…
Reference in New Issue
Block a user