The code assumed that the ELF contains some sections, but core dumps don't
have any of them, leading to nullptr dereference and out of bound indexing.
PS: Is it the only way to do pull request? I have to upload raw diff?
mwolff | |
vkrause |
The code assumed that the ELF contains some sections, but core dumps don't
have any of them, leading to nullptr dereference and out of bound indexing.
PS: Is it the only way to do pull request? I have to upload raw diff?
When i run the testsuite manually, I have a failure:
********* Start testing of ElfGNUSymbolVersioningTest ********* Config: Using QtTest library 5.5.1, Qt 5.5.1 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 5.3.1 20151207 (Red Hat 5.3.1-2)) PASS : ElfGNUSymbolVersioningTest::initTestCase() FAIL! : ElfGNUSymbolVersioningTest::testSymbolVersioning() 'symDefIndex > 0' returned FALSE. () Loc: [/home/developer/Projects/elf-dissector/tests/auto/elfgnusymbolversioningtest.cpp(60)] PASS : ElfGNUSymbolVersioningTest::testSymbolVersionDefinitions() PASS : ElfGNUSymbolVersioningTest::cleanupTestCase() Totals: 3 passed, 1 failed, 0 skipped, 0 blacklisted ********* Finished testing of ElfGNUSymbolVersioningTest *********
But when i run the tests from within QtCreator, all the test suites pass! (incl. ElfGNUSymbolVersioningTest)
Is there a way to run the autotests via a make command?
No Linters Available |
No Unit Test Coverage |
lib/checks/dependenciescheck.cpp | ||
---|---|---|
50 | the preferred style is const auto *dynamicSection = fileSet->file(i)->dynamicSection(); if (!dynamicSection) continue; foreach (const auto &needed, dynamicSection->neededLibraries()) { |
Hi,
I didn't take/want the kde dev account, and i was told to make sure that "one of the developers can push it for
you" == true.
Which is still not enough information: I would like to know if someone has actually pushed this review to mister master.
From here it looks accepted, but not merged.
git fetch github says: not here yet.