search

boostorg / spirit

Boost.org spirit module
http://boost.org/libs/spirit
391 stars 160 forks source link

Coverity static analysis defects #398

Open mloskot opened 5 years ago

mloskot commented 5 years ago

(copied from https://svn.boost.org/trac10/ticket/13331, only Spirit's parts from the attached report)

Boost 1.57.0

A static analysis tool called Coverity found medium and high defects in the boost source code. See attached file for defect type, defect category, filename and line number of defect.

Defect Type Defect Category Line Number Filename
Resource leak Resource leaks 271 /boost/include/boost/spirit/home/classic/core/non_terminal/impl/grammar.ipp
Wrapper object use after free Memory - illegal access 167 /boost/include/boost/spirit/home/classic/core/non_terminal/impl/grammar.ipp
Kojoley commented 5 years ago

Thanks for reporting.

Unfortunately, I see no potential problems on these lines (grammar.ipp#L271, grammar.ipp#L167).

mloskot commented 5 years ago

@Kojoley My aim was to clean some Trac issues and I did not want to close this particular one without letting the Spirit developers know (on Trac, it was assigned to None component, left unnoticed). So, if there actually is no issue, I'd close this. FYI, the Trac ticket has been closed.

Kojoley commented 5 years ago

I understand, thanks again, let leave the issue opened as I might be investigating these warnings in the future.