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 623733 - Incorrect use of uninitialised BriefInfo data
Incorrect use of uninitialised BriefInfo data
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.7.1
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2010-07-07 08:33 UTC by Simon
Modified: 2010-10-09 08:19 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix use of uninitialised BriefInfo (347 bytes, patch)
2010-07-07 08:34 UTC, Simon 		none Details | Review

Description Simon 2010-07-07 08:33:22 UTC 
I noticed an occasional segmentation fault with doxygen-1.7.1 when run on our doc sources, so I ran it through valgrind. There was just a single error in src/definition.cpp:472 where "briefLine" was be checked even though it was uninitialized.

Running valgrind with "--track-origins=yes" reveals this to have been allocation by the same function at line 469. A BriefInfo object is created but it's line number isn't set. Subsequently ClassDef::addMembersToTemplateInstance evaluates briefLine() using uninitialised data.
Comment 1 Simon 2010-07-07 08:34:41 UTC 
Created attachment 165407 [details] [review]
Fix use of uninitialised BriefInfo

Simple one-liner to ensure a newly-created BriefInfo has an initialised (to -1) line number.
Comment 2 Dimitri van Heesch 2010-07-09 20:39:53 UTC 
Thanks for finding this. I'll include your patch in the next subversion update.
Comment 3 Dimitri van Heesch 2010-10-09 08:19:44 UTC 
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.7.2. 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.