After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 735477 - [PATCH] Fix a potential null pointer dereference in src/memberdef.cpp
[PATCH] Fix a potential null pointer dereference in src/memberdef.cpp
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.8-GIT
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2014-08-26 19:56 UTC by Ettl Martin
Modified: 2014-12-25 16:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed fix (445 bytes, patch)
2014-08-26 19:56 UTC, Ettl Martin
none Details | Review

Description Ettl Martin 2014-08-26 19:56:50 UTC
Created attachment 284539 [details] [review]
proposed fix

Please review the attached patch, it fixes a potential null pointer dereference in src/memberdef.cpp. 

The statement mli.current() can return 0. 
Here is a traceback:
The function current() looks as followed (qtools/qlist.h, line 146):
type *current()   const   { return (type *)QGListIterator::get(); }

The function QGlistIterator::get() (qtools/qglist.h, 253) looks as followed:

inline QCollection::Item QGListIterator::get() const
{
  return curNode ? curNode->data : 0;
}

As you can see, it can return 0. This patch takes care of this potential issue.

Best regards and many thanks

Martin Ettl
Comment 1 Dimitri van Heesch 2014-08-30 17:14:46 UTC
Thanks, I'll include a variant of the patch in the next GIT update.
Comment 2 Dimitri van Heesch 2014-12-25 16:03:31 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.9. 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 (preferrably in the form of a self-contained example).