Avoid crash in pyexiv2 with some image files.

[mribbons]

Resolves #580

Issue was caused by attempting to read data in ImageUniqueID tag,
even though data is populated.

By only reading the values we need from metadata we avoid the issue
in pyexiv2.

Should also improve performance.

stacktrace of error:

.#0 strlen () at ../sysdeps/x86_64/strlen.S:106
.#1 0x00007fffdb154ce7 in exiv2wrapper::ExifTag::ExifTag(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, Exiv2::Exifdatum*, Exiv2::ExifData*, Exiv2::ByteOrder) () from /usr/lib/python2.7/dist-packages/libexiv2python.so
.#2 0x00007fffdb154eb4 in exiv2wrapper::Image::getExifTag(std::__cxx11::basic_string<char, std::char_traits, std::allocator >) ()
from /usr/lib/python2.7/dist-packages/libexiv2python.so
.#3 0x00007fffdb15f3e6 in boost::python::objects::caller_py_function_impl<boost::python::detail::caller<exiv2wrapper::ExifTag const (exiv2wrapper::Image::)(std::__cxx11::basic_string<char, std::char_traits, std::allocator >), boost::python::default_call_policies, boost::mpl::vector3<exiv2wrapper::ExifTag const, exiv2wrapper::Image&, std::__cxx11::basic_string<char, std::char_traits, std::allocator > > > >::operator()(_object, _object*) ()
from /usr/lib/python2.7/dist-packages/libexiv2python.so
.#4 0x00007ffff52845cd in boost::python::objects::function::call(_object*, _object*) const () from /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
.#5 0x00007ffff52847c8 in ?? () from /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
.#6 0x00007ffff528c823 in boost::python::detail::exception_handler::operator()(boost::function0 const&) const ()
from /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
.#7 0x00007fffdb15dd63 in boost::detail::function::function_obj_invoker2<boost::_bi::bind_t<bool, boost::python::detail::translate_exception<Exiv2::BasicError, void ()(Exiv2::BasicError const&)>, boost::_bi::list3<boost::arg<1>, boost::arg<2>, boost::_bi::value<void ()(Exiv2::BasicError const&)> > >, bool, boost::python::detail::exception_handler const&, boost::function0 const&>::invoke(boost::detail::function::function_buffer&, boost::python::detail::exception_handler const&, boost::function0 const&) () from /usr/lib/python2.7/dist-packages/libexiv2python.so
.#8 0x00007ffff528c7f8 in boost::python::detail::exception_handler::operator()(boost::function0 const&) const ()
from /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
.#9 0x00007ffff5d01ab8 in boost::python::detail::translate_exception<ecto::except::NullTendril, void ()(ecto::except::NullTendril const&)>::operator()(boost::python::detail::exception_handler const&, boost::function0 const&, void ()(ecto::except::NullTendril const&)) const ()
from /home/mribbons/OpenDroneMap170517/SuperBuild/install/lib/python2.7/dist-packages/ecto/ecto_main.so
.#10 0x00007ffff5d0099f in bool boost::_bi::list3<boost::arg<1>, boost::arg<2>, boost::_bi::value<void ()(ecto::except::NullTendril const&)> >::operator()<bool, boost::python::detail::translate_exception<ecto::except::NullTendril, void ()(ecto::except::NullTendril const&)>, boost::_bi::list2<boost::python::detail::exception_handler const&, boost::function0 const&> >(boost::_bi::type, boost::python::detail::translate_exception<ecto::except::NullTendril, void ()(ecto::except::NullTendril const&)>&, boost::_bi::list2<boost::python::detail::exception_handler const&, boost::function0 const&>&, long) ()
from /home/mribbons/OpenDroneMap170517/SuperBuild/install/lib/python2.7/dist-packages/ecto/ecto_main.so
.#11 0x00007ffff5cffbe7 in bool boost::_bi::bind_t<bool, boost::python::detail::translate_exception<ecto::except::NullTendril, void ()(ecto::except::NullTendril const&)>, boost::_bi::list3<boost::arg<1>, boost::arg<2>, boost::_bi::value<void ()(ecto::except::NullTendril const&)> > >::operator()<boost::python::detail::exception_handler, boost::function0 >(boost::python::detail::exception_handler const&, boost::function0 const&) ()
from /home/mribbons/OpenDroneMap170517/SuperBuild/install/lib/python2.7/dist-packages/ecto/ecto_main.so
.#12 0x00007ffff5cfeb54 in boost::detail::function::function_obj_invoker2<boost::_bi::bind_t<bool, boost::python::detail::translate_exception<ecto::except::NullTendril, void ()(ecto::except::NullTendril const&)>, boost::_bi::list3<boost::arg<1>, boost::arg<2>, boost::_bi::value<void (*)(ecto::except::NullTendril const&)> > >, bool, boost::python::detail::exception_handler const&, boost::function0 const&>::invoke(boost::detail::function::function_buffer&, boost::python::detail::exception_handler const&, boost::function0 const&) ()
from /home/mribbons/OpenDroneMap170517/SuperBuild/install/lib/python2.7/dist-packages/ecto/ecto_main.so
.#13 0x00007ffff528c7f8 in boost::python::detail::exception_handler::operator()(boost::function0 const&) const ()
from /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
...

[pierotofy]

Nice! Looks good to me.

[dakotabenjamin]

Any reason why you changed the file permissions? Otherwise looks good. I'll test and merge.

[mribbons]

Ok fixed that, didn't realise I could just push straight to my master branch to have this PR updated.

The permission change was caused by modifying the files over a poorly configured samba share.