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 736386 - [PATCH] Fix another potential null pointer dereference in src/util.cpp
[PATCH] Fix another potential null pointer dereference in src/util.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-09-10 08:31 UTC by Ettl Martin
Modified: 2014-12-25 16:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed fix (573 bytes, patch)
2014-09-10 08:31 UTC, Ettl Martin
none Details | Review

Description Ettl Martin 2014-09-10 08:31:31 UTC
Created attachment 285799 [details] [review]
proposed fix

Please review the attached patch. It fixes another potential null pointer dereference
in src/util.cpp.

This fixes Coverity Scan Issue CID-49997.

Best regards and many thanks

Martin Ettl
Comment 1 Dimitri van Heesch 2014-09-23 19:07:59 UTC
The proposed fix doesn't change the guard of the loop so it will not work.

I think the intention was:

for (;(srcA=srcAli.current()) && (dstA=dstAli.current());++srcAli,++dstAli)
{
  ...
}
Comment 2 Dimitri van Heesch 2014-12-25 16:02:50 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).