widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/ai_coverity into lp:widelands

To: mp+294132@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Mon, 09 May 2016 19:51:31 -0000
In-reply-to: <20160509142323.32147.42782.launchpad@ackee.canonical.com>
Reply-to: mp+294132@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Oh, sorry, coverity said:

 Variable "path" going out of scope leaks the storage it points to.

The path is initialized here: http://bazaar.launchpad.net/~widelands-dev/widelands/ai_coverity/view/head:/src/ai/defaultai.cc#L2975

I am simply not that good in C++ to say if it is enough to get rid of the variable
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_coverity/+merge/294132
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_coverity into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/ai_coverity into lp:widelands
From: TiborB, 2016-05-09