This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
1 | <?php |
||||||
2 | |||||||
3 | namespace XoopsModules\Xsitemap\Common; |
||||||
4 | |||||||
5 | /* |
||||||
6 | You may not change or alter any portion of this comment or credits |
||||||
7 | of supporting developers from this source code or any supporting source code |
||||||
8 | which is considered copyrighted (c) material of the original comment or credit authors. |
||||||
9 | |||||||
10 | This program is distributed in the hope that it will be useful, |
||||||
11 | but WITHOUT ANY WARRANTY; without even the implied warranty of |
||||||
12 | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. |
||||||
13 | */ |
||||||
14 | |||||||
15 | /** |
||||||
16 | * @copyright XOOPS Project (https://xoops.org) |
||||||
17 | * @license http://www.fsf.org/copyleft/gpl.html GNU public license |
||||||
18 | * @author mamba <[email protected]> |
||||||
19 | */ |
||||||
20 | trait VersionChecks |
||||||
21 | { |
||||||
22 | /** |
||||||
23 | * Verifies XOOPS version meets minimum requirements for this module |
||||||
24 | * @static |
||||||
25 | * @param \XoopsModule|null $module |
||||||
26 | * |
||||||
27 | * @param null|string $requiredVer |
||||||
28 | * @return bool true if meets requirements, false if not |
||||||
29 | */ |
||||||
30 | public static function checkVerXoops(\XoopsModule $module = null, $requiredVer = null) |
||||||
31 | { |
||||||
32 | $moduleDirName = \basename(\dirname(__DIR__)); |
||||||
33 | if (null === $module) { |
||||||
34 | $module = \XoopsModule::getByDirname($moduleDirName); |
||||||
35 | } |
||||||
36 | \xoops_loadLanguage('admin', $moduleDirName); |
||||||
37 | \xoops_loadLanguage('common', $moduleDirName); |
||||||
38 | //check for minimum XOOPS version |
||||||
39 | $currentVer = mb_substr(\XOOPS_VERSION, 6); // get the numeric part of string |
||||||
40 | if (null === $requiredVer) { |
||||||
41 | $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string |
||||||
0 ignored issues
–
show
Bug
introduced
by
Loading history...
|
|||||||
42 | } |
||||||
43 | $success = true; |
||||||
44 | if (\version_compare($currentVer, $requiredVer, '<')) { |
||||||
45 | $success = false; |
||||||
46 | $module->setErrors(\sprintf(\_AM_XSITEMAP_ERROR_BAD_XOOPS, $requiredVer, $currentVer)); |
||||||
47 | } |
||||||
48 | return $success; |
||||||
49 | } |
||||||
50 | |||||||
51 | /** |
||||||
52 | * Verifies PHP version meets minimum requirements for this module |
||||||
53 | * @static |
||||||
54 | * @param \XoopsModule|null $module |
||||||
55 | * |
||||||
56 | * @return bool true if meets requirements, false if not |
||||||
57 | */ |
||||||
58 | public static function checkVerPhp(\XoopsModule $module = null) |
||||||
59 | { |
||||||
60 | $moduleDirName = \basename(\dirname(__DIR__, 2)); |
||||||
61 | $moduleDirNameUpper = mb_strtoupper($moduleDirName); |
||||||
62 | if (null === $module) { |
||||||
63 | $module = \XoopsModule::getByDirname($moduleDirName); |
||||||
64 | } |
||||||
65 | \xoops_loadLanguage('admin', $moduleDirName); |
||||||
66 | \xoops_loadLanguage('common', $moduleDirName); |
||||||
67 | // check for minimum PHP version |
||||||
68 | $success = true; |
||||||
69 | $verNum = \PHP_VERSION; |
||||||
70 | $reqVer = &$module->getInfo('min_php'); |
||||||
71 | if (false !== $reqVer && '' !== $reqVer) { |
||||||
72 | if (\version_compare($verNum, $reqVer, '<')) { |
||||||
0 ignored issues
–
show
It seems like
$reqVer can also be of type array ; however, parameter $version2 of version_compare() does only seem to accept string , maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
73 | $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_PHP'), $reqVer, $verNum)); |
||||||
0 ignored issues
–
show
It seems like
$reqVer can also be of type array ; however, parameter $values of sprintf() does only seem to accept double|integer|string , maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
74 | $success = false; |
||||||
75 | } |
||||||
76 | } |
||||||
77 | return $success; |
||||||
78 | } |
||||||
79 | |||||||
80 | /** |
||||||
81 | * |
||||||
82 | * compares current module version with latest GitHub release |
||||||
83 | * @static |
||||||
84 | * @param \Xmf\Module\Helper $helper |
||||||
85 | * @param string|null $source |
||||||
86 | * @param string|null $default |
||||||
87 | * |
||||||
88 | * @return string|array info about the latest module version, if newer |
||||||
89 | */ |
||||||
90 | public static function checkVerModule($helper, $source = 'github', $default = 'master') |
||||||
91 | { |
||||||
92 | $moduleDirName = \basename(\dirname(__DIR__, 2)); |
||||||
93 | $moduleDirNameUpper = mb_strtoupper($moduleDirName); |
||||||
94 | $update = ''; |
||||||
95 | $repository = 'XoopsModules25x/' . $moduleDirName; |
||||||
96 | // $repository = 'XoopsModules25x/publisher'; //for testing only |
||||||
97 | $ret = ''; |
||||||
98 | $infoReleasesUrl = "https://api.github.com/repos/$repository/releases"; |
||||||
99 | if ('github' === $source) { |
||||||
100 | if (\function_exists('curl_init') && false !== ($curlHandle = \curl_init())) { |
||||||
101 | \curl_setopt($curlHandle, \CURLOPT_URL, $infoReleasesUrl); |
||||||
102 | \curl_setopt($curlHandle, \CURLOPT_RETURNTRANSFER, true); |
||||||
103 | \curl_setopt($curlHandle, \CURLOPT_SSL_VERIFYPEER, true); //TODO: how to avoid an error when 'Peer's Certificate issuer is not recognized' |
||||||
104 | \curl_setopt($curlHandle, \CURLOPT_HTTPHEADER, ["User-Agent:Publisher\r\n"]); |
||||||
105 | $curlReturn = \curl_exec($curlHandle); |
||||||
106 | if (false === $curlReturn) { |
||||||
107 | \trigger_error(\curl_error($curlHandle)); |
||||||
108 | } elseif (false !== \strpos($curlReturn, 'Not Found')) { |
||||||
0 ignored issues
–
show
It seems like
$curlReturn can also be of type true ; however, parameter $haystack of strpos() does only seem to accept string , maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
109 | \trigger_error('Repository Not Found: ' . $infoReleasesUrl); |
||||||
110 | } else { |
||||||
111 | $file = json_decode($curlReturn, false); |
||||||
0 ignored issues
–
show
It seems like
$curlReturn can also be of type true ; however, parameter $json of json_decode() does only seem to accept string , maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
112 | $latestVersionLink = \sprintf("https://github.com/$repository/archive/%s.zip", $file ? \reset($file)->tag_name : $default); |
||||||
113 | $latestVersion = $file[0]->tag_name; |
||||||
114 | $prerelease = $file[0]->prerelease; |
||||||
115 | if ('master' !== $latestVersionLink) { |
||||||
116 | $update = \constant('CO_' . $moduleDirNameUpper . '_' . 'NEW_VERSION') . $latestVersion; |
||||||
117 | } |
||||||
118 | //"PHP-standardized" version |
||||||
119 | $latestVersion = mb_strtolower($latestVersion); |
||||||
120 | if (false !== mb_strpos($latestVersion, 'final')) { |
||||||
121 | $latestVersion = \str_replace('_', '', mb_strtolower($latestVersion)); |
||||||
122 | $latestVersion = \str_replace('final', '', mb_strtolower($latestVersion)); |
||||||
123 | } |
||||||
124 | $moduleVersion = ($helper->getModule()->getInfo('version') . '_' . $helper->getModule()->getInfo('module_status')); |
||||||
0 ignored issues
–
show
Are you sure
$helper->getModule()->getInfo('module_status') of type array|string can be used in concatenation ?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
Are you sure
$helper->getModule()->getInfo('version') of type array|string can be used in concatenation ?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
125 | //"PHP-standardized" version |
||||||
126 | $moduleVersion = \str_replace(' ', '', mb_strtolower($moduleVersion)); |
||||||
127 | // $moduleVersion = '1.0'; //for testing only |
||||||
128 | // $moduleDirName = 'publisher'; //for testing only |
||||||
129 | if (!$prerelease && \version_compare($moduleVersion, $latestVersion, '<')) { |
||||||
130 | $ret = []; |
||||||
131 | $ret[] = $update; |
||||||
132 | $ret[] = $latestVersionLink; |
||||||
133 | } |
||||||
134 | } |
||||||
135 | \curl_close($curlHandle); |
||||||
136 | } |
||||||
137 | } |
||||||
138 | return $ret; |
||||||
139 | } |
||||||
140 | } |
||||||
141 |