Merge pull request cpp-netlib#553 from cnagune/0.11-devel-fix-decode-percent

fix boost::network::uri::decode error - out of range because of '%'

Author: deanberris

boost/network/uri/decode.hpp

@@ -10,6 +10,7 @@
 #include <boost/range/begin.hpp>
 #include <boost/range/end.hpp>
 #include <cassert>
+#include <stdexcept>
 
 namespace boost {
 namespace network {
@@ -58,9 +59,9 @@ OutputIterator decode(const InputIterator &in_begin,
   OutputIterator out = out_begin;
   while (it != in_end) {
     if (*it == '%') {
-      ++it;
+	  if (++it == in_end) throw std::out_of_range("unexpected end of stream");
       value_type v0 = detail::letter_to_hex(*it);
-      ++it;
+	  if (++it == in_end) throw std::out_of_range("unexpected end of stream");
       value_type v1 = detail::letter_to_hex(*it);
       ++it;
       *out++ = 0x10 * v0 + v1;

libs/network/test/uri/uri_encoding_test.cpp

@@ -47,3 +47,10 @@ BOOST_AUTO_TEST_CASE(decoding_multibyte_test) {
   uri::decode(encoded, std::back_inserter(instance));
   BOOST_CHECK_EQUAL(instance, unencoded);
 }
+
+BOOST_AUTO_TEST_CASE(decoding_throw_test) {
+  const std::string encoded("%");
+
+  std::string instance;
+  BOOST_CHECK_THROW(uri::decoded(encoded), std::out_of_range);
+}