[valgrind] gazebo::common::ModelDatabase::GetModelPath (Conditional jump uninitialised value)

[osrf-migration]

Original report (archived issue) by Jose Luis Rivero (Bitbucket: Jose Luis Rivero, GitHub: j-rivero).

---

While running valgrind in gazebo testing suite: Sensor_TEST.UpdateAfterReset

==31786== Thread 3:
==31786== Conditional jump or move depends on uninitialised value(s)
==31786==    at 0xC06A4E0: inflateReset2 (in /lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==31786==    by 0xC06A5D8: inflateInit2_ (in /lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==31786==    by 0x80A76D3: boost::iostreams::detail::zlib_base::do_init(boost::iostreams::zlib_params const&, bool, void* (*)(void*, unsigned int, unsigned int), void (*)(void*, void*), void*) (in /usr/lib/libboost_iostreams.so.1.46.1)
==31786==    by 0x4F692C9: void boost::iostreams::detail::zlib_base::init<std::allocator<char> >(boost::iostreams::zlib_params const&, bool, boost::iostreams::detail::zlib_allocator<std::allocator<char>, boost::iostreams::detail::zlib_allocator_traits<std::allocator<char> >::type>&) (zlib.hpp:185)
==31786==    by 0x4F681C0: boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >::zlib_decompressor_impl(boost::iostreams::zlib_params const&) (zlib.hpp:365)
==31786==    by 0x4F673BA: boost::iostreams::symmetric_filter<boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >, std::allocator<char> >::impl::impl<boost::iostreams::zlib_params>(int, boost::iostreams::zlib_params const&) (in /home/jrivero/code/gazebo/build/gazebo/common/libgazebo_common.so.1.8.2)
==31786==    by 0x4F667D7: boost::iostreams::symmetric_filter<boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >, std::allocator<char> >::symmetric_filter<boost::iostreams::zlib_params>(int, boost::iostreams::zlib_params const&) (in /home/jrivero/code/gazebo/build/gazebo/common/libgazebo_common.so.1.8.2)
==31786==    by 0x4F6566B: boost::iostreams::basic_zlib_decompressor<std::allocator<char> >::basic_zlib_decompressor(boost::iostreams::zlib_params const&, int) (zlib.hpp:416)
==31786==    by 0x4F64597: boost::iostreams::basic_gzip_decompressor<std::allocator<char> >::basic_gzip_decompressor(int, int) (gzip.hpp:736)
==31786==    by 0x4F60136: gazebo::common::ModelDatabase::GetModelPath(std::string const&, bool) (ModelDatabase.cc:488)
==31786==    by 0x4F862D4: gazebo::common::SystemPaths::FindFileURI(std::string const&) (SystemPaths.cc:278)
==31786==    by 0x4F5F698: gazebo::common::ModelDatabase::GetModelPath(std::string const&, bool) (ModelDatabase.cc:396)
==31786==  Uninitialised value was created by a heap allocation
==31786==    at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==31786==    by 0xC06A5B6: inflateInit2_ (in /lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==31786==    by 0x80A76D3: boost::iostreams::detail::zlib_base::do_init(boost::iostreams::zlib_params const&, bool, void* (*)(void*, unsigned int, unsigned int), void (*)(void*, void*), void*) (in /usr/lib/libboost_iostreams.so.1.46.1)
==31786==    by 0x4F692C9: void boost::iostreams::detail::zlib_base::init<std::allocator<char> >(boost::iostreams::zlib_params const&, bool, boost::iostreams::detail::zlib_allocator<std::allocator<char>, boost::iostreams::detail::zlib_allocator_traits<std::allocator<char> >::type>&) (zlib.hpp:185)
==31786==    by 0x4F681C0: boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >::zlib_decompressor_impl(boost::iostreams::zlib_params const&) (zlib.hpp:365)
==31786==    by 0x4F673BA: boost::iostreams::symmetric_filter<boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >, std::allocator<char> >::impl::impl<boost::iostreams::zlib_params>(int, boost::iostreams::zlib_params const&) (in /home/jrivero/code/gazebo/build/gazebo/common/libgazebo_common.so.1.8.2)
==31786==    by 0x4F667D7: boost::iostreams::symmetric_filter<boost::iostreams::detail::zlib_decompressor_impl<std::allocator<char> >, std::allocator<char> >::symmetric_filter<boost::iostreams::zlib_params>(int, boost::iostreams::zlib_params const&) (in /home/jrivero/code/gazebo/build/gazebo/common/libgazebo_common.so.1.8.2)
==31786==    by 0x4F6566B: boost::iostreams::basic_zlib_decompressor<std::allocator<char> >::basic_zlib_decompressor(boost::iostreams::zlib_params const&, int) (zlib.hpp:416)
==31786==    by 0x4F64597: boost::iostreams::basic_gzip_decompressor<std::allocator<char> >::basic_gzip_decompressor(int, int) (gzip.hpp:736)
==31786==    by 0x4F60136: gazebo::common::ModelDatabase::GetModelPath(std::string const&, bool) (ModelDatabase.cc:488)
==31786==    by 0x4F862D4: gazebo::common::SystemPaths::FindFileURI(std::string const&) (SystemPaths.cc:278)
==31786==    by 0x4F5F698: gazebo::common::ModelDatabase::GetModelPath(std::string const&, bool) (ModelDatabase.cc:396)

[osrf-migration]

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

---

• changed state from "new" to "on hold"

On hold until after June

[osrf-migration]

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).

---

• changed state from "on hold" to "open"

[osrf-migration]

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).

---

• changed priority from "major" to "trivial"

[osrf-migration]

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

---

• set version to "all"

[osrf-migration]

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

---

• changed state from "open" to "new"