lib/oe/qa: add explicit exception for 'file isn't an ELF'

(From OE-Core rev: 4c1fe0cbcb98b0a69ad5b3a04432055d773ee4ba)

Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Ross Burton
2016-02-24 13:31:40 +00:00
committed by Richard Purdie
parent 4553bb1b88
commit 4495e8bae0
3 changed files with 11 additions and 9 deletions

View File

@@ -792,9 +792,8 @@ def package_qa_walk(warnfuncs, errorfuncs, skip, package, d):
elf = oe.qa.ELFFile(path)
try:
elf.open()
except (IOError, ValueError):
except (IOError, oe.qa.NotELFFileError):
# IOError can happen if the packaging control files disappear,
# ValueError means the file isn't an ELF.
elf = None
for func in warnfuncs:
func(path, package, d, elf, warnings)

View File

@@ -80,7 +80,7 @@ python uninative_changeinterp () {
elf = oe.qa.ELFFile(f)
try:
elf.open()
except ValueError:
except oe.qa.NotELFFileError:
continue
#bb.warn("patchelf-uninative --set-interpreter %s %s" % (d.getVar("UNINATIVE_LOADER", True), f))

View File

@@ -1,3 +1,6 @@
class NotELFFileError(Exception):
pass
class ELFFile:
EI_NIDENT = 16
@@ -23,7 +26,7 @@ class ELFFile:
def my_assert(self, expectation, result):
if not expectation == result:
#print "'%x','%x' %s" % (ord(expectation), ord(result), self.name)
raise ValueError("%s is not an ELF" % self.name)
raise NotELFFileError("%s is not an ELF" % self.name)
def __init__(self, name, bits = 0):
self.name = name
@@ -32,7 +35,7 @@ class ELFFile:
def open(self):
if not os.path.isfile(self.name):
raise ValueError("%s is not a normal file" % self.name)
raise NotELFFileError("%s is not a normal file" % self.name)
self.file = file(self.name, "r")
self.data = self.file.read(ELFFile.EI_NIDENT+4)
@@ -49,24 +52,24 @@ class ELFFile:
self.bits = 64
else:
# Not 32-bit or 64.. lets assert
raise ValueError("ELF but not 32 or 64 bit.")
raise NotELFFileError("ELF but not 32 or 64 bit.")
elif self.bits == 32:
self.my_assert(self.data[ELFFile.EI_CLASS], chr(ELFFile.ELFCLASS32))
elif self.bits == 64:
self.my_assert(self.data[ELFFile.EI_CLASS], chr(ELFFile.ELFCLASS64))
else:
raise ValueError("Must specify unknown, 32 or 64 bit size.")
raise NotELFFileError("Must specify unknown, 32 or 64 bit size.")
self.my_assert(self.data[ELFFile.EI_VERSION], chr(ELFFile.EV_CURRENT) )
self.sex = self.data[ELFFile.EI_DATA]
if self.sex == chr(ELFFile.ELFDATANONE):
raise ValueError("self.sex == ELFDATANONE")
raise NotELFFileError("self.sex == ELFDATANONE")
elif self.sex == chr(ELFFile.ELFDATA2LSB):
self.sex = "<"
elif self.sex == chr(ELFFile.ELFDATA2MSB):
self.sex = ">"
else:
raise ValueError("Unknown self.sex")
raise NotELFFileError("Unknown self.sex")
def osAbi(self):
return ord(self.data[ELFFile.EI_OSABI])