diff options
author | Konstantin Käfer <mail@kkaefer.com> | 2016-07-19 16:01:24 +0200 |
---|---|---|
committer | Konstantin Käfer <mail@kkaefer.com> | 2016-07-19 16:01:24 +0200 |
commit | 559f3a654b976d41d0f3db3fa958ebcda3e97b1c (patch) | |
tree | 99853c1b2c19adeb1d3f3bfeb84910abe0fb6d47 | |
parent | 1bc4f0e0273d6be24de56512fd52a147ea74f9e1 (diff) | |
download | qtlocation-mapboxgl-559f3a654b976d41d0f3db3fa958ebcda3e97b1c.tar.gz |
[core] add check for missing sprite URL file extension
-rw-r--r-- | src/mbgl/util/mapbox.cpp | 8 | ||||
-rw-r--r-- | test/util/mapbox.cpp | 6 |
2 files changed, 14 insertions, 0 deletions
diff --git a/src/mbgl/util/mapbox.cpp b/src/mbgl/util/mapbox.cpp index 97c28c0aeb..28dedaf25c 100644 --- a/src/mbgl/util/mapbox.cpp +++ b/src/mbgl/util/mapbox.cpp @@ -83,10 +83,18 @@ std::string normalizeSpriteURL(const std::string& url, const std::string& access std::string id, extension; if (isDraft) { size_t index = pathname[3].find_first_of("@."); + if (index == std::string::npos) { + Log::Error(Event::ParseStyle, "Invalid sprite URL"); + return url; + } id = pathname[2]; extension = pathname[3].substr(index); } else { size_t index = pathname[2].find_first_of("@."); + if (index == std::string::npos) { + Log::Error(Event::ParseStyle, "Invalid sprite URL"); + return url; + } id = pathname[2].substr(0, index); extension = pathname[2].substr(index); } diff --git a/test/util/mapbox.cpp b/test/util/mapbox.cpp index 7f9e7ef592..35c478287c 100644 --- a/test/util/mapbox.cpp +++ b/test/util/mapbox.cpp @@ -66,6 +66,12 @@ TEST(Mapbox, SpriteURL) { EXPECT_EQ( "https://api.mapbox.com/styles/v1/mapbox/streets-v8/draft/sprite@2x.png?access_token=key", mbgl::util::mapbox::normalizeSpriteURL("mapbox://sprites/mapbox/streets-v8/draft@2x.png", "key")); + EXPECT_EQ( + "mapbox://sprites/mapbox/streets-v9?fresh=true.png", + mbgl::util::mapbox::normalizeSpriteURL( + "mapbox://sprites/mapbox/streets-v9?fresh=true.png", + "key")); + EXPECT_EQ("mapbox://////", mbgl::util::mapbox::normalizeSpriteURL("mapbox://////", "key")); } TEST(Mapbox, TileURL) { |