Don't allow a nested archive pointing to itself
PR binutils/15151 * archive.c (_bfd_find_nested_archive): Don't allow a nested archive pointing to itself. (_bfd_get_elt_at_filepos): Revert the last 2 changes.
This commit is contained in:
parent
f1dd87fcaf
commit
989fbeffb2
2 changed files with 14 additions and 14 deletions
|
@ -1,3 +1,10 @@
|
|||
2013-02-15 H.J. Lu <hongjiu.lu@intel.com>
|
||||
|
||||
PR binutils/15151
|
||||
* archive.c (_bfd_find_nested_archive): Don't allow a nested
|
||||
archive pointing to itself.
|
||||
(_bfd_get_elt_at_filepos): Revert the last 2 changes.
|
||||
|
||||
2013-02-15 Nick Clifton <nickc@redhat.com>
|
||||
|
||||
PR binutils/15140
|
||||
|
|
|
@ -379,6 +379,13 @@ _bfd_find_nested_archive (bfd *arch_bfd, const char *filename)
|
|||
bfd *abfd;
|
||||
const char *target;
|
||||
|
||||
/* PR 15140: Don't allow a nested archive pointing to itself. */
|
||||
if (filename_cmp (filename, arch_bfd->filename) == 0)
|
||||
{
|
||||
bfd_set_error (bfd_error_malformed_archive);
|
||||
return NULL;
|
||||
}
|
||||
|
||||
for (abfd = arch_bfd->nested_archives;
|
||||
abfd != NULL;
|
||||
abfd = abfd->archive_next)
|
||||
|
@ -617,8 +624,6 @@ _bfd_append_relative_path (bfd *arch, char *elt_name)
|
|||
bfd *
|
||||
_bfd_get_elt_at_filepos (bfd *archive, file_ptr filepos)
|
||||
{
|
||||
static file_ptr prev_filepos;
|
||||
static unsigned int dup_filepos_count = 0;
|
||||
struct areltdata *new_areldata;
|
||||
bfd *n_nfd;
|
||||
char *filename;
|
||||
|
@ -626,17 +631,6 @@ _bfd_get_elt_at_filepos (bfd *archive, file_ptr filepos)
|
|||
n_nfd = _bfd_look_for_bfd_in_cache (archive, filepos);
|
||||
if (n_nfd)
|
||||
return n_nfd;
|
||||
/* PR15140: Prevent an infinite recursion scanning a malformed nested archive. */
|
||||
if (filepos == prev_filepos)
|
||||
{
|
||||
if (++ dup_filepos_count > 100)
|
||||
{
|
||||
bfd_set_error (bfd_error_malformed_archive);
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
else
|
||||
dup_filepos_count = 0;
|
||||
|
||||
if (0 > bfd_seek (archive, filepos, SEEK_SET))
|
||||
return NULL;
|
||||
|
@ -645,7 +639,6 @@ _bfd_get_elt_at_filepos (bfd *archive, file_ptr filepos)
|
|||
return NULL;
|
||||
|
||||
filename = new_areldata->filename;
|
||||
prev_filepos = filepos;
|
||||
|
||||
if (bfd_is_thin_archive (archive))
|
||||
{
|
||||
|
|
Loading…
Reference in a new issue