New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

segfault in searchInputFiles if INPUT contains zero-length string (Origin: bugzilla #698007) #5144

Closed
doxygen opened this Issue Jul 2, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@doxygen
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity major in component general for ---
Reported in version 1.8.3.1 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2013-04-14 15:46:50 +0000, Ed Catmur wrote:

Searching for include files...
Searching for example files...
Searching for images...
Searching for dot files...
Searching for msc files...
Searching for files to exclude
Searching for files to process...

Program received signal SIGSEGV, Segmentation fault.
searchInputFiles (inputFiles=...) at doxygen.cpp:10067
10067 if (path.at(l-1)=='\' || path.at(l-1)=='/') path=path.left(l-1);
(gdb) bt full

0 searchInputFiles (inputFiles=...) at doxygen.cpp:10067

    path = {m_data = 0x0}
    alwaysRecursive = false
    killDict = 0x88a2850
    excludeNameDict = {<QDict<QCString>> = {<QGDict> = {<QCollection> = {
            _vptr.QCollection = 0x83a0428 <vtable for StringDict+8>, 
            del_item = true}, vec = 0x889daa8, vlen = 1009, numItems = 0, 
          keytype = 1, cases = 1, copyk = 1, 
          iterators = 0x0}, <No data fields>}, <No data fields>}

1 0x08083320 in parseInput () at doxygen.cpp:10264

    thisDir = {_vptr.QDir = 0x0, dPath = {static null = 
{<No data fields>}, d = 0x839e9ac, static shared_null = 0x879b4b0}, 
      fList = 0x6, fiList = 0xb7f92000, nameFilt = {
        static null = {<No data fields>}, d = 0xc, 
        static shared_null = 0x879b4b0}, filtS = -1209031306, 
      sortS = 142966656, dirty = 0, allDirs = 1}
    latexOutput = {m_data = 0x8895060 "/latex"}
    rtfOutput = {m_data = 0x0}
    layoutFile = {<QIODevice> = {
        _vptr.QIODevice = 0x8669a28 <vtable for QFile+8>, ioIndex = 0, 
        ioMode = 256, ioSt = 5}, fn = {static null = {<No data fields>}, 
        d = 0x8899a78, static shared_null = 0x879b4b0}, fh = 0x0, fd = 0, 
      length = 0, ext_f = false, d = 0xbfffeaf8, ungetchBuffer = {
        m_data = 0x0}}
    xmlOutput = {m_data = 0x0}
    s = 0xffffffff <Address 0xffffffff out of bounds>
    cacheSize = 0
    pid = 1
    defaultLayoutUsed = true
    rootNav = 0x8864104
    htmlOutput = {m_data = 0x88907f0 "/html"}
    manOutput = {m_data = 0x0}
    root = 0x88618f4

2 0x0804c88f in main (argc=1, argv=0xbfffed64) at main.cpp:37

No locals.

Patch to follow.

On 2013-04-14 15:51:25 +0000, Ed Catmur wrote:

Created attachment 241506
0001-Fix-crash-if-zero-length-string-in-INPUT.patch

On 2013-04-14 16:02:25 +0000, Ed Catmur wrote:

Created attachment 241507
https://bugzilla.gnome.org/show_bug.cgi?id=698007

On 2013-04-14 16:09:15 +0000, Kevin McBride wrote:

Thank you for taking the time to report this bug.

Although Dimitri will probably include your patch, could you attach a sample file w. doxyfile in a tar or zip so that bug triagers like myself can confirm this bug?

Because of this question, I am leaving the bug status as "Unconfirmed."

On 2013-04-14 23:53:13 +0000, Ed Catmur wrote:

Sorry, discovered the bug was triggered by stat failure running 32-bit userland on an xfs filesystem with 64-bit inode numbers. I'm looking to see if it may be possible to trigger the bug under more normal circumstances.

On 2013-04-19 18:04:55 +0000, Dimitri van Heesch wrote:

Confirmed. Should be fixed in the next subversion update.

On 2013-05-19 12:35:37 +0000, Dimitri van Heesch wrote:

This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.4. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information
that you think can be relevant.

@doxygen doxygen closed this Jul 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment