]> git.mxchange.org Git - friendica.git/commitdiff
proxy_url: Fix extension extraction for URLs containing a . after a ?
authorHypolite Petovan <ben.lort@gmail.com>
Tue, 20 Dec 2016 01:19:26 +0000 (20:19 -0500)
committerRoland Häder <roland@mxchange.org>
Wed, 21 Dec 2016 08:14:09 +0000 (09:14 +0100)
mod/proxy.php

index 45c9e945690b7242938009b584aaeea5ad795bf6..af0f9126164a8813a01778027e7851cec90409b2 100644 (file)
@@ -281,14 +281,14 @@ function proxy_url($url, $writemode = false, $size = '') {
 
        $longpath .= '/' . strtr(base64_encode($url), '+/', '-_');
 
-       // Checking for valid extensions. Only add them if they are safe
-       $pos = strrpos($url, '.');
-       if ($pos) {
-               $extension = strtolower(substr($url, $pos + 1));
-               $pos = strpos($extension, '?');
-               if ($pos) {
-                       $extension = substr($extension, 0, $pos);
-               }
+       // Extract the URL extension, disregarding GET parameters starting with ?
+       $question_mark_pos = strpos($url, '?');
+       if ($question_mark_pos === false) {
+               $question_mark_pos = strlen($url);
+       }
+       $dot_pos = strrpos($url, '.', $question_mark_pos - strlen($url));
+       if ($dot_pos !== false) {
+               $extension = strtolower(substr($url, $dot_pos + 1, $question_mark_pos - ($dot_pos + 1)));
        }
 
        $extensions = array('jpg', 'jpeg', 'gif', 'png');