[ABI] Return const std::string& for Statement::getFile()
authorJosh Stone <jistone@redhat.com>
Wed, 26 Jun 2013 20:32:52 +0000 (13:32 -0700)
committerJosh Stone <jistone@redhat.com>
Wed, 26 Jun 2013 20:32:52 +0000 (13:32 -0700)
commitf62b5688c4be399f1b5076857544dc488602bd7a
tree805f393685c89bd1b64ea74bd0e0f83605d6b782
parent1872e3b684d36bfd2ecf5e999f09875097e8cdbd
[ABI] Return const std::string& for Statement::getFile()

This was returning a std::string, and BPatch_statement::fileName() was
then returning c_str() from that.  But since that string was temporary,
the memory backing it will be immediately freed.

The change to const std::string& shouldn't make any practical API
difference to callers, but it is an ABI change. (Though even that is
only if that simple function is ever not inlined...)

Reported by Coverity as WRAPPER_ESCAPE (CWE-825).
symtabAPI/h/Module.h