seg fault in doxygen.cpp (1.8.8 / 1.8.9.1)

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

seg fault in doxygen.cpp (1.8.8 / 1.8.9.1)

johnk
There are numerous debug statements in doxygen.cpp using an unchecked
templSpec.data() which causes seg faults.  Should be changed to:

templSpec.isEmpty()?"":templSpec.data()


e.g.
           Debug::print(Debug::Classes,0,
                        "    New undocumented base class `%s'
baseClassName=%s templSpec=%s isArtificial=%d\n",
biName.data(),baseClassName.data(),templSpec.data(),isArtificial
                       );


becomes


           Debug::print(Debug::Classes,0,
                        "    New undocumented base class `%s'
baseClassName=%s templSpec=%s isArtificial=%d\n",
biName.data(),baseClassName.data(),templSpec.isEmpty()?"":templSpec.data(),isArtificial
                       );


------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Doxygen-develop mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/doxygen-develop
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: seg fault in doxygen.cpp (1.8.8 / 1.8.9.1)

Dimitri van Heesch-2
Hi Johnk,

I've pushed this commit to deal with this in a more structural way:
https://github.com/doxygen/doxygen/commit/0831c71c05c9204839e187759f13303e64783730
(note that on most platforms printing a NULL pointer does not result in a segfault).

Regards,
  Dimitri

> On 02 Apr 2015, at 21:07 , johnk <[hidden email]> wrote:
>
> There are numerous debug statements in doxygen.cpp using an unchecked
> templSpec.data() which causes seg faults.  Should be changed to:
>
> templSpec.isEmpty()?"":templSpec.data()
>
>
> e.g.
>           Debug::print(Debug::Classes,0,
>                        "    New undocumented base class `%s'
> baseClassName=%s templSpec=%s isArtificial=%d\n",
> biName.data(),baseClassName.data(),templSpec.data(),isArtificial
>                       );
>
>
> becomes
>
>
>           Debug::print(Debug::Classes,0,
>                        "    New undocumented base class `%s'
> baseClassName=%s templSpec=%s isArtificial=%d\n",
> biName.data(),baseClassName.data(),templSpec.isEmpty()?"":templSpec.data(),isArtificial
>                       );
>
>
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming The Go Parallel Website, sponsored
> by Intel and developed in partnership with Slashdot Media, is your hub for all
> things parallel software development, from weekly thought leadership blogs to
> news, videos, case studies, tutorials and more. Take a look and join the
> conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> Doxygen-develop mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/doxygen-develop


------------------------------------------------------------------------------
BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
Develop your own process in accordance with the BPMN 2 standard
Learn Process modeling best practices with Bonita BPM through live exercises
http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_
source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
_______________________________________________
Doxygen-develop mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/doxygen-develop
Loading...